[PATCH] drm: check for NULL parameter in exported drm_get_format_name() function.

2016-11-22 Thread Ville Syrjälä
On Tue, Nov 22, 2016 at 12:35:53PM -0500, Rob Clark wrote: > On Tue, Nov 22, 2016 at 12:31 PM, Ville Syrjälä > wrote: > > On Tue, Nov 22, 2016 at 12:23:59PM -0500, Rob Clark wrote: > >> On Tue, Nov 22, 2016 at 11:50 AM, Ville Syrjälä > >> wrote: > >> > On Tue, Nov 22, 2016 at 04:41:06PM +

[PATCH] drm: check for NULL parameter in exported drm_get_format_name() function.

2016-11-22 Thread Ville Syrjälä
On Tue, Nov 22, 2016 at 12:23:59PM -0500, Rob Clark wrote: > On Tue, Nov 22, 2016 at 11:50 AM, Ville Syrjälä > wrote: > > On Tue, Nov 22, 2016 at 04:41:06PM +, Liviu Dudau wrote: > >> drm_get_format_name() de-references the buf parameter without checking > >> if the pointer was not NULL. Giv

[PATCH] drm: check for NULL parameter in exported drm_get_format_name() function.

2016-11-22 Thread Ville Syrjälä
On Tue, Nov 22, 2016 at 04:41:06PM +, Liviu Dudau wrote: > drm_get_format_name() de-references the buf parameter without checking > if the pointer was not NULL. Given that the function is EXPORT-ed, lets > sanitise the parameters before proceeding. > > Fixes: b3c11ac267d461d3d5 ("drm: move all

[PATCH] drm: check for NULL parameter in exported drm_get_format_name() function.

2016-11-22 Thread Liviu Dudau
On Tue, Nov 22, 2016 at 12:23:59PM -0500, Rob Clark wrote: > On Tue, Nov 22, 2016 at 11:50 AM, Ville Syrjälä > wrote: > > On Tue, Nov 22, 2016 at 04:41:06PM +, Liviu Dudau wrote: > >> drm_get_format_name() de-references the buf parameter without checking > >> if the pointer was not NULL. Giv

[PATCH] drm: check for NULL parameter in exported drm_get_format_name() function.

2016-11-22 Thread Liviu Dudau
On Tue, Nov 22, 2016 at 01:15:08PM -0500, Sean Paul wrote: > On Tue, Nov 22, 2016 at 1:06 PM, Ville Syrjälä > wrote: > > On Tue, Nov 22, 2016 at 12:35:53PM -0500, Rob Clark wrote: > >> On Tue, Nov 22, 2016 at 12:31 PM, Ville Syrjälä > >> wrote: > >> > On Tue, Nov 22, 2016 at 12:23:59PM -0500,

[PATCH] drm: check for NULL parameter in exported drm_get_format_name() function.

2016-11-22 Thread Rob Clark
On Tue, Nov 22, 2016 at 1:47 PM, Liviu Dudau wrote: > On Tue, Nov 22, 2016 at 01:15:08PM -0500, Sean Paul wrote: >> On Tue, Nov 22, 2016 at 1:06 PM, Ville Syrjälä >> wrote: >> > On Tue, Nov 22, 2016 at 12:35:53PM -0500, Rob Clark wrote: >> >> On Tue, Nov 22, 2016 at 12:31 PM, Ville Syrjälä >>

[PATCH] drm: check for NULL parameter in exported drm_get_format_name() function.

2016-11-22 Thread Liviu Dudau
drm_get_format_name() de-references the buf parameter without checking if the pointer was not NULL. Given that the function is EXPORT-ed, lets sanitise the parameters before proceeding. Fixes: b3c11ac267d461d3d5 ("drm: move allocation out of drm_get_format_name()) Cc: Eric Engestrom Cc: Rob Clark

[PATCH] drm: check for NULL parameter in exported drm_get_format_name() function.

2016-11-22 Thread Sean Paul
On Tue, Nov 22, 2016 at 1:06 PM, Ville Syrjälä wrote: > On Tue, Nov 22, 2016 at 12:35:53PM -0500, Rob Clark wrote: >> On Tue, Nov 22, 2016 at 12:31 PM, Ville Syrjälä >> wrote: >> > On Tue, Nov 22, 2016 at 12:23:59PM -0500, Rob Clark wrote: >> >> On Tue, Nov 22, 2016 at 11:50 AM, Ville Syrjäl

[PATCH] drm: check for NULL parameter in exported drm_get_format_name() function.

2016-11-22 Thread Rob Clark
On Tue, Nov 22, 2016 at 12:31 PM, Ville Syrjälä wrote: > On Tue, Nov 22, 2016 at 12:23:59PM -0500, Rob Clark wrote: >> On Tue, Nov 22, 2016 at 11:50 AM, Ville Syrjälä >> wrote: >> > On Tue, Nov 22, 2016 at 04:41:06PM +, Liviu Dudau wrote: >> >> drm_get_format_name() de-references the buf

[PATCH] drm: check for NULL parameter in exported drm_get_format_name() function.

2016-11-22 Thread Rob Clark
On Tue, Nov 22, 2016 at 11:50 AM, Ville Syrjälä wrote: > On Tue, Nov 22, 2016 at 04:41:06PM +, Liviu Dudau wrote: >> drm_get_format_name() de-references the buf parameter without checking >> if the pointer was not NULL. Given that the function is EXPORT-ed, lets >> sanitise the parameters be