On 1/14/20 3:43 PM, Igor Druzhinin wrote: > fc_disc_gpn_id_resp() should be the last function using it so free it > here to avoid memory leak. > > Signed-off-by: Igor Druzhinin <igor.druzhinin@xxxxxxxxxx> > --- > drivers/scsi/libfc/fc_disc.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/scsi/libfc/fc_disc.c b/drivers/scsi/libfc/fc_disc.c > index 9c5f7c9..2b865c6 100644 > --- a/drivers/scsi/libfc/fc_disc.c > +++ b/drivers/scsi/libfc/fc_disc.c > @@ -628,6 +628,8 @@ static void fc_disc_gpn_id_resp(struct fc_seq *sp, struct fc_frame *fp, > } > out: > kref_put(&rdata->kref, fc_rport_destroy); > + if (!IS_ERR(fp)) > + fc_frame_free(fp); > } > > /** > Reviewed-by: Hannes Reinecke <hare@xxxxxxx> Cheers, Hannes -- Dr. Hannes Reinecke Kernel Storage Architect hare@xxxxxxx +49 911 74053 688 SUSE Software Solutions Germany GmbH, Maxfeldstr. 5, 90409 Nürnberg HRB 36809 (AG Nürnberg), GF: Felix Imendörffer
