Re: [PATCH net v3 1/2] net: usb: pegasus: Check the return value of get_geristers() and friends;

On Tue, 3 Aug 2021 20:25:23 +0300 Petko Manolov petko.manolov@konsulko.com wrote:

From: Petko Manolov petkan@nucleusys.com

Certain call sites of get_geristers() did not do proper error handling. This could be a problem as get_geristers() typically return the data via pointer to a buffer. If an error occurred the code is carelessly manipulating the wrong data.

Signed-off-by: Petko Manolov petkan@nucleusys.com

drivers/net/usb/pegasus.c | 108 ++++++++++++++++++++++++++------------ 1 file changed, 75 insertions(+), 33 deletions(-)

All's good! Thank you :)

Reviewed-by: Pavel Skripkin paskripkin@gmail.com

With regards, Pavel Skripkin