diff options
| author | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2011-10-08 11:17:40 +0300 |
|---|---|---|
| committer | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2011-10-08 11:20:59 +0300 |
| commit | ad885e2677754c8adbfa847d20812ce6b7594eb6 (patch) | |
| tree | 9e67f6c334575ad8e3f8dd51dbffd12f378224d6 /src/backend/parser/parse_expr.c | |
| parent | 1ae092ae14b0eaf5a2d3ae8908d0edfd3cc39dd4 (diff) | |
| download | postgresql-ad885e2677754c8adbfa847d20812ce6b7594eb6.tar.gz postgresql-ad885e2677754c8adbfa847d20812ce6b7594eb6.zip | |
Don't let transform_null_equals=on affect CASE foo WHEN NULL ... constructs.
transform_null_equals is only supposed to affect "foo = NULL" expressions
given directly by the user, not the internal "foo = NULL" expression
generated from CASE-WHEN.
This fixes bug #6242, reported by Sergey. Backpatch to all supported
branches.
Diffstat (limited to 'src/backend/parser/parse_expr.c')
| -rw-r--r-- | src/backend/parser/parse_expr.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/src/backend/parser/parse_expr.c b/src/backend/parser/parse_expr.c index 65d03adc494..79328c99797 100644 --- a/src/backend/parser/parse_expr.c +++ b/src/backend/parser/parse_expr.c @@ -836,12 +836,15 @@ transformAExprOp(ParseState *pstate, A_Expr *a) /* * Special-case "foo = NULL" and "NULL = foo" for compatibility with * standards-broken products (like Microsoft's). Turn these into IS NULL - * exprs. + * exprs. (If either side is a CaseTestExpr, then the expression was + * generated internally from a CASE-WHEN expression, and + * transform_null_equals does not apply.) */ if (Transform_null_equals && list_length(a->name) == 1 && strcmp(strVal(linitial(a->name)), "=") == 0 && - (exprIsNullConstant(lexpr) || exprIsNullConstant(rexpr))) + (exprIsNullConstant(lexpr) || exprIsNullConstant(rexpr)) && + (!IsA(lexpr, CaseTestExpr) && !IsA(rexpr, CaseTestExpr))) { NullTest *n = makeNode(NullTest); |