diff options
| author | Andrew Tridgell <tridge@samba.org> | 2006-09-28 06:44:47 +0000 |
|---|---|---|
| committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 14:20:25 -0500 |
| commit | 6cce709d08579f4e00b44b692332a557b0ea3b86 (patch) | |
| tree | 505c7a289a01a0ff2b67d4a2ccb8b66f90577bc6 /source/libcli/auth | |
| parent | c178c84f01166609e6bd3393d39fb0034130167b (diff) | |
| download | samba-6cce709d08579f4e00b44b692332a557b0ea3b86.tar.gz | |
r18971: avoid strndup is a few places. Fixes a minor memory leak, and should
fix RPC-LSA on AIX.
Diffstat (limited to 'source/libcli/auth')
| -rw-r--r-- | source/libcli/auth/session.c | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/source/libcli/auth/session.c b/source/libcli/auth/session.c index 280a0d282c0..430eecd78fb 100644 --- a/source/libcli/auth/session.c +++ b/source/libcli/auth/session.c @@ -97,7 +97,8 @@ DATA_BLOB sess_encrypt_string(const char *str, const DATA_BLOB *session_key) caller should free the returned string */ -char *sess_decrypt_string(DATA_BLOB *blob, const DATA_BLOB *session_key) +char *sess_decrypt_string(TALLOC_CTX *mem_ctx, + DATA_BLOB *blob, const DATA_BLOB *session_key) { DATA_BLOB out; int slen; @@ -107,7 +108,7 @@ char *sess_decrypt_string(DATA_BLOB *blob, const DATA_BLOB *session_key) return NULL; } - out = data_blob(NULL, blob->length); + out = data_blob_talloc(mem_ctx, NULL, blob->length); if (!out.data) { return NULL; } @@ -117,19 +118,23 @@ char *sess_decrypt_string(DATA_BLOB *blob, const DATA_BLOB *session_key) if (IVAL(out.data, 4) != 1) { DEBUG(0,("Unexpected revision number %d in session crypted string\n", IVAL(out.data, 4))); + data_blob_free(&out); return NULL; } slen = IVAL(out.data, 0); if (slen > blob->length - 8) { DEBUG(0,("Invalid crypt length %d\n", slen)); + data_blob_free(&out); return NULL; } - ret = strndup((const char *)(out.data+8), slen); + ret = talloc_strndup(mem_ctx, (const char *)(out.data+8), slen); data_blob_free(&out); + DEBUG(0,("decrypted string '%s' of length %d\n", ret, slen)); + return ret; } |