diff options
| author | Max Kellermann <max@duempel.org> | 2016-03-21 07:33:12 -0400 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2016-04-06 08:39:56 -0400 |
| commit | 88336e174645948da269e1812f138f727cd2896b (patch) | |
| tree | 76f99303d1df74dc3f616e0323b3579cd28d8bb5 /drivers/media/media-devnode.c | |
| parent | 952f8eef901b170dbe6b48e80f098be5d835a82c (diff) | |
[media] media-devnode: add missing mutex lock in error handler
We should protect the device unregister patch too, at the error
condition.
Signed-off-by: Max Kellermann <max@duempel.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Diffstat (limited to 'drivers/media/media-devnode.c')
| -rw-r--r-- | drivers/media/media-devnode.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/media/media-devnode.c b/drivers/media/media-devnode.c index 29409f440f1c..64a4b1ef3dcd 100644 --- a/drivers/media/media-devnode.c +++ b/drivers/media/media-devnode.c | |||
| @@ -267,8 +267,11 @@ int __must_check media_devnode_register(struct media_devnode *mdev, | |||
| 267 | return 0; | 267 | return 0; |
| 268 | 268 | ||
| 269 | error: | 269 | error: |
| 270 | mutex_lock(&media_devnode_lock); | ||
| 270 | cdev_del(&mdev->cdev); | 271 | cdev_del(&mdev->cdev); |
| 271 | clear_bit(mdev->minor, media_devnode_nums); | 272 | clear_bit(mdev->minor, media_devnode_nums); |
| 273 | mutex_unlock(&media_devnode_lock); | ||
| 274 | |||
| 272 | return ret; | 275 | return ret; |
| 273 | } | 276 | } |
| 274 | 277 | ||
