summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatthias Dieter Wallnöfer <mwallnoefer@yahoo.de>2009-11-06 18:35:17 +0100
committerMatthias Dieter Wallnöfer <mwallnoefer@yahoo.de>2010-03-02 18:02:00 +0100
commite50c9826794f4a80b7200712afacb0752ccf8d6f (patch)
tree599e14ecbd512cfbf5fde5e7af9edb9fb6f97e36
parent5c73ecd2b3b178dd57d7251342e966ea4f8044e0 (diff)
downloadsamba-e50c9826794f4a80b7200712afacb0752ccf8d6f.tar.gz
samba-e50c9826794f4a80b7200712afacb0752ccf8d6f.tar.bz2
samba-e50c9826794f4a80b7200712afacb0752ccf8d6f.zip
LDB:NSS - make LDB "signed-safe" on counter variables
"i" needs to be unsigned on both places since it counts till a "count" variable of a "struct ldb_result" object which itself is unsigned. I see counting variables much better as "unsigned" since in most cases we don't use negative values at all. We've only to be careful on binary searches and downto counts regarding them.
-rw-r--r--source4/lib/ldb/nssldb/ldb-nss.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/source4/lib/ldb/nssldb/ldb-nss.c b/source4/lib/ldb/nssldb/ldb-nss.c
index 8f7321031b..92b0635561 100644
--- a/source4/lib/ldb/nssldb/ldb-nss.c
+++ b/source4/lib/ldb/nssldb/ldb-nss.c
@@ -191,7 +191,7 @@ NSS_STATUS _ldb_nss_fill_group(struct group *result,
size_t len;
size_t bufpos;
size_t lsize;
- int i;
+ unsigned int i;
bufpos = 0;
@@ -280,7 +280,7 @@ NSS_STATUS _ldb_nss_fill_initgr(gid_t group,
struct ldb_result *grlist)
{
NSS_STATUS ret;
- int i;
+ unsigned int i;
for (i = 0; i < grlist->count; i++) {