kernel: nvidia-oot: Static Analysis for S01/08 P11

Fix CERT STR07-C coverity defects for cdi_mgr.c tegra_rtcpu.h

Jira CAMERASW-30850

Change-Id: I86c4cec8cdda2a1f16d5ca7ea7fbeed3a0318ad1
Signed-off-by: ryanli <ryanli@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nv-oot/+/3282774
Reviewed-by: Semi Malinen <smalinen@nvidia.com>
GVS: buildbot_gerritrpt <buildbot_gerritrpt@nvidia.com>
Reviewed-by: Ankur Pawar <ankurp@nvidia.com>
Reviewed-by: Sumeet Gupta <sumeetg@nvidia.com>
This commit is contained in:
ryanli
2025-01-13 09:43:54 +00:00
committed by Jon Hunter
parent b577e6d0e9
commit 7c4412a765
2 changed files with 10 additions and 4 deletions

View File

@@ -1299,8 +1299,14 @@ static void cdi_mgr_dev_ins(struct work_struct *work)
dev_dbg(cdi_mgr->dev, "%s - %s\n", __func__, np->full_name); dev_dbg(cdi_mgr->dev, "%s - %s\n", __func__, np->full_name);
sname = of_get_property(np, "cdi-dev", NULL); sname = of_get_property(np, "cdi-dev", NULL);
if (sname) if (sname) {
strncpy(d_cfg.drv_name, sname, sizeof(d_cfg.drv_name) - 8); ssize_t sz_sname = 0;
sz_sname = strscpy(d_cfg.drv_name, sname, sizeof(d_cfg.drv_name) - 8);
if (sz_sname < 0) {
pr_err("File name too large: %s\n", sname);
return;
}
}
for_each_child_of_node(np, subdev) { for_each_child_of_node(np, subdev) {
err = of_property_read_u32(subdev, "addr", &val); err = of_property_read_u32(subdev, "addr", &val);

View File

@@ -120,11 +120,11 @@ TRACE_EVENT(rtcpu_string,
__entry->tstamp = tstamp; __entry->tstamp = tstamp;
__entry->id = id; __entry->id = id;
__entry->len = len; __entry->len = len;
strscpy(__entry->data, data, sizeof(__entry->data)); (void)strscpy(__entry->data, data, sizeof(__entry->data));
), ),
TP_printk("tstamp:%llu id:0x%08x str:\"%.*s\"", TP_printk("tstamp:%llu id:0x%08x str:\"%.*s\"",
__entry->tstamp, __entry->id, __entry->tstamp, __entry->id,
(int)__entry->len, __entry->data) (int)(sizeof(__entry->data)), __entry->data)
); );
DEFINE_EVENT(rtcpu__dump, rtcpu_bulk, DEFINE_EVENT(rtcpu__dump, rtcpu_bulk,