]> 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:27 +0000 (21:56 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 30 Apr 2018 01:56:27 +0000 (21:56 -0400)
commitc5e46c7c3b9333616de9946f8e63df1576a19800
treead64ec6765b34d2d579d81445d4d84134b7ebf92
parenteb16011f4c08a8c2ddfbf0637b862ee1918dd9b2
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