On 02/10/2012 11:35 AM, Wanlong Gao wrote:
> change to use SAFE_STRTOL instead in libmem.
>
> Signed-off-by: Wanlong Gao <[email protected]>
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;
> - 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);
> }
------------------------------------------------------------------------------
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/
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list
