summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJeremy Allison <jra@samba.org>2012-09-11 16:16:37 -0700
committerJeremy Allison <jra@samba.org>2012-09-12 03:00:20 +0200
commitbd2f1604d73f05f3b2f151a81f09824c7bb99ab5 (patch)
treea59d5c29f680ccab00c86e7bb0c1823a34d0eb63
parent4e595f4bf08489cb69e871093804b9069d63d9f5 (diff)
downloadsamba-bd2f1604d73f05f3b2f151a81f09824c7bb99ab5.tar.gz
samba-bd2f1604d73f05f3b2f151a81f09824c7bb99ab5.tar.bz2
samba-bd2f1604d73f05f3b2f151a81f09824c7bb99ab5.zip
Make metze happy and the code clearer :-).
Ensure we know after the destructor fires we're never going to look at this again. Autobuild-User(master): Jeremy Allison <jra@samba.org> Autobuild-Date(master): Wed Sep 12 03:00:21 CEST 2012 on sn-devel-104
-rw-r--r--source3/smbd/smb2_sesssetup.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/source3/smbd/smb2_sesssetup.c b/source3/smbd/smb2_sesssetup.c
index 61b5519d97..2599d2a63d 100644
--- a/source3/smbd/smb2_sesssetup.c
+++ b/source3/smbd/smb2_sesssetup.c
@@ -446,6 +446,12 @@ struct smbd_smb2_session_setup_state {
static int pp_self_ref_destructor(struct smbd_smb2_session_setup_state **pp_state)
{
(*pp_state)->session = NULL;
+ /*
+ * To make things clearer, ensure the pp_self_ref
+ * pointer is nulled out. We're never going to
+ * access this again.
+ */
+ (*pp_state)->pp_self_ref = NULL;
return 0;
}