aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGerald Schaefer <gerald.schaefer@de.ibm.com>2013-02-26 14:33:54 +0100
committerEric Munson <emunson@bert.(none)>2013-03-03 18:39:25 -0500
commit7caefa7221c5b09bc5a9f77e715807bcc85fdfba (patch)
treec896b752a1640c0e495443b4069ad92c91fad829
parentee00e3102ec8354175b2ce99eb5350d7026fd67c (diff)
downloadlibhugetlbfs-7caefa7221c5b09bc5a9f77e715807bcc85fdfba.tar.gz
Terminate path with '\0' in find_mounts()
commit 8c075e33 introduced a regression. The previous sscanf() implicitly did a null termination on path, while the new code misses the null termination. This can result in a failing hugetlbfs_test_path() and an error message like: "libhugetlbfs: WARNING: Hugepage size 1048576 unavailable" Reported-by: Vitaly Mayatskih <v.mayatskih@gmail.com> Signed-off-by: Gerald Schaefer <gerald.schaefer@de.ibm.com> Signed-off-by: Eric B Munson <emunson@mgebm.net>
-rw-r--r--hugeutils.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/hugeutils.c b/hugeutils.c
index 6a4dc85..53a7fbd 100644
--- a/hugeutils.c
+++ b/hugeutils.c
@@ -665,6 +665,7 @@ static void find_mounts(void)
continue;
strncpy(path, match, end - match);
+ path[end - match] = '\0';
if ((hugetlbfs_test_path(path) == 1) &&
!(access(path, R_OK | W_OK | X_OK)))
add_hugetlbfs_mount(path, 0);