Defend against 'update oid'. Someday we might want to support

that, but it'd be a New Feature, wouldn't it ... in the meantime,
avoiding a backend crash seems worthwhile.

diff --git a/src/backend/parser/parse_target.c b/src/backend/parser/parse_target.c
index b2a2af6..b237605 100644 (file)
--- a/src/backend/parser/parse_target.c
+++ b/src/backend/parser/parse_target.c
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/parser/parse_target.c,v 1.34 1999/02/03 21:16:59 momjian Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/parser/parse_target.c,v 1.35 1999/04/29 03:01:50 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -216,6 +216,8 @@ MakeTargetEntryExpr(ParseState *pstate,
                rd = pstate->p_target_relation;
                Assert(rd != NULL);
                resdomno = attnameAttNum(rd, colname);
+               if (resdomno <= 0)
+                       elog(ERROR, "Cannot assign to system attribute '%s'", colname);
                attrisset = attnameIsSet(rd, colname);
                attrtype = attnumTypeId(rd, resdomno);
                if ((arrayRef != NIL) && (lfirst(arrayRef) == NIL))