[PATCH] Fix coverity issue in ctdb

Martin Schwenke martin at meltin.net
Fri Sep 8 06:51:44 UTC 2017


On Thu, 7 Sep 2017 11:43:49 +1000, Amitay Isaacs via samba-technical
<samba-technical at lists.samba.org> wrote:

> On Wed, Sep 6, 2017 at 4:03 PM, Volker Lendecke <Volker.Lendecke at sernet.de>
> wrote:
> 
> > On Wed, Sep 06, 2017 at 11:34:32AM +1000, Amitay Isaacs via
> > samba-technical wrote:  
> > > Hi,
> > >
> > > Please review and push.  
> >
> > Are you sure that fixes 1414746? I don't fully understand the CID, but
> > my impression is that the the overflow is not from assigning ssize_t
> > to int.
> >
> >  
> You are correct. It's not the fix for actual CID.  Actual CID is the
> overflow of db_flags.
> 
> Here are the modified patches.
> 
> - first patch is fixing the overflow (int --> ssize_t)
> - second patch should fix the CID

Reviewed-by: Martin Schwenke <martin at meltin.net>

peace & happiness,
martin



More information about the samba-technical mailing list