Unneeded NULL-pointer check in FreeSpaceMapTruncateRel
Hi all,
In the category of nitpicky-code-style-issues, FreeSpaceMapTruncateRel
is doing a NULL-pointer check for something that has been dereferenced
on all the code paths leading to this check.
(Yes, that's not interesting for common humans, Coverity sees things
based on correctness).
Regards,
--
Michael
Attachments:
20150630_freespace_ptrcheck.patchtext/x-patch; charset=US-ASCII; name=20150630_freespace_ptrcheck.patchDownload+1-2
Hi,
On 2015-06-30 21:53:07 +0900, Michael Paquier wrote:
In the category of nitpicky-code-style-issues, FreeSpaceMapTruncateRel
is doing a NULL-pointer check for something that has been dereferenced
on all the code paths leading to this check.
(Yes, that's not interesting for common humans, Coverity sees things
based on correctness).
Can you, in the future, batch these together into one thread, perhaps
with one message below an introductory one for each patch? Unless
they'll get applied immediately it'll be hard to follow the different
threads.
Greetings,
Andres Freund
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
On Tue, Jun 30, 2015 at 9:57 PM, Andres Freund <andres@anarazel.de> wrote:
On 2015-06-30 21:53:07 +0900, Michael Paquier wrote:
In the category of nitpicky-code-style-issues, FreeSpaceMapTruncateRel
is doing a NULL-pointer check for something that has been dereferenced
on all the code paths leading to this check.
(Yes, that's not interesting for common humans, Coverity sees things
based on correctness).Can you, in the future, batch these together into one thread, perhaps
with one message below an introductory one for each patch? Unless
they'll get applied immediately it'll be hard to follow the different
threads.
I'll try.
--
Michael
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers