diff options
author | Volker Lendecke <vlendec@samba.org> | 2006-03-12 18:01:36 +0000 |
---|---|---|
committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 11:15:20 -0500 |
commit | 84e9e73f3c71a0ccef76d56bc72dcd21160ed286 (patch) | |
tree | 19f45cbbb13badd0fd62a8e606759a0524b8d9be | |
parent | 811ae2b21f98bd8926f8edd70de19fe18265e28e (diff) | |
download | samba-84e9e73f3c71a0ccef76d56bc72dcd21160ed286.tar.gz samba-84e9e73f3c71a0ccef76d56bc72dcd21160ed286.tar.xz samba-84e9e73f3c71a0ccef76d56bc72dcd21160ed286.zip |
r14249: We've dereferenced the_acl before, no point in checking.
Fix Coverity bug # 128.
Volker
-rw-r--r-- | source/lib/secdesc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source/lib/secdesc.c b/source/lib/secdesc.c index a443f90dbb4..11f963fa0e8 100644 --- a/source/lib/secdesc.c +++ b/source/lib/secdesc.c @@ -433,7 +433,7 @@ SEC_DESC_BUF *se_create_child_secdesc(TALLOC_CTX *ctx, SEC_DESC *parent_ctr, if (!(new_ace_list = TALLOC_ARRAY(ctx, SEC_ACE, the_acl->num_aces))) return NULL; - for (i = 0; the_acl && i < the_acl->num_aces; i++) { + for (i = 0; i < the_acl->num_aces; i++) { SEC_ACE *ace = &the_acl->ace[i]; SEC_ACE *new_ace = &new_ace_list[new_ace_list_ndx]; uint8 new_flags = 0; |