Re: [PATCH v3 0/2] Parfait changes

2018-03-05 Thread Daniel Vetter
On Mon, Mar 05, 2018 at 11:14:11AM -0500, Joe Moriarty wrote:
> On 2/20/2018 2:11 PM, Joe Moriarty wrote:
> > The following patch(s) are bugs found by the static compiler
> > 'Parfait'.  Care was taken to make sure false positive results
> > were removed from this patchset.
> > 
> > Parfait Overview
> > 
> > 
> > https://labs.oracle.com/pls/apex/f?p=labs:49:P49_PROJECT_ID:13
> > 
> > v1:
> > Initial release
> > 
> > v2:
> > - Split original v1 patch into 4 separate patches per request
> > from Jani Nikula
> > - Fixed system hang during boot up on test machine.
> > 
> > v3:
> > - Made changes requested by Daniel Vetter.
> > 
> > Joe Moriarty (2):
> >drm: NULL pointer dereference [null-pointer-deref] (CWE 476) problem
> >drm: NULL pointer dereference [null-pointer-deref] (CWE 476) problem
> > 
> >   drivers/gpu/drm/drm_drv.c| 2 +-
> >   drivers/gpu/drm/drm_vblank.c | 9 +
> >   2 files changed, 10 insertions(+), 1 deletion(-)
> > 
> Ping, Waiting on review/addition of these 2 patches to drm-misc-next.

I was on vacation, and I guess no one else cared. Btw, minor process nit:
Including internal review tags is fairly pointless, since generally I have
no idea about how it happened (there's no public record), nor do I know
the experience level of your reviewer. That makes the r-b tag fairly
value-less in this case. If you want to included it's much better to do
that review on the m-l, after submission.

Anyway, thanks for respinning, patches merged.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [PATCH v3 0/2] Parfait changes

2018-03-05 Thread Joe Moriarty

On 2/20/2018 2:11 PM, Joe Moriarty wrote:

The following patch(s) are bugs found by the static compiler
'Parfait'.  Care was taken to make sure false positive results
were removed from this patchset.

Parfait Overview


https://labs.oracle.com/pls/apex/f?p=labs:49:P49_PROJECT_ID:13

v1:
Initial release

v2:
- Split original v1 patch into 4 separate patches per request
from Jani Nikula
- Fixed system hang during boot up on test machine.

v3:
- Made changes requested by Daniel Vetter.

Joe Moriarty (2):
   drm: NULL pointer dereference [null-pointer-deref] (CWE 476) problem
   drm: NULL pointer dereference [null-pointer-deref] (CWE 476) problem

  drivers/gpu/drm/drm_drv.c| 2 +-
  drivers/gpu/drm/drm_vblank.c | 9 +
  2 files changed, 10 insertions(+), 1 deletion(-)


Ping, Waiting on review/addition of these 2 patches to drm-misc-next.

Thanks,
Joe
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel


[PATCH v3 0/2] Parfait changes

2018-02-20 Thread Joe Moriarty
The following patch(s) are bugs found by the static compiler
'Parfait'.  Care was taken to make sure false positive results
were removed from this patchset.

Parfait Overview


https://labs.oracle.com/pls/apex/f?p=labs:49:P49_PROJECT_ID:13

v1:
Initial release

v2:
- Split original v1 patch into 4 separate patches per request
from Jani Nikula
- Fixed system hang during boot up on test machine.

v3:
- Made changes requested by Daniel Vetter.

Joe Moriarty (2):
  drm: NULL pointer dereference [null-pointer-deref] (CWE 476) problem
  drm: NULL pointer dereference [null-pointer-deref] (CWE 476) problem

 drivers/gpu/drm/drm_drv.c| 2 +-
 drivers/gpu/drm/drm_vblank.c | 9 +
 2 files changed, 10 insertions(+), 1 deletion(-)

-- 
2.15.0

___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel