[PATCH - COVERITY] Small patch for talloc/testsuite: Fix CID 1291641 - Logically dead code

Andreas Schneider asn at samba.org
Tue Apr 26 18:41:32 UTC 2016


On Tuesday, 26 April 2016 11:34:58 CEST Jeremy Allison wrote:
> On Tue, Apr 26, 2016 at 01:09:23PM +0200, Robin Hack wrote:
> > Hello.
> > 
> > I don't think that snprintf() can fail in this case but proper check is
> > better than nothing.
> > 
> > Sorry... not another badlock yet ;P.
> 
> Don't even *JOKE* about that :-). It really wasn't funny to
> deal with :-).
> 
> LGTM. Second Team reviewer ?

RB+

-- 
Andreas Schneider                   GPG-ID: CC014E3D
Samba Team                             asn at samba.org
www.samba.org



More information about the samba-technical mailing list