diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2005-10-03 23:43:45 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2005-10-03 23:43:45 +0000 |
| commit | 31d276d0ed6d26f633a6348c8f9602bfb660650e (patch) | |
| tree | c466b32c695da4fbeb6a7579050d4b7b96a2cc76 /src/backend/commands/copy.c | |
| parent | 4082f5e34f20872ab150b5707bc6d662cbe9b757 (diff) | |
| download | postgresql-REL7_4_9.tar.gz postgresql-REL7_4_9.zip | |
COPY's test for read-only transaction was backward; it prohibited COPY TOREL7_4_9
where it should prohibit COPY FROM. Found by Alon Goldshuv.
Diffstat (limited to 'src/backend/commands/copy.c')
| -rw-r--r-- | src/backend/commands/copy.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/backend/commands/copy.c b/src/backend/commands/copy.c index 95f860200c0..c4e261549a2 100644 --- a/src/backend/commands/copy.c +++ b/src/backend/commands/copy.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/commands/copy.c,v 1.213.2.1 2004/01/18 02:15:57 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/commands/copy.c,v 1.213.2.2 2005/10/03 23:43:45 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -752,7 +752,8 @@ DoCopy(const CopyStmt *stmt) rel = heap_openrv(relation, (is_from ? RowExclusiveLock : AccessShareLock)); /* check read-only transaction */ - if (XactReadOnly && !is_from && !isTempNamespace(RelationGetNamespace(rel))) + if (XactReadOnly && is_from && + !isTempNamespace(RelationGetNamespace(rel))) ereport(ERROR, (errcode(ERRCODE_READ_ONLY_SQL_TRANSACTION), errmsg("transaction is read-only"))); |