]> 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)
commitd86e9e21058cfd53f0b6a3cabee68446aa255775
tree5d5ea487982ebb456d2b46154129b2b897aace9f
parent217d8f3a19aeae6a221c61487f1758a53dda31c8
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