On Wed, Jul 24, 2019 at 10:36:42AM -0400, Asmaa Mnebhi wrote: > Signed-off-by: Asmaa Mnebhi <as...@mellanox.com>
The patch is, of course, fine, but you should add some info about how it was found and a Reported-by: tag. Thanks, -corey > --- > drivers/char/ipmi/ipmb_dev_int.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/char/ipmi/ipmb_dev_int.c > b/drivers/char/ipmi/ipmb_dev_int.c > index 5720433..285e0b8 100644 > --- a/drivers/char/ipmi/ipmb_dev_int.c > +++ b/drivers/char/ipmi/ipmb_dev_int.c > @@ -76,7 +76,7 @@ static ssize_t ipmb_read(struct file *file, char __user > *buf, size_t count, > struct ipmb_dev *ipmb_dev = to_ipmb_dev(file); > struct ipmb_request_elem *queue_elem; > struct ipmb_msg msg; > - ssize_t ret; > + ssize_t ret = 0; > > memset(&msg, 0, sizeof(msg)); > > -- > 2.1.2 >
