Remove useless pg_strdup() operations.
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 4 Sep 2016 16:33:58 +0000 (12:33 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 4 Sep 2016 16:33:58 +0000 (12:33 -0400)
commita2d75b67bccd24d17e328360080e4877d23bc369
tree00ea7b2868a852fc7b8a10b0e026189afcfc95cd
parente21db14b8a6696a2b704b89df9c4be9cd0ea8a33
Remove useless pg_strdup() operations.

split_to_stringlist() doesn't modify its first argument nor expect it
to remain valid after exit, so there's no need to duplicate the optarg
string at the call sites.  Per Coverity.  (This has been wrong all along,
but commit 052cc223d changed the useless calls from "strdup" to
"pg_strdup", which apparently made Coverity think it's a new bug.
It's not, but it's also not worth back-patching.)
src/test/regress/pg_regress.c