r3805: Fix the LSA portions of the RPC-SAMSYNC test - I was not using the LSA

secrets interface correctly.  (New interface added).

Andrew Bartlett

diff --git a/source/libcli/auth/session.c b/source/libcli/auth/session.c
index 598f2d4f2814202dd853bc7bb450b913ceea9419..fdb6462b005e9318b9813259ee423d412a821314 100644 (file)
--- a/source/libcli/auth/session.c
+++ b/source/libcli/auth/session.c
@@ -131,3 +131,80 @@ char *sess_decrypt_string(DATA_BLOB *blob, const DATA_BLOB *session_key)
 
        return ret;
 }
+
+/*
+  a convenient wrapper around sess_crypt_blob() for DATA_BLOBs, using the LSA convention
+
+  note that we round the length to a multiple of 8. This seems to be needed for 
+  compatibility with windows
+
+  caller should free using data_blob_free()
+*/
+DATA_BLOB sess_encrypt_blob(TALLOC_CTX *mem_ctx, DATA_BLOB *blob_in, const DATA_BLOB *session_key)
+{
+       DATA_BLOB ret, src;
+       int dlen = (blob_in->length+7) & ~7;
+
+       src = data_blob_talloc(mem_ctx, NULL, 8+dlen);
+       if (!src.data) {
+               return data_blob(NULL, 0);
+       }
+
+       ret = data_blob(NULL, 8+dlen);
+       if (!ret.data) {
+               data_blob_free(&src);
+               return data_blob(NULL, 0);
+       }
+
+       SIVAL(src.data, 0, blob_in->length);
+       SIVAL(src.data, 4, 1);
+       memset(src.data+8, 0, dlen);
+       memcpy(src.data+8, blob_in->data, blob_in->length);
+
+       sess_crypt_blob(&ret, &src, session_key, True);
+       
+       data_blob_free(&src);
+
+       return ret;
+}
+
+/*
+  a convenient wrapper around sess_crypt_blob() for strings, using the LSA convention
+
+  caller should free the returned string
+*/
+DATA_BLOB sess_decrypt_blob(TALLOC_CTX *mem_ctx, DATA_BLOB *blob, const DATA_BLOB *session_key)
+{
+       DATA_BLOB out;
+       int slen;
+       DATA_BLOB ret;
+
+       if (blob->length < 8) {
+               return data_blob(NULL, 0);
+       }
+       
+       out = data_blob_talloc(mem_ctx, NULL, blob->length);
+       if (!out.data) {
+               return data_blob(NULL, 0);
+       }
+
+       sess_crypt_blob(&out, blob, session_key, False);
+
+       slen = IVAL(out.data, 0);
+       if (slen > blob->length - 8) {
+               DEBUG(0,("Invalid crypt length %d\n", slen));
+               return data_blob(NULL, 0);
+       }
+
+       if (IVAL(out.data, 4) != 1) {
+               DEBUG(0,("Unexpected revision number %d in session crypted string\n",
+                        IVAL(out.data, 4)));
+               return data_blob(NULL, 0);
+       }
+               
+       ret = data_blob_talloc(mem_ctx, out.data+8, slen);
+
+       data_blob_free(&out);
+
+       return ret;
+}

diff --git a/source/torture/rpc/samsync.c b/source/torture/rpc/samsync.c
index 9b5df2b6814847ef42edbd5afff6ae6679a30698..96c7846c59f8b26935d7b3014c418d3cf69ee6d8 100644 (file)
--- a/source/torture/rpc/samsync.c
+++ b/source/torture/rpc/samsync.c
@@ -583,9 +583,7 @@ static BOOL samsync_handle_secret(TALLOC_CTX *mem_ctx, struct samsync_state *sam
                lsa_blob1.data = q.out.new_val->buf->data;
                lsa_blob1.length = q.out.new_val->buf->length;
 
-               lsa_blob_out = data_blob(NULL, lsa_blob1.length);
-
-               sess_crypt_blob(&lsa_blob_out, &lsa_blob1, &session_key, 0);
+               lsa_blob_out = sess_decrypt_blob(mem_ctx, &lsa_blob1, &session_key);
                
                if (new->secret.length != lsa_blob_out.length) {
                        printf("Returned secret %s doesn't match: %d != %d\n",