[PATCH] libmem: use SAFE_STRTOL instead

change to use SAFE_STRTOL instead in libmem.

Signed-off-by: Wanlong Gao <gaowanlong@...>
---
 testcases/kernel/mem/lib/mem.c |   12 +++---------
 1 files changed, 3 insertions(+), 9 deletions(-)

diff --git a/testcases/kernel/mem/lib/mem.c b/testcases/kernel/mem/lib/mem.c
index e25b5be..cce7751 100644
--- a/testcases/kernel/mem/lib/mem.c
+++ b/testcases/kernel/mem/lib/mem.c
@@ -13,6 +13,7 @@
 #include "usctest.h"
 #include "../include/mem.h"
 #include "config.h"
+#include "safe_macros.h"
 #if HAVE_NUMA_H && HAVE_LINUX_MEMPOLICY_H && HAVE_NUMAIF_H \
 	&& HAVE_MPOL_CONSTANTS
 #include <numaif.h>
@@ -769,8 +770,7 @@ void set_sys_tune(char *sys_file, long tune, int check)
 long get_sys_tune(char *sys_file)
 {
 	int fd;
-	long tune;
-	char buf[BUFSIZ], path[BUFSIZ], *endptr;
+	char buf[BUFSIZ], path[BUFSIZ];

 	snprintf(path, BUFSIZ, "%s%s", PATH_SYSVM, sys_file);
 	fd = open(path, O_RDONLY);
@@ -780,11 +780,5 @@ long get_sys_tune(char *sys_file)
 		tst_brkm(TBROK|TERRNO, cleanup, "read %s", sys_file);
 	close(fd);

-	tune = strtol(buf, &endptr, 10);
-	if (tune == LONG_MAX || tune == LONG_MIN)
-		tst_brkm(TBROK|TERRNO, cleanup, "strtol");
-	if (endptr == buf || (*endptr != '\0' && *endptr != '\n'))
-		tst_brkm(TBROK, cleanup, "Invalid value: %s", buf);
-
-	return tune;
+	return SAFE_STRTOL(cleanup, buf, LONG_MIN, LONG_MAX);
 }
--

-- 
1.7.9

------------------------------------------------------------------------------
Virtualization & Cloud Management Using Capacity Planning
Cloud computing makes use of virtualization - but cloud computing 
also focuses on allowing computing to be delivered as a service.
http://www.accelacomm.com/jaw/sfnl/114/51521223/