diff options
| author | Dan Carpenter <error27@gmail.com> | 2010-10-25 05:02:56 (GMT) |
|---|---|---|
| committer | Steve French <sfrench@us.ibm.com> | 2010-10-26 18:22:38 (GMT) |
| commit | b235f371a2572d7c86a121d96d889eee02ed00e2 (patch) | |
| tree | 72ae25c83a6ba59926a599e28d47b67bfa0ad863 | |
| parent | 21e733930be6458e0c33482b6783e7c15ba984eb (diff) | |
| download | linux-fsl-qoriq-b235f371a2572d7c86a121d96d889eee02ed00e2.tar.xz | |
cifs: cifs_convert_address() returns zero on error
The cifs_convert_address() returns zero on error but this caller is
testing for negative returns.
Btw. "i" is unsigned here, so it's never negative.
Reviewed-by: Jeff Layton <jlayton@samba.org>
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>
| -rw-r--r-- | fs/cifs/connect.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/cifs/connect.c b/fs/cifs/connect.c index c5807d3..dd9a4ae 100644 --- a/fs/cifs/connect.c +++ b/fs/cifs/connect.c @@ -1067,7 +1067,7 @@ cifs_parse_mount_options(char *options, const char *devname, } i = cifs_convert_address((struct sockaddr *)&vol->srcaddr, value, strlen(value)); - if (i < 0) { + if (i == 0) { printk(KERN_WARNING "CIFS: Could not parse" " srcaddr: %s\n", value); |