Afer commit 1fa5ae857bb1 ("driver core: get rid of struct device's bus_id string array"), the name of device is allocated dynamically, so call kfree_const() to freed it in error case. Fixes: 1fa5ae857bb1 ("driver core: get rid of struct device's bus_id string array") Signed-off-by: Yang Yingliang <yangyingliang@xxxxxxxxxx> --- drivers/media/v4l2-core/v4l2-dev.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/media/v4l2-core/v4l2-dev.c b/drivers/media/v4l2-core/v4l2-dev.c index 397d553177fa..1fce1c29c60f 100644 --- a/drivers/media/v4l2-core/v4l2-dev.c +++ b/drivers/media/v4l2-core/v4l2-dev.c @@ -1036,6 +1036,9 @@ int __video_register_device(struct video_device *vdev, ret = device_register(&vdev->dev); if (ret < 0) { pr_err("%s: device_register failed\n", __func__); + kfree_const(vdev->dev.kobj.name); + /* set it to null to avoid callers using a bad pointer */ + vdev->dev.kobj.name = NULL; goto cleanup; } /* Register the release callback that will be called when the last -- 2.25.1