pgsql: Remove useless pg_strdup() operations.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Remove useless pg_strdup() operations.
Date: 2016-09-04 16:34:04
Message-ID: E1bgaMu-0000ad-9Q@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

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.)

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/a2d75b67bccd24d17e328360080e4877d23bc369

Modified Files
--------------
src/test/regress/pg_regress.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2016-09-04 17:20:06 pgsql: Update release notes to mention need for ALTER EXTENSION UPDATE.
Previous Message Heikki Linnakangas 2016-09-04 12:02:12 pgsql: Clarify the new Red-Black post-order traversal code a bit.