diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2004-08-07 17:40:49 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2004-08-07 17:40:49 +0000 |
| commit | b2b5656362098a262dbfea7d6cbf3d5a3a083614 (patch) | |
| tree | 32b7ce0b4bf2ed6f41fa4c78ba466e8bead473c7 /src/backend/rewrite/rewriteHandler.c | |
| parent | 752089ea41a3cf73573cc45f544c409b3c4a3e1a (diff) | |
| download | postgresql-b2b5656362098a262dbfea7d6cbf3d5a3a083614.tar.gz postgresql-b2b5656362098a262dbfea7d6cbf3d5a3a083614.zip | |
Don't try to rewrite NEW references in a utility statement in a rule.
There won't be any, and in fact there won't even be an RTE for NEW,
which was leading to a core dump in CVS tip. 7.4 and earlier manage
not to crash when applying ResolveNew in this scenario, but I think
it was just good fortune that they didn't. Per report from
Bernd Helmle.
Diffstat (limited to 'src/backend/rewrite/rewriteHandler.c')
| -rw-r--r-- | src/backend/rewrite/rewriteHandler.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/backend/rewrite/rewriteHandler.c b/src/backend/rewrite/rewriteHandler.c index bccc369dc09..fadc7416ad9 100644 --- a/src/backend/rewrite/rewriteHandler.c +++ b/src/backend/rewrite/rewriteHandler.c @@ -7,7 +7,7 @@ * Portions Copyright (c) 1994, Regents of the University of California * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/rewrite/rewriteHandler.c,v 1.140 2004/06/16 01:26:46 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/rewrite/rewriteHandler.c,v 1.141 2004/08/07 17:40:49 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -208,7 +208,8 @@ rewriteRuleAction(Query *parsetree, * apply it to sub_action; we have to remember to update the sublink * inside rule_action, too. */ - if (event == CMD_INSERT || event == CMD_UPDATE) + if ((event == CMD_INSERT || event == CMD_UPDATE) && + sub_action->commandType != CMD_UTILITY) { sub_action = (Query *) ResolveNew((Node *) sub_action, new_varno, |