projects
/
openwrt
/
staging
/
blogic.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
0ad4dc8
)
drm/i915: Check C for null pointer rather than B
author
Colin Ian King
<
[email protected]
>
Thu, 18 May 2017 13:39:42 +0000
(14:39 +0100)
committer
Chris Wilson
<
[email protected]
>
Thu, 18 May 2017 13:52:02 +0000
(14:52 +0100)
There are two occasions where pointer B is being check for a NULL
when it should be pointer C instead. Fix these.
Detected by CoverityScan, CID#
1436348
,
1436349
("Logically Dead Code")
Fixes: 47624cc3301b60 ("drm/i915: Import the kfence selftests for i915_sw_fence")
Signed-off-by: Colin Ian King <
[email protected]
>
Link:
http://patchwork.freedesktop.org/patch/msgid/
[email protected]
Signed-off-by: Chris Wilson <
[email protected]
>
drivers/gpu/drm/i915/selftests/i915_sw_fence.c
patch
|
blob
|
history
diff --git
a/drivers/gpu/drm/i915/selftests/i915_sw_fence.c
b/drivers/gpu/drm/i915/selftests/i915_sw_fence.c
index 98baf10c28c6a5ab06078cab2e51b7b209a84568..c31d439fae3a174f09af813fd7a7657035013ac4 100644
(file)
--- a/
drivers/gpu/drm/i915/selftests/i915_sw_fence.c
+++ b/
drivers/gpu/drm/i915/selftests/i915_sw_fence.c
@@
-311,7
+311,7
@@
static int test_AB_C(void *arg)
}
C = alloc_fence();
- if (!
B
) {
+ if (!
C
) {
ret = -ENOMEM;
goto err_B;
}
@@
-388,7
+388,7
@@
static int test_C_AB(void *arg)
}
C = alloc_fence();
- if (!
B
) {
+ if (!
C
) {
ret = -ENOMEM;
goto err_B;
}