diff options
author | Eric Sandeen <sandeen@redhat.com> | 2011-09-16 15:49:32 -0500 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2011-09-16 18:45:14 -0400 |
commit | 9f7c3afaef7af307ae9c0fa3648c2e5c61f468b8 (patch) | |
tree | d5f526b914fedf27d31c1c7f22cf71e3661e123a /misc/mke2fs.c | |
parent | 14ed2edb26e94b41ac06d81804b3e6af576b5c79 (diff) | |
download | e2fsprogs-9f7c3afaef7af307ae9c0fa3648c2e5c61f468b8.tar.gz |
mke2fs: Do not let -t or -T be specified more than once
In addition to not making sense, it causes a memory leak
when fs_type gets overwritten.
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
Diffstat (limited to 'misc/mke2fs.c')
-rw-r--r-- | misc/mke2fs.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/misc/mke2fs.c b/misc/mke2fs.c index 437b4955..ff5d54bc 100644 --- a/misc/mke2fs.c +++ b/misc/mke2fs.c @@ -1445,9 +1445,19 @@ profile_error: super_only = 1; break; case 't': + if (fs_type) { + com_err(program_name, 0, + _("The -t option may only be used once")); + exit(1); + } fs_type = strdup(optarg); break; case 'T': + if (usage_types) { + com_err(program_name, 0, + _("The -T option may only be used once")); + exit(1); + } usage_types = strdup(optarg); break; case 'U': |