s3:ntlmssp: use a 'bool ok' variable to make the code more readable
authorStefan Metzmacher <metze@samba.org>
Thu, 7 Jan 2010 09:43:23 +0000 (10:43 +0100)
committerGünther Deschner <gd@samba.org>
Wed, 24 Mar 2010 16:34:56 +0000 (17:34 +0100)
metze

Signed-off-by: Günther Deschner <gd@samba.org>
source3/libsmb/ntlmssp_sign.c

index 37bb0efdeb382cac3bdc496c81868e77d91f8a8d..9c7bc241e1422cafae9cf0fda181522b187f360a 100644 (file)
@@ -125,10 +125,16 @@ static NTSTATUS ntlmssp_make_packet_signature(struct ntlmssp_state *ntlmssp_stat
                dump_data_pw("ntlmssp v2 sig ", sig->data, sig->length);
 
        } else {
+               bool ok;
                uint32_t crc;
 
                crc = crc32_calc_buffer(data, length);
-               if (!msrpc_gen(ntlmssp_state, sig, "dddd", NTLMSSP_SIGN_VERSION, 0, crc, ntlmssp_state->ntlmv1_seq_num)) {
+
+               ok = msrpc_gen(ntlmssp_state,
+                              sig, "dddd",
+                              NTLMSSP_SIGN_VERSION, 0, crc,
+                              ntlmssp_state->ntlmv1_seq_num);
+               if (!ok) {
                        return NT_STATUS_NO_MEMORY;
                }
 
@@ -285,10 +291,16 @@ NTSTATUS ntlmssp_seal_packet(struct ntlmssp_state *ntlmssp_state,
                                           sig->data+4, 8);
                }
        } else {
+               bool ok;
                uint32_t crc;
 
                crc = crc32_calc_buffer(data, length);
-               if (!msrpc_gen(ntlmssp_state, sig, "dddd", NTLMSSP_SIGN_VERSION, 0, crc, ntlmssp_state->ntlmv1_seq_num)) {
+
+               ok = msrpc_gen(ntlmssp_state,
+                              sig, "dddd",
+                              NTLMSSP_SIGN_VERSION, 0, crc,
+                              ntlmssp_state->ntlmv1_seq_num);
+               if (!ok) {
                        return NT_STATUS_NO_MEMORY;
                }