[PATCH] Fix 2 Coverity findings
Martin Schwenke
martin at meltin.net
Tue Feb 16 12:23:33 UTC 2016
Hi Volker,
On Tue, 16 Feb 2016 11:28:49 +0100, Volker Lendecke
<Volker.Lendecke at SerNet.DE> wrote:
> Review appreciated!
OK, it took me a little while to get my head around this one. There
used to be a level of indirection there and now it is gone. So now I'm
taking the address of something and checking that it is non-NULL. Yeah,
that's a bit stupid. :-)
Hmmm... instead you're checking that the pointer to the array can't be
NULL. That can never happen in the current code (famous last words!),
but it seems like a reasonable improvement.
However, I was confused by the combination of the improvement and
removal of the useless code. It could be clearer as 2 separate patches.
Either way...
Reviewed-by: Martin Schwenke <martin at meltin.net>
Please push with something else? Bed time... at least I understand
it now and it won't keep me awake... :-)
Thanks...
peace & happiness,
martin
More information about the samba-technical
mailing list