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