On 09/18/2012 08:13 AM, Namhyung Kim wrote: > As strncpy doesn't guarantee the destination buffer is NULL- > terminated, do it explicitly. > > Signed-off-by: Namhyung Kim > --- > 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); > > } Your patch has been merged Thank You -C