[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[tor-commits] [tor/release-0.4.5] Fix parsing bug in linux get_total_system_memory().



commit 690c7be2537dc30be2d1f4978af7df77c46c2782
Author: Nick Mathewson <nickm@xxxxxxxxxxxxxx>
Date:   Wed Mar 3 14:52:15 2021 -0500

    Fix parsing bug in linux get_total_system_memory().
    
    Use find_str_at_start_of_line(), not strstr() here: we don't want
    to match "MemTotal: " if it appears in the middle of a line.
    
    Fixes #40315; bugfix on 0.2.5.4-alpha.
---
 changes/bug40315             | 5 +++++
 src/lib/meminfo/.may_include | 1 +
 src/lib/meminfo/meminfo.c    | 3 ++-
 3 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/changes/bug40315 b/changes/bug40315
new file mode 100644
index 0000000000..9e9c740d96
--- /dev/null
+++ b/changes/bug40315
@@ -0,0 +1,5 @@
+  o Minor bugfixes (Linux, relay):
+    - Fix a bug in determining total available system memory that would have
+      been triggered if the format of /proc/meminfo had ever changed
+      to include "MemTotal:" in the middle of a line. Fixes bug 40315;
+      bugfix on 0.2.5.4-alpha.
diff --git a/src/lib/meminfo/.may_include b/src/lib/meminfo/.may_include
index 9e4d25fd6a..12fe36d134 100644
--- a/src/lib/meminfo/.may_include
+++ b/src/lib/meminfo/.may_include
@@ -5,4 +5,5 @@ lib/fs/*.h
 lib/log/*.h
 lib/malloc/*.h
 lib/meminfo/*.h
+lib/string/*.h
 lib/testsupport/*.h
diff --git a/src/lib/meminfo/meminfo.c b/src/lib/meminfo/meminfo.c
index b7d991e410..77da579f99 100644
--- a/src/lib/meminfo/meminfo.c
+++ b/src/lib/meminfo/meminfo.c
@@ -17,6 +17,7 @@
 #include "lib/fs/files.h"
 #include "lib/log/log.h"
 #include "lib/malloc/malloc.h"
+#include "lib/string/util_string.h"
 
 #ifdef HAVE_FCNTL_H
 #include <fcntl.h>
@@ -65,7 +66,7 @@ get_total_system_memory_impl(void)
   s = read_file_to_str_until_eof(fd, 65536, &file_size);
   if (!s)
     goto err;
-  cp = strstr(s, "MemTotal:");
+  cp = find_str_at_start_of_line(s, "MemTotal:");
   if (!cp)
     goto err;
   /* Use the system sscanf so that space will match a wider number of space */



_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits