Revert "avoid SMB_ASSERT(!req->request_counters_updated); on re-dispatched requests"

This reverts commit d5f6579317977618ecb215e709861ff510709ae9.

diff --git a/source3/smbd/globals.h b/source3/smbd/globals.h
index 171bed2ed31aed977dad0bddfb2fd633e39e3960..1e4cfc1e4667768c5b63d2f3e027129ddcb3cd8b 100644 (file)
--- a/source3/smbd/globals.h
+++ b/source3/smbd/globals.h
@@ -723,7 +723,6 @@ struct smbd_smb2_request {
         * had been updated in dispatch, so that they need to be
         * adapted again in reply.
         */
-       bool re_dispatching;
        bool request_counters_updated;
        uint64_t channel_generation;
 

diff --git a/source3/smbd/smb2_create.c b/source3/smbd/smb2_create.c
index 3b1fb25f9b60444e77b0356afab4e88d9f6a541b..6cf3499c411c8dad84afc4aaf996ddcb5a95fe42 100644 (file)
--- a/source3/smbd/smb2_create.c
+++ b/source3/smbd/smb2_create.c
@@ -1745,7 +1745,6 @@ static void smbd_smb2_create_request_dispatch_immediate(struct tevent_context *c
                "re-dispatching mid %llu\n",
                (unsigned long long)mid ));
 
-       smb2req->re_dispatching = true;
        status = smbd_smb2_request_dispatch(smb2req);
        if (!NT_STATUS_IS_OK(status)) {
                smbd_server_connection_terminate(smb2req->xconn,

diff --git a/source3/smbd/smb2_server.c b/source3/smbd/smb2_server.c
index 6871d98e4384ba13eafa60c6dcc1cbabe93c25c5..5a98a2e82d4d4e21b9f07ed4d28f966dffe15621 100644 (file)
--- a/source3/smbd/smb2_server.c
+++ b/source3/smbd/smb2_server.c
@@ -2210,10 +2210,6 @@ static NTSTATUS smbd_smb2_request_dispatch_update_counts(
        bool update_open = false;
        NTSTATUS status = NT_STATUS_OK;
 
-       if (!smb2req->re_dispatching) {
-               return NT_STATUS_OK;
-       }
-
        SMB_ASSERT(!req->request_counters_updated);
 
        if (xconn->protocol < PROTOCOL_SMB2_22) {
@@ -2349,9 +2345,7 @@ NTSTATUS smbd_smb2_request_dispatch(struct smbd_smb2_request *req)
 
        DO_PROFILE_INC(request);
 
-       if (!smb2req->re_dispatching) {
-               SMB_ASSERT(!req->request_counters_updated);
-       }
+       SMB_ASSERT(!req->request_counters_updated);
 
        /* TODO: verify more things */
 
@@ -2857,7 +2851,6 @@ static NTSTATUS smbd_smb2_request_reply(struct smbd_smb2_request *req)
 
        /* MS-SMB2: 3.3.4.1 Sending Any Outgoing Message */
        smbd_smb2_request_reply_update_counts(req);
-       req->re_dispatching = false;
 
        if (req->do_encryption &&
            (firsttf->iov_len == 0) &&