diff options
author | Steve French <sfrench@us.ibm.com> | 2006-03-09 17:21:45 -0500 |
---|---|---|
committer | Steve French <sfrench@us.ibm.com> | 2006-03-09 17:21:45 -0500 |
commit | 88274815f7477dc7550439413ab87c5ce4c5a623 (patch) | |
tree | 433e191b5b08742a57f2339008cdf24ca5b51221 /fs/cifs/cifssmb.c | |
parent | f6e77c94796edad172602234826a022d463d910e (diff) |
[CIFS] Fix two remaining coverity scan tool warnings.
Signed-off-by: Steve French <sfrench@us.ibm.com>
Diffstat (limited to 'fs/cifs/cifssmb.c')
-rw-r--r-- | fs/cifs/cifssmb.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/fs/cifs/cifssmb.c b/fs/cifs/cifssmb.c index 3e89fbbd160f..a5941872d3f7 100644 --- a/fs/cifs/cifssmb.c +++ b/fs/cifs/cifssmb.c | |||
@@ -3061,7 +3061,8 @@ findFirstRetry: | |||
3061 | pSMB->TotalParameterCount = cpu_to_le16(params); | 3061 | pSMB->TotalParameterCount = cpu_to_le16(params); |
3062 | pSMB->ParameterCount = pSMB->TotalParameterCount; | 3062 | pSMB->ParameterCount = pSMB->TotalParameterCount; |
3063 | pSMB->ParameterOffset = cpu_to_le16( | 3063 | pSMB->ParameterOffset = cpu_to_le16( |
3064 | offsetof(struct smb_com_transaction2_ffirst_req, SearchAttributes) - 4); | 3064 | offsetof(struct smb_com_transaction2_ffirst_req, SearchAttributes) |
3065 | - 4); | ||
3065 | pSMB->DataCount = 0; | 3066 | pSMB->DataCount = 0; |
3066 | pSMB->DataOffset = 0; | 3067 | pSMB->DataOffset = 0; |
3067 | pSMB->SetupCount = 1; /* one byte, no need to make endian neutral */ | 3068 | pSMB->SetupCount = 1; /* one byte, no need to make endian neutral */ |
@@ -3084,12 +3085,12 @@ findFirstRetry: | |||
3084 | (struct smb_hdr *) pSMBr, &bytes_returned, 0); | 3085 | (struct smb_hdr *) pSMBr, &bytes_returned, 0); |
3085 | cifs_stats_inc(&tcon->num_ffirst); | 3086 | cifs_stats_inc(&tcon->num_ffirst); |
3086 | 3087 | ||
3087 | if (rc) {/* BB add logic to retry regular search if Unix search rejected unexpectedly by server */ | 3088 | if (rc) {/* BB add logic to retry regular search if Unix search |
3089 | rejected unexpectedly by server */ | ||
3088 | /* BB Add code to handle unsupported level rc */ | 3090 | /* BB Add code to handle unsupported level rc */ |
3089 | cFYI(1, ("Error in FindFirst = %d", rc)); | 3091 | cFYI(1, ("Error in FindFirst = %d", rc)); |
3090 | 3092 | ||
3091 | if (pSMB) | 3093 | cifs_buf_release(pSMB); |
3092 | cifs_buf_release(pSMB); | ||
3093 | 3094 | ||
3094 | /* BB eventually could optimize out free and realloc of buf */ | 3095 | /* BB eventually could optimize out free and realloc of buf */ |
3095 | /* for this case */ | 3096 | /* for this case */ |