Because it's better than a segfault if it's called improperly,
and it makes static checkers happier.
Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxx>
---
utils.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/utils.c b/utils.c
index 54d577c..a38ac70 100644
--- a/utils.c
+++ b/utils.c
@@ -1382,7 +1382,7 @@ u64 parse_size(char *s)
char c;
u64 mult = 1;
- for (i=0 ; s[i] && isdigit(s[i]) ; i++) ;
+ for (i=0 ; s && s[i] && isdigit(s[i]) ; i++) ;
if (!i) {
fprintf(stderr, "ERROR: size value is empty\n");
exit(50);
--
1.7.1
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html