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

From: Pavel Skripkin <[email-masked]>
Date: Tue, 3 Aug 2021 20:49:46 +0300

On Tue,  3 Aug 2021 20:25:23 +0300
Petko Manolov <[email-masked]> wrote:

> From: Petko Manolov <[email-masked]>
> 
> 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 <[email-masked]>
> ---
>  drivers/net/usb/pegasus.c | 108
> ++++++++++++++++++++++++++------------ 1 file changed, 75
> insertions(+), 33 deletions(-)
> 

All's good! Thank you :)

Reviewed-by: Pavel Skripkin <[email-masked]>

With regards,
Pavel Skripkin