diff options
author | Jim Meyering <meyering@redhat.com> | 2012-04-20 21:27:26 +0200 |
---|---|---|
committer | Hugo Mills <hugo@carfax.org.uk> | 2012-06-05 19:56:20 +0100 |
commit | 8ad8e789fc03423a9f1d33051b592419399184bb (patch) | |
tree | dc577d4651138bca17253e6982472505c6d2a2c9 /mkfs.c | |
parent | 54e345b5c2339dee170595f686dbd26937620262 (diff) |
mkfs: avoid heap-buffer-read-underrun for zero-length "size" arg
* mkfs.c (parse_size): ./mkfs.btrfs -A '' would read and possibly
write the byte before beginning of strdup'd heap buffer. All other
size-accepting options were similarly affected.
Reviewed-by: Josef Bacik <josef@redhat.com>
Diffstat (limited to 'mkfs.c')
-rw-r--r-- | mkfs.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -63,7 +63,7 @@ static u64 parse_size(char *s) s = strdup(s); - if (!isdigit(s[len - 1])) { + if (len && !isdigit(s[len - 1])) { c = tolower(s[len - 1]); switch (c) { case 'g': |