diff options
author | KAMBAROV, ZAUR <kambarov@berkeley.edu> | 2005-06-28 23:45:11 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@ppc970.osdl.org> | 2005-06-29 00:20:34 -0400 |
commit | c7f1721ef284c6e8257c7471a02148db76105036 (patch) | |
tree | 6c16fdc37f5bdd46f38e5dd8054b0b1c7363833c /fs/ext3/super.c | |
parent | ec471dc484b8ca5352903ee28796b8b248313547 (diff) |
[PATCH] coverity: fs/ext3/super.c: match_int return check
The return value of "match_int" is checked 27 out of 28 times
In lib/parser.c
142 /**
143 * match_int: - scan a decimal representation of an integer from a substring_t
144 * @s: substring_t to be scanned
145 * @result: resulting integer on success
146 *
147 * Description: Attempts to parse the &substring_t @s as a decimal integer. On
148 * success, sets @result to the integer represented by the string and returns 0.
149 * Returns either -ENOMEM or -EINVAL on failure.
150 */
151 int match_int(substring_t *s, int *result)
152 {
153 return match_number(s, result, 0);
154 }
Signed-off-by: Zaur Kambarov <zkambarov@coverity.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'fs/ext3/super.c')
-rw-r--r-- | fs/ext3/super.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/ext3/super.c b/fs/ext3/super.c index b4b3e8a3913..a6d1779d7de 100644 --- a/fs/ext3/super.c +++ b/fs/ext3/super.c | |||
@@ -944,7 +944,8 @@ clear_qf_name: | |||
944 | "for remount\n"); | 944 | "for remount\n"); |
945 | return 0; | 945 | return 0; |
946 | } | 946 | } |
947 | match_int(&args[0], &option); | 947 | if (match_int(&args[0], &option) != 0) |
948 | return 0; | ||
948 | *n_blocks_count = option; | 949 | *n_blocks_count = option; |
949 | break; | 950 | break; |
950 | case Opt_nobh: | 951 | case Opt_nobh: |