]> granicus.if.org Git - procps-ng/commitdiff
library: size two read buffers with manifest constants
authorJim Warner <james.warner@comcast.net>
Mon, 29 Jun 2020 05:00:00 +0000 (00:00 -0500)
committerCraig Small <csmall@dropbear.xyz>
Sun, 5 Jul 2020 11:13:01 +0000 (21:13 +1000)
The meminfo & vmstat file read buffers are now defined
through symbolic constants next to the symbolic names.

[ recent attention to the /proc/stat buffer size led ]
[ to this (hopefully) better approach to definitions ]

Reference(s):
commit 81f4a6acdf204c5c898fa707e285cdbd2389150e

Signed-off-by: Jim Warner <james.warner@comcast.net>
proc/meminfo.c
proc/vmstat.c

index 668dcb616dfd7f578663a2ceb7cb4ce9bbb52d63..73035877a5a8e3c406d5e904e00eff633f13c442 100644 (file)
@@ -32,7 +32,8 @@
 #include <proc/meminfo.h>
 
 
-#define MEMINFO_FILE "/proc/meminfo"
+#define MEMINFO_FILE  "/proc/meminfo"
+#define MEMINFO_BUFF  8192
 
 
 struct meminfo_data {
@@ -606,7 +607,7 @@ static int meminfo_read_failed (
  /* a 'memory history reference' macro for readability,
     so we can focus the field names ... */
  #define mHr(f) info->hist.new. f
-    char buf[8192];
+    char buf[MEMINFO_BUFF];
     char *head, *tail;
     int size;
     unsigned long *valptr;
index ab404bdf6bd9759ec483050ca98806b0f68f31a2..36a892325102bcfc84cf51c8dd888d3cc0008574 100644 (file)
@@ -38,7 +38,8 @@
 #include <proc/vmstat.h>
 
 
-#define VMSTAT_FILE "/proc/vmstat"
+#define VMSTAT_FILE  "/proc/vmstat"
+#define VMSTAT_BUFF  8192
 
 /*
  *  Perhaps someday we'll all learn what is in these fields. But |
@@ -1067,7 +1068,7 @@ static int vmstat_make_hash_failed (
 static int vmstat_read_failed (
         struct vmstat_info *info)
 {
-    char buf[8192];
+    char buf[VMSTAT_BUFF];
     char *head, *tail;
     int size;
     unsigned long *valptr;