From: Matthias Dieter Wallnöfer <mdw@samba.org>
Date: Wed, 16 Feb 2011 08:22:43 +0000 (+0100)
Subject: libcli/security/security_descriptor.c - fix three wrong counter variables
X-Git-Tag: tevent-0.9.11~446
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=d2b336ae319053dee96476131a3dc0c45b306b5f;p=thirdparty%2Fsamba.git

libcli/security/security_descriptor.c - fix three wrong counter variables

These strictly need to be "uint32_t" since "acl*->num_aces" has been
defined by this type.

This counter patchset has been reviewed by Andrew Bartlett.
---

diff --git a/libcli/security/security_descriptor.c b/libcli/security/security_descriptor.c
index 60f9de8f879..574dd4e317b 100644
--- a/libcli/security/security_descriptor.c
+++ b/libcli/security/security_descriptor.c
@@ -85,7 +85,7 @@ struct security_acl *security_acl_concatenate(TALLOC_CTX *mem_ctx,
                                               const struct security_acl *acl2)
 {
         struct security_acl *nacl;
-        int i;
+        uint32_t i;
 
         if (!acl1 && !acl2)
                 return NULL;
@@ -270,7 +270,7 @@ static NTSTATUS security_descriptor_acl_del(struct security_descriptor *sd,
 					    bool sacl_del,
 					    const struct dom_sid *trustee)
 {
-	int i;
+	uint32_t i;
 	bool found = false;
 	struct security_acl *acl = NULL;
 
@@ -364,7 +364,7 @@ bool security_ace_equal(const struct security_ace *ace1,
 bool security_acl_equal(const struct security_acl *acl1, 
 			const struct security_acl *acl2)
 {
-	int i;
+	uint32_t i;
 
 	if (acl1 == acl2) return true;
 	if (!acl1 || !acl2) return false;