[PATCH v4 3/3] drm: fix error routines in drm_open_helper

2013-07-02 Thread Chris Wilson
On Tue, Jul 02, 2013 at 09:53:28AM +0900, Seung-Woo Kim wrote: > From: YoungJun Cho > > There are missing parts to handle error in drm_open_helper(). > The priv->minor, assigned by idr_find() which can return NULL, > should be checked whether it is NULL or not before referencing it. > put_pid(),

[PATCH v4 3/3] drm: fix error routines in drm_open_helper

2013-07-02 Thread Seung-Woo Kim
From: YoungJun Cho There are missing parts to handle error in drm_open_helper(). The priv->minor, assigned by idr_find() which can return NULL, should be checked whether it is NULL or not before referencing it. put_pid(), drm_gem_release(), and drm_prime_destory_file_private() should be called wh

Re: [PATCH v4 3/3] drm: fix error routines in drm_open_helper

2013-07-02 Thread Chris Wilson
On Tue, Jul 02, 2013 at 09:53:28AM +0900, Seung-Woo Kim wrote: > From: YoungJun Cho > > There are missing parts to handle error in drm_open_helper(). > The priv->minor, assigned by idr_find() which can return NULL, > should be checked whether it is NULL or not before referencing it. > put_pid(),

[PATCH v4 3/3] drm: fix error routines in drm_open_helper

2013-07-01 Thread Seung-Woo Kim
From: YoungJun Cho There are missing parts to handle error in drm_open_helper(). The priv->minor, assigned by idr_find() which can return NULL, should be checked whether it is NULL or not before referencing it. put_pid(), drm_gem_release(), and drm_prime_destory_file_private() should be called wh