Re: [PATCH] scsi: fcoe: add missed kfree() in an error path

2020-07-09 Thread Jing Xiangfeng
On 2020/7/9 15:22, Markus Elfring wrote: fcoe_fdmi_info() misses to call kfree() in an error path. Add the missed function call to fix it. I suggest to use an additional jump target for the completion of the desired exception handling. … +++ b/drivers/scsi/fcoe/fcoe.c @@ -830,6 +830,7 @@

Re: [PATCH] scsi: fcoe: add missed kfree() in an error path

2020-07-09 Thread Markus Elfring
>>> fcoe_fdmi_info() misses to call kfree() in an error path. >>> Add the missed function call to fix it. >> >> I suggest to use an additional jump target for the completion >> of the desired exception handling. >> >> >> … >>> +++ b/drivers/scsi/fcoe/fcoe.c >>> @@ -830,6 +830,7 @@ static void fcoe_

Re: [PATCH] scsi: fcoe: add missed kfree() in an error path

2020-07-08 Thread Jing Xiangfeng
On 2020/7/7 19:38, Markus Elfring wrote: fcoe_fdmi_info() misses to call kfree() in an error path. Add the missed function call to fix it. I suggest to use an additional jump target for the completion of the desired exception handling. … +++ b/drivers/scsi/fcoe/fcoe.c @@ -830,6 +830,7 @@

Re: [PATCH] scsi: fcoe: add missed kfree() in an error path

2020-07-07 Thread Markus Elfring
> fcoe_fdmi_info() misses to call kfree() in an error path. > Add the missed function call to fix it. I suggest to use an additional jump target for the completion of the desired exception handling. … > +++ b/drivers/scsi/fcoe/fcoe.c > @@ -830,6 +830,7 @@ static void fcoe_fdmi_info(struct fc_lpo