diff options
author | Steve French <sfrench@us.ibm.com> | 2007-10-17 17:31:52 -0400 |
---|---|---|
committer | Steve French <sfrench@us.ibm.com> | 2007-10-17 17:31:52 -0400 |
commit | d5d18501090179d557a4ca976d1c30bfaf5de091 (patch) | |
tree | 95775567aa85bf5bf2fa3ee8bf743b4d17b0152a /fs/cifs/netmisc.c | |
parent | c18c732ec6bf372aa959ca6534cbfc32e464defd (diff) |
[CIFS] Fix minor problems noticed by scan
Coverity scan pointed out some minor possible errors.
Signed-off-by: Steve French <sfrench@us.ibm.com>
Diffstat (limited to 'fs/cifs/netmisc.c')
-rw-r--r-- | fs/cifs/netmisc.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/fs/cifs/netmisc.c b/fs/cifs/netmisc.c index 2bfed3f45d0f..9ae4941cd61d 100644 --- a/fs/cifs/netmisc.c +++ b/fs/cifs/netmisc.c | |||
@@ -899,8 +899,11 @@ struct timespec cnvrtDosUnixTm(__u16 date, __u16 time) | |||
899 | cERROR(1, ("illegal hours %d", st->Hours)); | 899 | cERROR(1, ("illegal hours %d", st->Hours)); |
900 | days = sd->Day; | 900 | days = sd->Day; |
901 | month = sd->Month; | 901 | month = sd->Month; |
902 | if ((days > 31) || (month > 12)) | 902 | if ((days > 31) || (month > 12)) { |
903 | cERROR(1, ("illegal date, month %d day: %d", month, days)); | 903 | cERROR(1, ("illegal date, month %d day: %d", month, days)); |
904 | if (month > 12) | ||
905 | month = 12; | ||
906 | } | ||
904 | month -= 1; | 907 | month -= 1; |
905 | days += total_days_of_prev_months[month]; | 908 | days += total_days_of_prev_months[month]; |
906 | days += 3652; /* account for difference in days between 1980 and 1970 */ | 909 | days += 3652; /* account for difference in days between 1980 and 1970 */ |