pgsql: Remove useless pg_strdup() operations. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Remove useless pg_strdup() operations.
Date
Msg-id E1bgaMu-0000ad-9Q@gemulon.postgresql.org
Whole thread Raw
List 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(-)


pgsql-committers by date:

Previous
From: Heikki Linnakangas
Date:
Subject: pgsql: Clarify the new Red-Black post-order traversal code a bit.
Next
From: Tom Lane
Date:
Subject: pgsql: Update release notes to mention need for ALTER EXTENSION UPDATE.