RE: [PATCH v2 0/4] libusbg: Fix potential issues found with coverity

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

I have just go through you patches once again and test it. Each commit
is able to build. I have also tested it on my board and didn't find any
regression. Please update commit messages according to my comments and
patches will look good to me.

--
BR's
Krzysztof Opasiak

> -----Original Message-----
> From: linux-usb-owner@xxxxxxxxxxxxxxx [mailto:linux-usb-
> owner@xxxxxxxxxxxxxxx] On Behalf Of Philippe De Swert
> Sent: Tuesday, May 13, 2014 4:13 PM
> To: Krzysztof Opasiak; linux-usb@xxxxxxxxxxxxxxx
> Cc: mporter@xxxxxxxxxx; Philippe De Swert
> Subject: [PATCH v2 0/4] libusbg: Fix potential issues found with
> coverity
> 
> This patch series fixes a number of issues found with coverity in
> libusbg
> A pull request has also be made:
> https://github.com/libusbg/libusbg/pull/4
> 
> This is the second version where we use a different fix for the
> readlink,
> actually use the right USB_MAX_STR_LENGTH define. I also fixed an
> issue
> I missed first time around.
> 
> Philippe De Swert (4):
>   libusbg: Fix readlink/buffer overrun issue. CID#56130, CID#56129
>   libusbg: Fix buffer overrun issue. CID#56128
>   libusbg: Do not try to dereference func when it is NULL.
> CID#56127
>   libusbg: Do not dereference usb config attributes when they are
> NULL.
>     CID#56126
> 
>  src/usbg.c | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
> 
> --
> 1.8.1.2
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-
> usb" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html


--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux