Skip to content

Commit aadfd6e

Browse files
committed
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.
1 parent b6959ee commit aadfd6e

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

src/backend/parser/parse_expr.c

+5-2
Original file line numberDiff line numberDiff line change
@@ -587,12 +587,15 @@ transformAExprOp(ParseState *pstate, A_Expr *a)
587587
/*
588588
* Special-case "foo = NULL" and "NULL = foo" for compatibility with
589589
* standards-broken products (like Microsoft's). Turn these into IS NULL
590-
* exprs.
590+
* exprs. (If either side is a CaseTestExpr, then the expression was
591+
* generated internally from a CASE-WHEN expression, and
592+
* transform_null_equals does not apply.)
591593
*/
592594
if (Transform_null_equals &&
593595
list_length(a->name) == 1 &&
594596
strcmp(strVal(linitial(a->name)), "=") == 0 &&
595-
(exprIsNullConstant(lexpr) || exprIsNullConstant(rexpr)))
597+
(exprIsNullConstant(lexpr) || exprIsNullConstant(rexpr)) &&
598+
(!IsA(lexpr, CaseTestExpr) && !IsA(rexpr, CaseTestExpr)))
596599
{
597600
NullTest *n = makeNode(NullTest);
598601

0 commit comments

Comments
 (0)