> Subject: Re: [PATCH] drm/fb-helper: Do the mode_set.connectors ZERO_SIZE_PTR > check > > On Thu, 06 Mar 2014, Xiubo Li <li.xi...@freescale.com> wrote: > > Since we cannot make sure the 'max_conn_count' will always be none > > zero from the users, and then if max_conn_count equals to zero, the > > kcalloc() will return ZERO_SIZE_PTR, which equals to ((void *)16). > > > > So this patch fix this via doing the zero pionter check of it. > > Please just add a check for max_conn_count == 0 up front and handle it. > > BR, > Jani. >
Yes, that's one better choice. See the next version please. Thanks very much! -- Best Regards, Xiubo > > > > > Signed-off-by: Xiubo Li <li.xi...@freescale.com> > > --- > > drivers/gpu/drm/drm_fb_helper.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/drm_fb_helper.c > b/drivers/gpu/drm/drm_fb_helper.c > > index 3d13ca6e2..c6680ef 100644 > > --- a/drivers/gpu/drm/drm_fb_helper.c > > +++ b/drivers/gpu/drm/drm_fb_helper.c > > @@ -536,7 +536,7 @@ int drm_fb_helper_init(struct drm_device *dev, > > sizeof(struct drm_connector *), > > GFP_KERNEL); > > > > - if (!fb_helper->crtc_info[i].mode_set.connectors) > > + if > > (ZERO_OR_NULL_PTR(fb_helper->crtc_info[i].mode_set.connectors)) > > goto out_free; > > fb_helper->crtc_info[i].mode_set.num_connectors = 0; > > } > > -- > > 1.8.4 > > > > > > _______________________________________________ > > dri-devel mailing list > > dri-de...@lists.freedesktop.org > > http://lists.freedesktop.org/mailman/listinfo/dri-devel > > -- > Jani Nikula, Intel Open Source Technology Center > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
