diff options
author | Sachin Kamat <sachin.kamat@linaro.org> | 2012-11-19 04:44:59 -0500 |
---|---|---|
committer | Dave Airlie <airlied@redhat.com> | 2012-11-20 00:41:00 -0500 |
commit | 66141d3d86f38701d657ef9ba973233bce53da79 (patch) | |
tree | 8e7ceaf10f6ab392baa2254461340870e21e9ea1 | |
parent | 8c5eaca012f963fa9724665f8a0eca19e3cb7d2a (diff) |
drm/drm_stub: Remove unnecessary null check before kfree.
kfree on a null argument is a no-op.
Silences the following smatch warning:
drivers/gpu/drm/drm_stub.c:496 drm_put_dev() info:
redundant null check on dev->devname calling kfree()
Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Signed-off-by: Dave Airlie <airlied@redhat.com>
-rw-r--r-- | drivers/gpu/drm/drm_stub.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/gpu/drm/drm_stub.c b/drivers/gpu/drm/drm_stub.c index 581e61d04057..6fcdd8ebcb3d 100644 --- a/drivers/gpu/drm/drm_stub.c +++ b/drivers/gpu/drm/drm_stub.c | |||
@@ -492,10 +492,7 @@ void drm_put_dev(struct drm_device *dev) | |||
492 | drm_put_minor(&dev->primary); | 492 | drm_put_minor(&dev->primary); |
493 | 493 | ||
494 | list_del(&dev->driver_item); | 494 | list_del(&dev->driver_item); |
495 | if (dev->devname) { | 495 | kfree(dev->devname); |
496 | kfree(dev->devname); | ||
497 | dev->devname = NULL; | ||
498 | } | ||
499 | kfree(dev); | 496 | kfree(dev); |
500 | } | 497 | } |
501 | EXPORT_SYMBOL(drm_put_dev); | 498 | EXPORT_SYMBOL(drm_put_dev); |