On Thu, Sep 07, 2017 at 01:54:35PM +0200, Christoph Hellwig wrote: > The job structure is allocated as part of the request, so we should not > free it in the error path of bsg_prepare_job. > > Signed-off-by: Christoph Hellwig <[email protected]> > Cc: [email protected] > --- > block/bsg-lib.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/block/bsg-lib.c b/block/bsg-lib.c > index dd56d7460cb9..c587c71d78af 100644 > --- a/block/bsg-lib.c > +++ b/block/bsg-lib.c > @@ -154,7 +154,6 @@ static int bsg_prepare_job(struct device *dev, struct > request *req) > failjob_rls_rqst_payload: > kfree(job->request_payload.sg_list); > failjob_rls_job: > - kfree(job); > return -ENOMEM; > } > > -- > 2.11.0 >
Reviewed-by: Ming Lei <[email protected]> -- Ming
