diff options
| author | Michael Adam <obnox@samba.org> | 2016-03-01 20:40:47 +0100 |
|---|---|---|
| committer | Karolin Seeger <kseeger@samba.org> | 2016-03-14 10:02:14 +0100 |
| commit | 2d822821647a69d82327402e6be9c4294a08cc6b (patch) | |
| tree | 4e4df702ac1b59beb6fef42438cb5176676e8eff /source3 | |
| parent | 5782344a5775356b37cafe8b6631786fc6ca6364 (diff) | |
| download | samba-2d822821647a69d82327402e6be9c4294a08cc6b.tar.gz | |
smbd: fix crash in smbXsrv_client_global_remove()
Probably copy-n-paste error.
Uncovered by the multi-channel-related tests we're
currently writing to exercise this code more.
Pair-Programmed-With: Guenther Deschner <gd@samba.org>
Signed-off-by: Michael Adam <obnox@samba.org>
Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
(cherry picked from commit 04265199b389a7b38c3047985da68e8442a0b746)
BUG: https://bugzilla.samba.org/show_bug.cgi?id=11779
Diffstat (limited to 'source3')
| -rw-r--r-- | source3/smbd/smbXsrv_client.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/source3/smbd/smbXsrv_client.c b/source3/smbd/smbXsrv_client.c index 2dd4cad85a8..d8ba2f7843d 100644 --- a/source3/smbd/smbXsrv_client.c +++ b/source3/smbd/smbXsrv_client.c @@ -428,7 +428,6 @@ static NTSTATUS smbXsrv_client_global_store(struct smbXsrv_client_global0 *globa static NTSTATUS smbXsrv_client_global_remove(struct smbXsrv_client_global0 *global) { - struct smbXsrv_client_globalB global_blob; TDB_DATA key; NTSTATUS status; @@ -453,11 +452,8 @@ static NTSTATUS smbXsrv_client_global_remove(struct smbXsrv_client_global0 *glob return status; } global->stored = false; - if (DEBUGLVL(DBGLVL_DEBUG)) { - DBG_DEBUG("key '%s' delete\n", - hex_encode_talloc(global->db_rec, key.dptr, key.dsize)); - NDR_PRINT_DEBUG(smbXsrv_client_globalB, &global_blob); - } + DBG_DEBUG("key '%s' delete\n", + hex_encode_talloc(global->db_rec, key.dptr, key.dsize)); TALLOC_FREE(global->db_rec); |