]> 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)
commit8d9e4d13dd6edd4165f1f82f06e07c6fccf1c0ea
tree7df52c632324549ea8806e4ead8cd09a206362dd
parent9e4caa2748f469d3f37a68f5012a2993dad0db10
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