From: Juergen Lock Date: Fri, 16 Aug 2013 18:00:24 +0000 (-0300) Subject: upstream: [media] media: rc: rdev->open or rdev->close can be NULL X-Git-Tag: submit/tizen/20141121.110247~919 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=1b0d952834c22debbf33470cbeef02ba6f0c03a6;p=platform%2Fkernel%2Flinux-3.10.git upstream: [media] media: rc: rdev->open or rdev->close can be NULL At least technisat-usb2.c doesn't set these... Signed-off-by: Juergen Lock Signed-off-by: Mauro Carvalho Chehab --- diff --git a/drivers/media/rc/rc-main.c b/drivers/media/rc/rc-main.c index 5e5a3cb..893bd1fb 100644 --- a/drivers/media/rc/rc-main.c +++ b/drivers/media/rc/rc-main.c @@ -712,7 +712,7 @@ int rc_open(struct rc_dev *rdev) return -EINVAL; mutex_lock(&rdev->lock); - if (!rdev->users++) + if (!rdev->users++ && rdev->open != NULL) rval = rdev->open(rdev); if (rval) @@ -736,7 +736,7 @@ void rc_close(struct rc_dev *rdev) if (rdev) { mutex_lock(&rdev->lock); - if (!--rdev->users) + if (!--rdev->users && rdev->close != NULL) rdev->close(rdev); mutex_unlock(&rdev->lock);