So reviewing the libXi security fix made me double take, there was a
few lines like

            if (sizeof(xDeviceAbsCalibState) > nbytes)
                goto out;
            size += sizeof(XDeviceAbsCalibState);

Now either size is always 0 going into this, in which case why += or
the check before is wrong.

Looking at the code size seems to be always 0, so why the +=, was
there a possibility of this looping at some point?

Don't think there's a bug it was just something that made me have to
check twice.

Dave.
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to