[PATCH] Fix coverity issue in ctdb
Martin Schwenke
martin at meltin.net
Fri Sep 8 11:51:41 UTC 2017
On Fri, 8 Sep 2017 16:51:44 +1000, Martin Schwenke via samba-technical
<samba-technical at lists.samba.org> wrote:
> 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:
> [...]
> > >
> > > 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>
... and pushed.
peace & happiness,
martin
More information about the samba-technical
mailing list