]> 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)
commit8109a3c14486ff204b27a9c6e6df6a79c0735219
tree5af101c5eea0682e1a31df9a58a1a9b1c3e90e96
parent59c2df3ae8947c4c06217d1b330cfa529f480e17
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