From 53bedaa4c1a47644e8c0a8dc5be07bedb3bca422 Mon Sep 17 00:00:00 2001 From: nekral-guest Date: Sun, 25 May 2008 23:25:33 +0000 Subject: [PATCH] * libmisc/getlong.c: Avoid implicit conversion of pointers / chars to booleans. * libmisc/getlong.c: Add brackets. --- ChangeLog | 10 ++++++++-- libmisc/getlong.c | 5 +++-- 2 files changed, 11 insertions(+), 4 deletions(-) diff --git a/ChangeLog b/ChangeLog index cad11f96..6bce5ff2 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,7 +1,13 @@ 2008-05-26 Nicolas François - * libmisc/valid.c: Avoid implicit conversion of pointers /chars to - booleans. + * libmisc/getlong.c: Avoid implicit conversion of pointers / chars + to booleans. + * libmisc/getlong.c: Add brackets. + +2008-05-26 Nicolas François + + * libmisc/valid.c: Avoid implicit conversion of pointers / chars + to booleans. * libmisc/valid.c: Add brackets. 2008-05-26 Nicolas François diff --git a/libmisc/getlong.c b/libmisc/getlong.c index 3dd5614d..7ea3a80d 100644 --- a/libmisc/getlong.c +++ b/libmisc/getlong.c @@ -1,5 +1,5 @@ /* - * Copyright (c) 2007 , Nicolas François + * Copyright (c) 2007 - 2008, Nicolas François * All rights reserved. * * Redistribution and use in source and binary forms, with or without @@ -40,8 +40,9 @@ int getlong(const char *numstr, long int *result) char *endptr; val = strtol (numstr, &endptr, 10); - if (*endptr || errno == ERANGE) + if (('\0' != *endptr) || (ERANGE == errno)) { return 0; + } *result = val; return 1;