Re: [PATCH] drm: Check if the allocation has succeeded before dereferencing newmode

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Mar 3, 2014 at 6:59 PM, Damien Lespiau <damien.lespiau@xxxxxxxxx> wrote:
> We allocate memory in drm_display_mode_from_vic_index() and use it
> without checking the pointer is valid. Fix that.
>
> Signed-off-by: Damien Lespiau <damien.lespiau@xxxxxxxxx>

Reviewed-by: Alex Deucher <alexander.deucher@xxxxxxx>

> ---
>  drivers/gpu/drm/drm_edid.c | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
> index f8d8a1d..f3cde90 100644
> --- a/drivers/gpu/drm/drm_edid.c
> +++ b/drivers/gpu/drm/drm_edid.c
> @@ -2580,6 +2580,9 @@ drm_display_mode_from_vic_index(struct drm_connector *connector,
>                 return NULL;
>
>         newmode = drm_mode_duplicate(dev, &edid_cea_modes[cea_mode]);
> +       if (!newmode)
> +               return NULL;
> +
>         newmode->vrefresh = 0;
>
>         return newmode;
> --
> 1.8.3.1
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux