diff options
author | Max Kellermann <max@duempel.org> | 2016-03-21 09:30:33 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2016-05-07 10:24:02 -0400 |
commit | c56d34a73ea3622fbc3f5a85140f2353f14b591c (patch) | |
tree | a2eed876ded1fc63afc231728dca17ada05b8c64 | |
parent | 47cae1e1cfdb6a07ca19c61345c4d96f828b14d7 (diff) |
[media] drivers/media/media-device: move debug log before _devnode_unregister()
After media_devnode_unregister(), the struct media_device may be freed
already, and dereferencing it may crash.
Signed-off-by: Max Kellermann <max@duempel.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
-rw-r--r-- | drivers/media/media-device.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/media/media-device.c b/drivers/media/media-device.c index 898a3cf814ba..de92a6bf3751 100644 --- a/drivers/media/media-device.c +++ b/drivers/media/media-device.c | |||
@@ -801,9 +801,8 @@ void media_device_unregister(struct media_device *mdev) | |||
801 | mutex_unlock(&mdev->graph_mutex); | 801 | mutex_unlock(&mdev->graph_mutex); |
802 | 802 | ||
803 | device_remove_file(&mdev->devnode.dev, &dev_attr_model); | 803 | device_remove_file(&mdev->devnode.dev, &dev_attr_model); |
804 | dev_dbg(mdev->dev, "Media device unregistering\n"); | ||
804 | media_devnode_unregister(&mdev->devnode); | 805 | media_devnode_unregister(&mdev->devnode); |
805 | |||
806 | dev_dbg(mdev->dev, "Media device unregistered\n"); | ||
807 | } | 806 | } |
808 | EXPORT_SYMBOL_GPL(media_device_unregister); | 807 | EXPORT_SYMBOL_GPL(media_device_unregister); |
809 | 808 | ||