diff options
author | Jeremy Allison <jra@samba.org> | 2012-09-11 16:16:37 -0700 |
---|---|---|
committer | Jeremy Allison <jra@samba.org> | 2012-09-12 03:00:20 +0200 |
commit | bd2f1604d73f05f3b2f151a81f09824c7bb99ab5 (patch) | |
tree | a59d5c29f680ccab00c86e7bb0c1823a34d0eb63 | |
parent | 4e595f4bf08489cb69e871093804b9069d63d9f5 (diff) | |
download | samba-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.c | 6 |
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; } |