Messages in this thread | | | From | "james qian wang (Arm Technology China)" <> | Subject | Re: drm/komeda: prevent memory leak in komeda_wb_connector_add | Date | Fri, 27 Sep 2019 08:08:59 +0000 |
| |
On Tue, Sep 24, 2019 at 11:30:30PM -0500, Navid Emamdoost wrote: > In komeda_wb_connector_add if drm_writeback_connector_init fails the > allocated memory for kwb_conn should be released. > > Signed-off-by: Navid Emamdoost <navid.emamdoost@gmail.com> > --- > drivers/gpu/drm/arm/display/komeda/komeda_wb_connector.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_wb_connector.c b/drivers/gpu/drm/arm/display/komeda/komeda_wb_connector.c > index 2851cac94d86..75133f967fdb 100644 > --- a/drivers/gpu/drm/arm/display/komeda/komeda_wb_connector.c > +++ b/drivers/gpu/drm/arm/display/komeda/komeda_wb_connector.c > @@ -166,8 +166,10 @@ static int komeda_wb_connector_add(struct komeda_kms_dev *kms, > &komeda_wb_encoder_helper_funcs, > formats, n_formats); > komeda_put_fourcc_list(formats); > - if (err) > + if (err) { > + kfree(kwb_conn); > return err; > + }
Thank you for the fix.
Will push it to drm-misc-fixes
Reviewed-by: James Qian Wang (Arm Technology China) <james.qian.wang@arm.com>
> > drm_connector_helper_add(&wb_conn->base, &komeda_wb_conn_helper_funcs); >
| |