aboutsummaryrefslogtreecommitdiffstats
path: root/fs/ext3
diff options
context:
space:
mode:
authorKAMBAROV, ZAUR <kambarov@berkeley.edu>2005-06-28 23:45:11 -0400
committerLinus Torvalds <torvalds@ppc970.osdl.org>2005-06-29 00:20:34 -0400
commitc7f1721ef284c6e8257c7471a02148db76105036 (patch)
tree6c16fdc37f5bdd46f38e5dd8054b0b1c7363833c /fs/ext3
parentec471dc484b8ca5352903ee28796b8b248313547 (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')
-rw-r--r--fs/ext3/super.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/ext3/super.c b/fs/ext3/super.c
index b4b3e8a39131..a6d1779d7de4 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: