Re: [PATCH v3 0/2] Parfait changes

[Daniel Vetter <daniel@ffwll.ch>]

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