Skip to content

Commit fece841

Browse files
committed
rpmsg: Strcpy is not safe, use strscpy_pad() instead
JIRA: https://issues.redhat.com/browse/RHEL-78824 commit 99de650 Author: Saud Farooqui <farooqui_saud@hotmail.com> Date: Thu Jun 23 14:20:47 2022 +0500 rpmsg: Strcpy is not safe, use strscpy_pad() instead Replace strcpy() with strscpy_pad() for copying the rpmsg device name in rpmsg_register_device_override(). Reported-by: kernel test robot <lkp@intel.com> Signed-off-by: Saud Farooqui <farooqui_saud@hotmail.com> Link: https://lore.kernel.org/r/PA4P189MB14210AA95DCA3715AFA7F4A68BB59@PA4P189MB1421.EURP189.PROD.OUTLOOK.COM Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org> Signed-off-by: Jared Kangas <jkangas@redhat.com>
1 parent 47b2dd2 commit fece841

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

drivers/rpmsg/rpmsg_core.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -604,7 +604,7 @@ int rpmsg_register_device_override(struct rpmsg_device *rpdev,
604604
int ret;
605605

606606
if (driver_override)
607-
strcpy(rpdev->id.name, driver_override);
607+
strscpy_pad(rpdev->id.name, driver_override, RPMSG_NAME_SIZE);
608608

609609
dev_set_name(dev, "%s.%s.%d.%d", dev_name(dev->parent),
610610
rpdev->id.name, rpdev->src, rpdev->dst);

0 commit comments

Comments
 (0)