Fix silly mistake in expand_indexqual_rowcompare --- in converting a forboth()

into an iteration over three parallel lists, I had accidentally put the lnext
steps outside the loop.  Sigh.  Per bug #3938.

diff --git a/src/backend/optimizer/path/indxpath.c b/src/backend/optimizer/path/indxpath.c
index 2ed794fba00ece7c73a2c0841c1bcdeb552c4841..e07343cba99e9e4198b0407e59bfc06bf2ac95c9 100644 (file)
--- a/src/backend/optimizer/path/indxpath.c
+++ b/src/backend/optimizer/path/indxpath.c
@@ -2619,9 +2619,9 @@ expand_indexqual_rowcompare(RestrictInfo *rinfo,
                                                 op_strategy, lefttype, righttype, opfam);
                        }
                        new_ops = lappend_oid(new_ops, expr_op);
+                       lefttypes_cell = lnext(lefttypes_cell);
+                       righttypes_cell = lnext(righttypes_cell);
                }
-               lefttypes_cell = lnext(lefttypes_cell);
-               righttypes_cell = lnext(righttypes_cell);
        }
 
        /* If we have more than one matching col, create a subset rowcompare */