]> granicus.if.org Git - postgresql/commit
Fix bogus list-iteration code in pg_regress.c, affecting ecpg tests only.
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 30 Apr 2018 01:56:28 +0000 (21:56 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 30 Apr 2018 01:56:28 +0000 (21:56 -0400)
commit938c6f42d8524ccc8828aa5a1a46578e4226562d
tree4823623d8cb881d755871fe6cce02b11e500d7cd
parentd6ec3d2375ac3b94d2480c01d7ee7739b236642c
Fix bogus list-iteration code in pg_regress.c, affecting ecpg tests only.

While looking at a recent buildfarm failure in the ecpg tests, I wondered
why the pg_regress output claimed the stderr part of the test failed, when
the regression diffs were clearly for the stdout part.  Looking into it,
the reason is that pg_regress.c's logic for iterating over three parallel
lists is wrong, and has been wrong since it was written: it advances the
"tag" pointer at a different place in the loop than the other two pointers.
Fix that.
src/test/regress/pg_regress.c