Hello there,

linux-4.16-rc1/drivers/misc/ocxl/file.c:320]: (style) Checking if unsigned 
variable 'used' is less than zero.

Source code is

       used = append_xsl_error(ctx, &header, buf + sizeof(header));
        if (used < 0)
            return used;

Suggest put return value from function into signed variable, sanity check it,

then assign it to an unsigned variable.

Also, use of the gcc compiler flag -Wtype-limits will show up this kind of 
problem in future.


David Binderman

Reply via email to