Wanlong Gao | 10 Feb 04:35
Favicon
Gravatar

[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)
(Continue reading)

Caspar Zhang | 10 Feb 06:34
Gravatar

Re: [PATCH] libmem: use SAFE_STRTOL instead

On 02/10/2012 11:35 AM, Wanlong Gao wrote:
> change to use SAFE_STRTOL instead in libmem.
> 
> Signed-off-by: Wanlong Gao <gaowanlong@...>

Hi, Committed.

Thanks,
Caspar

> ---
>  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;
(Continue reading)


Gmane