Re: [Powertop] [PATCH 2/2] devlist: NULL terminate struct devuser fields

As strncpy doesn't guarantee the destination buffer is NULL- terminated, do it explicitly. Signed-off-by: Namhyung Kim <namhyung(a)gmail.com&gt; --- src/devlist.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/devlist.cpp b/src/devlist.cpp index 828f47f..71898af 100644 --- a/src/devlist.cpp +++ b/src/devlist.cpp @@ -147,7 +147,9 @@ void collect_open_devices(void) continue; dev->pid = strtoull(entry->d_name, NULL, 10); strncpy(dev->device, link, 251); + dev->device[251] = '\0'; strncpy(dev->comm, read_sysfs_string("/proc/%s/comm", entry->d_name).c_str(), 31); + dev->comm[31] = '\0'; target->push_back(dev); }