media: use strscpy() instead of strlcpy()

The implementation of strscpy() is more robust and safer.

That's now the recommended way to copy NUL terminated strings.

Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
Reviewed-by: Kees Cook <keescook@chromium.org>
Acked-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>

1 files changed, 3 insertions, 3 deletions

diff --git a/drivers/media/platform/vimc/vimc-capture.c b/drivers/media/platform/vimc/vimc-capture.c
index ec68feaac378..3f7e9ed56633 100644
--- a/drivers/media/platform/vimc/vimc-capture.c
+++ b/drivers/media/platform/vimc/vimc-capture.c
@@ -71,8 +71,8 @@ static int vimc_cap_querycap(struct file *file, void *priv,
 {
         struct vimc_cap_device *vcap = video_drvdata(file);
 
-        strlcpy(cap->driver, KBUILD_MODNAME, sizeof(cap->driver));
-        strlcpy(cap->card, KBUILD_MODNAME, sizeof(cap->card));
+        strscpy(cap->driver, KBUILD_MODNAME, sizeof(cap->driver));
+        strscpy(cap->card, KBUILD_MODNAME, sizeof(cap->card));
         snprintf(cap->bus_info, sizeof(cap->bus_info),
                  "platform:%s", vcap->vdev.v4l2_dev->name);
 
@@ -476,7 +476,7 @@ static int vimc_cap_comp_bind(struct device *comp, struct device *master,
         vdev->queue = q;
         vdev->v4l2_dev = v4l2_dev;
         vdev->vfl_dir = VFL_DIR_RX;
-        strlcpy(vdev->name, pdata->entity_name, sizeof(vdev->name));
+        strscpy(vdev->name, pdata->entity_name, sizeof(vdev->name));
         video_set_drvdata(vdev, &vcap->ved);
 
         /* Register the video_device with the v4l2 and the media framework */