diff options
| author | Jeremy Allison <jra@samba.org> | 2003-02-21 06:25:58 +0000 |
|---|---|---|
| committer | Jeremy Allison <jra@samba.org> | 2003-02-21 06:25:58 +0000 |
| commit | 486fa37dc67b5b3a9534479fed55ae9298fb2097 (patch) | |
| tree | ce9670ad3e37d5aeb7eff7ee4731b35ee5d46e73 /source3 | |
| parent | e5249ff9b04c1e95da56d578b0704dcdc153c08a (diff) | |
| download | samba-486fa37dc67b5b3a9534479fed55ae9298fb2097.tar.gz samba-486fa37dc67b5b3a9534479fed55ae9298fb2097.tar.bz2 samba-486fa37dc67b5b3a9534479fed55ae9298fb2097.zip | |
Added comments to make it clearer when we're assigning a pointer that it
must not be freed afterwards.
Jeremy.
(This used to be commit 4015e39d3666dbe240808c9007a8b8faca012a3d)
Diffstat (limited to 'source3')
| -rw-r--r-- | source3/smbd/posix_acls.c | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/source3/smbd/posix_acls.c b/source3/smbd/posix_acls.c index 0c8b08e4d5..ee8a4430e1 100644 --- a/source3/smbd/posix_acls.c +++ b/source3/smbd/posix_acls.c @@ -1000,8 +1000,16 @@ Deny entry after Allow entry. Failing to set on file %s.\n", fsp->fsp_name )); return False; } + /* + * We must not free current_ace here as its + * pointer is now owned by the dir_ace list. + */ current_ace = dup_ace; } else { + /* + * We must not free current_ace here as its + * pointer is now owned by the dir_ace list. + */ current_ace = NULL; } } @@ -1036,6 +1044,10 @@ Deny entry after Allow entry. Failing to set on file %s.\n", fsp->fsp_name )); print_canon_ace( current_ace, 0); } all_aces_are_inherit_only = False; + /* + * We must not free current_ace here as its + * pointer is now owned by the file_ace list. + */ current_ace = NULL; } |