]> granicus.if.org Git - procps-ng/commitdiff
proc/sysinfo.c: Ensure null-termination in getstat().
authorQualys Security Advisory <qsa@qualys.com>
Thu, 1 Jan 1970 00:00:00 +0000 (00:00 +0000)
committerCraig Small <csmall@enc.com.au>
Fri, 18 May 2018 21:32:22 +0000 (07:32 +1000)
There was a "buff[BUFFSIZE-1] = 0;" but there may be garbage between
what is read() (less than BUFFSIZE-1 bytes) and this null byte. Reuse
the construct from the preceding getrunners().

proc/sysinfo.c

index 42646942bdeae18719e64e1f218f50feae74f7f4..4b2090be948ca54281dd1fc5f1a65b8a9d875162 100644 (file)
@@ -524,7 +524,7 @@ void getstat(jiff *restrict cuse, jiff *restrict cice, jiff *restrict csys, jiff
   int need_vmstat_file = 0;
   int need_proc_scan = 0;
   const char* b;
-  buff[BUFFSIZE-1] = 0;  /* ensure null termination in buffer */
+  memset(buff, '\0', BUFFSIZE);  /* ensure null termination in buffer */
 
   if(fd){
     lseek(fd, 0L, SEEK_SET);