[src/trunk]: src/sys/external/bsd/drm2/linux Allow fence_get(NULL).

[riastradh <riastradh%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/1b1c1f142eac
branches:  trunk
changeset: 365013:1b1c1f142eac
user:      riastradh <riastradh%NetBSD.org@localhost>
date:      Sat Sep 01 22:28:04 2018 +0000

description:
Allow fence_get(NULL).

This is kind of silly: if the caller has exclusive access to the
fence, they should be able to guarantee it's there, so this could
serve to suppress legitimate bugs.  But better to preserve Linux
semantics here than to be picky about API design.

Bug by me, fix from Tobias Ulmer in PR kern/53565.

diffstat:

 sys/external/bsd/drm2/linux/linux_fence.c |  7 ++++---
 1 files changed, 4 insertions(+), 3 deletions(-)

diffs (28 lines):

diff -r cbc968e705b4 -r 1b1c1f142eac sys/external/bsd/drm2/linux/linux_fence.c
--- a/sys/external/bsd/drm2/linux/linux_fence.c Sat Sep 01 22:01:03 2018 +0000
+++ b/sys/external/bsd/drm2/linux/linux_fence.c Sat Sep 01 22:28:04 2018 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: linux_fence.c,v 1.12 2018/08/28 15:04:02 riastradh Exp $       */
+/*     $NetBSD: linux_fence.c,v 1.13 2018/09/01 22:28:04 riastradh Exp $       */
 
 /*-
  * Copyright (c) 2018 The NetBSD Foundation, Inc.
@@ -30,7 +30,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: linux_fence.c,v 1.12 2018/08/28 15:04:02 riastradh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: linux_fence.c,v 1.13 2018/09/01 22:28:04 riastradh Exp $");
 
 #include <sys/atomic.h>
 #include <sys/condvar.h>
@@ -179,7 +179,8 @@
 fence_get(struct fence *fence)
 {
 
-       kref_get(&fence->refcount);
+       if (fence)
+               kref_get(&fence->refcount);
        return fence;
 }