Messages in this thread | | | From | Jani Nikula <> | Subject | Re: [PATCH v2] i915: correctly handling failed allocation | Date | Mon, 04 Jan 2016 16:06:03 +0200 |
| |
On Wed, 30 Dec 2015, Insu Yun <wuninsu@gmail.com> wrote: > Since devm_kzalloc can be failed, it needs to be checked > if not, NULL dereference could be happened. > > Signed-off-by: Insu Yun <wuninsu@gmail.com>
Pushed to drm-intel-next-queued, thanks for the patch.
BR, Jani.
> --- > drivers/gpu/drm/i915/intel_dsi_panel_vbt.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c b/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c > index a5e99ac..aa1f7bc 100644 > --- a/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c > +++ b/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c > @@ -666,6 +666,8 @@ struct drm_panel *vbt_panel_init(struct intel_dsi *intel_dsi, u16 panel_id) > > /* This is cheating a bit with the cleanup. */ > vbt_panel = devm_kzalloc(dev->dev, sizeof(*vbt_panel), GFP_KERNEL); > + if (!vbt_panel) > + return NULL; > > vbt_panel->intel_dsi = intel_dsi; > drm_panel_init(&vbt_panel->panel);
-- Jani Nikula, Intel Open Source Technology Center
| |