Commit8064a49

committed

get_relattval() should treat a NULL constant as a non-constant expression,

since it has no way to indicate to its caller that the constant isactually NULL. This prevents coredump in cases like WHERE textfield < null::text;

1 parent3e51872 commit8064a49Copy full SHA for 8064a49

File tree

-2

lines changed

-2

lines changed

Lines changed: 3 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@`
`8`	`8`	`*`
`9`	`9`	`*`
`10`	`10`	`* IDENTIFICATION`
`11`		`- * $Header: /cvsroot/pgsql/src/backend/optimizer/util/clauses.c,v 1.65 2000/04/12 17:15:24 momjian Exp $`
	`11`	`+ * $Header: /cvsroot/pgsql/src/backend/optimizer/util/clauses.c,v 1.66 2000/04/16 01:55:45 tgl Exp $`
`12`	`12`	`*`
`13`	`13`	`* HISTORY`
`14`	`14`	`* AUTHORDATEMAJOR EVENT`
`@@ -790,7 +790,8 @@ get_relattval(Node *clause,`
`790`	`790`
`791`	`791`	`other= (*flag==0) ?left :right;`
`792`	`792`
`793`		`-if (IsA(other,Const))`
	`793`	`+if (IsA(other,Const)&&`
	`794`	`+!((Const*)other)->constisnull)`
`794`	`795`	`{`
`795`	`796`	`constval= ((Const)other)->constvalue;`
`796`	`797`	`*flag \|=SEL_CONSTANT;`

Comments

(0)