diff options
| author | Günther Deschner <gd@samba.org> | 2007-05-03 12:29:32 +0000 |
|---|---|---|
| committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 12:19:52 -0500 |
| commit | b213b35e08cb53eec47ceae87a52d3b0832a5914 (patch) | |
| tree | cb1a4812eafd25e1eb3d3feef25dfed62ba70931 /source3/libsmb/samlogon_cache.c | |
| parent | b48096e54618218d7484e70066d4f2459d4c172c (diff) | |
| download | samba-b213b35e08cb53eec47ceae87a52d3b0832a5914.tar.gz samba-b213b35e08cb53eec47ceae87a52d3b0832a5914.tar.bz2 samba-b213b35e08cb53eec47ceae87a52d3b0832a5914.zip | |
r22647: Avoid leaking a full info3 structure on each winbindd cached login by making
netsamlogon_cache_get() return a talloc'ed structure.
Guenther
(This used to be commit 5b149967cc3ab68057db015e67b688c9b9577f0d)
Diffstat (limited to 'source3/libsmb/samlogon_cache.c')
| -rw-r--r-- | source3/libsmb/samlogon_cache.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/source3/libsmb/samlogon_cache.c b/source3/libsmb/samlogon_cache.c index 3edbbaa2c1..0791cd80e4 100644 --- a/source3/libsmb/samlogon_cache.c +++ b/source3/libsmb/samlogon_cache.c @@ -192,10 +192,13 @@ NET_USER_INFO_3* netsamlogon_cache_get( TALLOC_CTX *mem_ctx, const DOM_SID *user data = tdb_fetch_bystring( netsamlogon_tdb, keystr ); if ( data.dptr ) { - - if ( (user = SMB_MALLOC_P(NET_USER_INFO_3)) == NULL ) + + + user = TALLOC_ZERO_P(mem_ctx, NET_USER_INFO_3); + if (user == NULL) { return NULL; - + } + prs_init( &ps, 0, mem_ctx, UNMARSHALL ); prs_give_memory( &ps, (char *)data.dptr, data.dsize, True ); @@ -247,7 +250,6 @@ BOOL netsamlogon_cache_have(const DOM_SID *user_sid) result = (user != NULL); talloc_destroy(mem_ctx); - SAFE_FREE(user); return result; } |