Signed-off-by: Pavel Shilovsky <[email protected]>
---
 fs/cifs/connect.c |  136 ++++++++++++++++++++++++++++-------------------------
 1 files changed, 72 insertions(+), 64 deletions(-)

diff --git a/fs/cifs/connect.c b/fs/cifs/connect.c
index 26498d3..aa2925b 100644
--- a/fs/cifs/connect.c
+++ b/fs/cifs/connect.c
@@ -466,6 +466,76 @@ check_rfc1002_header(struct TCP_Server_Info *server, char 
*buf)
        return true;
 }
 
+static struct mid_q_entry *
+find_cifs_mid(struct TCP_Server_Info *server, struct smb_hdr *buf,
+             int *length, bool isLargeBuf, bool *isMultiRsp, char **pbigbuf)
+{
+       struct list_head *tmp, *tmp2;
+       struct mid_q_entry *mid_entry = NULL;
+
+       spin_lock(&GlobalMid_Lock);
+       list_for_each_safe(tmp, tmp2, &server->pending_mid_q) {
+               mid_entry = list_entry(tmp, struct mid_q_entry, qhead);
+
+               if (mid_entry->mid != buf->Mid ||
+                   mid_entry->midState != MID_REQUEST_SUBMITTED ||
+                   mid_entry->command != buf->Command) {
+                       mid_entry = NULL;
+                       continue;
+               }
+
+               if (*length == 0 && check2ndT2(buf, server->maxBuf) > 0) {
+                       /* We have a multipart transact2 resp */
+                       *isMultiRsp = true;
+                       if (mid_entry->resp_buf) {
+                               /* merge response - fix up 1st*/
+                               *length = coalesce_t2(buf, mid_entry->resp_buf);
+                               if (*length > 0) {
+                                       *length = 0;
+                                       mid_entry->multiRsp = true;
+                                       break;
+                               } else {
+                                       /* all parts received or
+                                        * packet is malformed
+                                        */
+                                       mid_entry->multiEnd = true;
+                                       goto multi_t2_fnd;
+                               }
+                       } else {
+                               if (!isLargeBuf) {
+                                       /*
+                                        * FIXME: switch to already
+                                        *        allocated largebuf?
+                                        */
+                                       cERROR(1, "1st trans2 resp "
+                                                 "needs bigbuf");
+                               } else {
+                                       /* Have first buffer */
+                                       mid_entry->resp_buf = buf;
+                                       mid_entry->largeBuf = true;
+                                       *pbigbuf = NULL;
+                               }
+                       }
+                       break;
+               }
+               mid_entry->resp_buf = buf;
+               mid_entry->largeBuf = isLargeBuf;
+multi_t2_fnd:
+               if (*length == 0)
+                       mid_entry->midState = MID_RESPONSE_RECEIVED;
+               else
+                       mid_entry->midState = MID_RESPONSE_MALFORMED;
+#ifdef CONFIG_CIFS_STATS2
+               mid_entry->when_received = jiffies;
+#endif
+               list_del_init(&mid_entry->qhead);
+               break;
+       }
+       spin_unlock(&GlobalMid_Lock);
+
+       return mid_entry;
+}
+
 static int
 cifs_demultiplex_thread(struct TCP_Server_Info *server)
 {
@@ -577,72 +647,10 @@ incomplete_rcv:
                        cifs_dump_mem("Bad SMB: ", smb_buffer,
                                        min_t(unsigned int, total_read, 48));
 
-               mid_entry = NULL;
                server->lstrp = jiffies;
 
-               spin_lock(&GlobalMid_Lock);
-               list_for_each_safe(tmp, tmp2, &server->pending_mid_q) {
-                       mid_entry = list_entry(tmp, struct mid_q_entry, qhead);
-
-                       if (mid_entry->mid != smb_buffer->Mid ||
-                           mid_entry->midState != MID_REQUEST_SUBMITTED ||
-                           mid_entry->command != smb_buffer->Command) {
-                               mid_entry = NULL;
-                               continue;
-                       }
-
-                       if (length == 0 &&
-                           check2ndT2(smb_buffer, server->maxBuf) > 0) {
-                               /* We have a multipart transact2 resp */
-                               isMultiRsp = true;
-                               if (mid_entry->resp_buf) {
-                                       /* merge response - fix up 1st*/
-                                       length = coalesce_t2(smb_buffer,
-                                                       mid_entry->resp_buf);
-                                       if (length > 0) {
-                                               length = 0;
-                                               mid_entry->multiRsp = true;
-                                               break;
-                                       } else {
-                                               /* all parts received or
-                                                * packet is malformed
-                                                */
-                                               mid_entry->multiEnd = true;
-                                               goto multi_t2_fnd;
-                                       }
-                               } else {
-                                       if (!isLargeBuf) {
-                                               /*
-                                                * FIXME: switch to already
-                                                *        allocated largebuf?
-                                                */
-                                               cERROR(1, "1st trans2 resp "
-                                                         "needs bigbuf");
-                                       } else {
-                                               /* Have first buffer */
-                                               mid_entry->resp_buf =
-                                                        smb_buffer;
-                                               mid_entry->largeBuf = true;
-                                               bigbuf = NULL;
-                                       }
-                               }
-                               break;
-                       }
-                       mid_entry->resp_buf = smb_buffer;
-                       mid_entry->largeBuf = isLargeBuf;
-multi_t2_fnd:
-                       if (length == 0)
-                               mid_entry->midState = MID_RESPONSE_RECEIVED;
-                       else
-                               mid_entry->midState = MID_RESPONSE_MALFORMED;
-#ifdef CONFIG_CIFS_STATS2
-                       mid_entry->when_received = jiffies;
-#endif
-                       list_del_init(&mid_entry->qhead);
-                       break;
-               }
-               spin_unlock(&GlobalMid_Lock);
-
+               mid_entry = find_cifs_mid(server, smb_buffer, &length,
+                                         isLargeBuf, &isMultiRsp, &bigbuf);
                if (mid_entry != NULL) {
                        mid_entry->callback(mid_entry);
                        /* Was previous buf put in mpx struct for multi-rsp? */
-- 
1.7.1

--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to