]> granicus.if.org Git - procps-ng/commitdiff
from Folkert van Heusden's code review
authoralbert <>
Sat, 28 Dec 2002 23:57:10 +0000 (23:57 +0000)
committeralbert <>
Sat, 28 Dec 2002 23:57:10 +0000 (23:57 +0000)
proc/ksym.c
sysctl.c

index e9bd8203d630cec40705497d4e05293c30412386..ce43db61ca9f31de71b8ec4944ae8c1c64b66742 100644 (file)
@@ -479,7 +479,7 @@ int open_psdb_message(const char *restrict override, void (*message)(const char
   };
   struct stat sbuf;
   struct utsname uts;
-  char path[64];
+  char path[128];
   const char **fmt = sysmap_paths;
   const char *sm;
 
@@ -509,9 +509,10 @@ int open_psdb_message(const char *restrict override, void (*message)(const char
 
   // finally, search for the System.map file
   uname(&uts);
+  path[sizeof path - 1] = '\0';
   do{
     int did_ksyms = 0;
-    snprintf(path, sizeof path, *fmt, uts.release);
+    snprintf(path, sizeof path - 1, *fmt, uts.release);
     if(!stat(path, &sbuf)){
       if (did_ksyms++) read_and_parse();
       if (sysmap_mmap(path, message)) return 0;
index 5e10d1599ee7ff7b60d78cbfa9aee7a7c02f62a7..613688cd77f0be861d42bc5837c144511f5be160 100644 (file)
--- a/sysctl.c
+++ b/sysctl.c
@@ -313,9 +313,9 @@ static int WriteSetting(const char *setting) {
  *
  */
 static int Preload(const char *restrict const filename) {
+   char oneline[256];
+   char buffer[256];
    FILE *fp;
-   char oneline[257];
-   char buffer[257];
    char *t;
    int n = 0;
    int rc = 0;
@@ -326,8 +326,7 @@ static int Preload(const char *restrict const filename) {
       return -1;
    }
 
-   while (fgets(oneline, 256, fp)) {
-      oneline[256] = 0;
+   while (fgets(oneline, sizeof oneline, fp)) {
       n++;
       t = StripLeadingAndTrailingSpaces(oneline);