Add explicit buf arg to cli_check_sign_mac

diff --git a/source/libsmb/cliconnect.c b/source/libsmb/cliconnect.c
index 9c27d30166df48d82b3560d105bd2da360f51639..912b841d5ebcaa586af2ec9ce7cde805db97d7fa 100644 (file)
--- a/source/libsmb/cliconnect.c
+++ b/source/libsmb/cliconnect.c
@@ -759,7 +759,7 @@ static NTSTATUS cli_session_setup_ntlmssp(struct cli_state *cli, const char *use
                           for checking the first reply from the server */
                        cli_calculate_sign_mac(cli, cli->outbuf);
                        
-                       if (!cli_check_sign_mac(cli)) {
+                       if (!cli_check_sign_mac(cli, cli->inbuf)) {
                                nt_status = NT_STATUS_ACCESS_DENIED;
                        }
                }

diff --git a/source/libsmb/clientgen.c b/source/libsmb/clientgen.c
index 2fd304f13500b8cd948700f624d16884eb2e9b70..ccd1cc67d527b5f9a33c24eb0569e7fa7c3107c8 100644 (file)
--- a/source/libsmb/clientgen.c
+++ b/source/libsmb/clientgen.c
@@ -164,7 +164,7 @@ bool cli_receive_smb(struct cli_state *cli)
                return false;
        }
 
-       if (!cli_check_sign_mac(cli)) {
+       if (!cli_check_sign_mac(cli, cli->inbuf)) {
                /*
                 * If we get a signature failure in sessionsetup, then
                 * the server sometimes just reflects the sent signature

diff --git a/source/libsmb/smb_signing.c b/source/libsmb/smb_signing.c
index eeaf28c3d1d095876cbe8eb6b1e9e570c9143e13..bd6d97123d66ae4094fc7f53ac6eaf4be322a6d9 100644 (file)
--- a/source/libsmb/smb_signing.c
+++ b/source/libsmb/smb_signing.c
@@ -584,9 +584,9 @@ void cli_calculate_sign_mac(struct cli_state *cli, char *buf)
  *         which had a bad checksum, True otherwise.
  */
  
-bool cli_check_sign_mac(struct cli_state *cli) 
+bool cli_check_sign_mac(struct cli_state *cli, char *buf)
 {
-       if (!cli->sign_info.check_incoming_message(cli->inbuf, &cli->sign_info, True)) {
+       if (!cli->sign_info.check_incoming_message(buf, &cli->sign_info, True)) {
                free_signing_context(&cli->sign_info);  
                return False;
        }