summaryrefslogtreecommitdiff
path: root/source4/lib/ldb/ldb_tdb
diff options
context:
space:
mode:
Diffstat (limited to 'source4/lib/ldb/ldb_tdb')
-rw-r--r--source4/lib/ldb/ldb_tdb/ldb_index.c38
-rw-r--r--source4/lib/ldb/ldb_tdb/ldb_match.c9
-rw-r--r--source4/lib/ldb/ldb_tdb/ldb_pack.c6
-rw-r--r--source4/lib/ldb/ldb_tdb/ldb_search.c8
-rw-r--r--source4/lib/ldb/ldb_tdb/ldb_tdb.c18
5 files changed, 44 insertions, 35 deletions
diff --git a/source4/lib/ldb/ldb_tdb/ldb_index.c b/source4/lib/ldb/ldb_tdb/ldb_index.c
index b4ca666287..8a26e2d197 100644
--- a/source4/lib/ldb/ldb_tdb/ldb_index.c
+++ b/source4/lib/ldb/ldb_tdb/ldb_index.c
@@ -46,7 +46,7 @@ struct dn_list {
*/
static void dn_list_free(struct ldb_context *ldb, struct dn_list *list)
{
- int i;
+ unsigned int i;
for (i=0;i<list->count;i++) {
ldb_free(ldb, list->dn[i]);
}
@@ -80,7 +80,7 @@ static char *ldb_dn_key(struct ldb_context *ldb,
static int ldb_msg_find_idx(const struct ldb_message *msg, const char *attr,
int *v_idx, const char *key)
{
- int i, j;
+ unsigned int i, j;
for (i=0;i<msg->num_elements;i++) {
if (ldb_attr_cmp(msg->elements[i].name, key) == 0) {
const struct ldb_message_element *el =
@@ -113,7 +113,8 @@ static int ltdb_index_dn_simple(struct ldb_context *ldb,
struct dn_list *list)
{
char *dn = NULL;
- int ret, i, j;
+ int ret;
+ unsigned int i, j;
struct ldb_message msg;
list->count = 0;
@@ -186,7 +187,7 @@ static int ltdb_index_dn_objectclass(struct ldb_context *ldb,
struct dn_list *list)
{
struct ltdb_private *ltdb = ldb->private_data;
- int i;
+ unsigned int i;
int ret;
const char *target = tree->u.simple.value.data;
static int list_union(struct ldb_context *,
@@ -200,7 +201,7 @@ static int ltdb_index_dn_objectclass(struct ldb_context *ldb,
for (i=0;i<ltdb->cache.subclasses.num_elements;i++) {
struct ldb_message_element *el = &ltdb->cache.subclasses.elements[i];
if (ldb_attr_cmp(el->name, target) == 0) {
- int j;
+ unsigned int j;
for (j=0;j<el->num_values;j++) {
struct ldb_parse_tree tree2;
struct dn_list list2;
@@ -252,7 +253,7 @@ static int list_intersect(struct ldb_context *ldb,
struct dn_list *list, const struct dn_list *list2)
{
struct dn_list list3;
- int i;
+ unsigned int i;
if (list->count == 0 || list2->count == 0) {
/* 0 & X == 0 */
@@ -293,7 +294,7 @@ static int list_intersect(struct ldb_context *ldb,
static int list_union(struct ldb_context *ldb,
struct dn_list *list, const struct dn_list *list2)
{
- int i;
+ unsigned int i;
char **d;
unsigned int count = list->count;
@@ -343,7 +344,8 @@ static int ltdb_index_dn_or(struct ldb_context *ldb,
const struct ldb_message *index_list,
struct dn_list *list)
{
- int ret, i;
+ unsigned int i;
+ int ret;
ret = -1;
list->dn = NULL;
@@ -416,7 +418,8 @@ static int ltdb_index_dn_and(struct ldb_context *ldb,
const struct ldb_message *index_list,
struct dn_list *list)
{
- int ret, i;
+ unsigned int i;
+ int ret;
ret = -1;
list->dn = NULL;
@@ -499,8 +502,7 @@ static int ldb_index_filter(struct ldb_context *ldb, struct ldb_parse_tree *tree
const struct dn_list *dn_list,
const char * const attrs[], struct ldb_message ***res)
{
- int i;
- unsigned int count = 0;
+ unsigned int count = 0, i;
for (i=0;i<dn_list->count;i++) {
struct ldb_message msg;
@@ -608,7 +610,7 @@ static int ltdb_index_add1_add(struct ldb_context *ldb,
char *dn)
{
struct ldb_val *v2;
- int i;
+ unsigned int i;
/* for multi-valued attributes we can end up with repeats */
for (i=0;i<msg->elements[idx].num_values;i++) {
@@ -640,7 +642,8 @@ static int ltdb_index_add1(struct ldb_context *ldb, char *dn,
{
struct ldb_message msg;
char *dn_key;
- int ret, i, added=0, added_dn=0;
+ int ret, added=0, added_dn=0;
+ unsigned int i;
dn_key = ldb_dn_key(ldb, el->name, &el->values[v_idx]);
if (!dn_key) {
@@ -704,7 +707,8 @@ static int ltdb_index_add1(struct ldb_context *ldb, char *dn,
int ltdb_index_add(struct ldb_context *ldb, const struct ldb_message *msg)
{
struct ltdb_private *ltdb = ldb->private_data;
- int ret, i, j;
+ int ret;
+ unsigned int i, j;
if (ltdb->cache.indexlist.num_elements == 0) {
/* no indexed fields */
@@ -737,7 +741,8 @@ static int ltdb_index_del1(struct ldb_context *ldb, const char *dn,
{
struct ldb_message msg;
char *dn_key;
- int ret, i, j;
+ int ret, i;
+ unsigned int j;
dn_key = ldb_dn_key(ldb, el->name, &el->values[v_idx]);
if (!dn_key) {
@@ -794,7 +799,8 @@ static int ltdb_index_del1(struct ldb_context *ldb, const char *dn,
int ltdb_index_del(struct ldb_context *ldb, const struct ldb_message *msg)
{
struct ltdb_private *ltdb = ldb->private_data;
- int ret, i, j;
+ int ret;
+ unsigned int i, j;
/* find the list of indexed fields */
if (ltdb->cache.indexlist.num_elements == 0) {
diff --git a/source4/lib/ldb/ldb_tdb/ldb_match.c b/source4/lib/ldb/ldb_tdb/ldb_match.c
index 05a2826d4d..963048915a 100644
--- a/source4/lib/ldb/ldb_tdb/ldb_match.c
+++ b/source4/lib/ldb/ldb_tdb/ldb_match.c
@@ -140,7 +140,7 @@ static int ldb_val_equal_objectclass(struct ldb_context *ldb,
const struct ldb_val *v1, const struct ldb_val *v2)
{
struct ltdb_private *ltdb = ldb->private_data;
- int i;
+ unsigned int i;
if (ldb_val_equal_case_insensitive(v1, v2) == 1) {
return 1;
@@ -149,7 +149,7 @@ static int ldb_val_equal_objectclass(struct ldb_context *ldb,
for (i=0;i<ltdb->cache.subclasses.num_elements;i++) {
struct ldb_message_element *el = &ltdb->cache.subclasses.elements[i];
if (ldb_attr_cmp(el->name, v2->data) == 0) {
- int j;
+ unsigned int j;
for (j=0;j<el->num_values;j++) {
if (ldb_val_equal_objectclass(ldb, v1, &el->values[j])) {
return 1;
@@ -259,7 +259,7 @@ static int match_leaf(struct ldb_context *ldb,
const char *base,
enum ldb_scope scope)
{
- int i, j;
+ unsigned int i, j;
if (!scope_match(msg->dn, base, scope)) {
return 0;
@@ -304,7 +304,8 @@ int ldb_message_match(struct ldb_context *ldb,
const char *base,
enum ldb_scope scope)
{
- int v, i;
+ unsigned int i;
+ int v;
switch (tree->operation) {
case LDB_OP_SIMPLE:
diff --git a/source4/lib/ldb/ldb_tdb/ldb_pack.c b/source4/lib/ldb/ldb_tdb/ldb_pack.c
index fcbc9bd56d..ad856d2faf 100644
--- a/source4/lib/ldb/ldb_tdb/ldb_pack.c
+++ b/source4/lib/ldb/ldb_tdb/ldb_pack.c
@@ -69,7 +69,7 @@ int ltdb_pack_data(struct ldb_context *ldb,
const struct ldb_message *message,
struct TDB_DATA *data)
{
- int i, j, real_elements=0;
+ unsigned int i, j, real_elements=0;
size_t size;
char *p;
size_t len;
@@ -141,7 +141,7 @@ int ltdb_pack_data(struct ldb_context *ldb,
void ltdb_unpack_data_free(struct ldb_context *ldb,
struct ldb_message *message)
{
- int i;
+ unsigned int i;
for (i=0;i<message->num_elements;i++) {
if (message->elements[i].values) ldb_free(ldb, message->elements[i].values);
@@ -166,7 +166,7 @@ int ltdb_unpack_data(struct ldb_context *ldb,
{
char *p;
unsigned int remaining;
- int i, j;
+ unsigned int i, j;
unsigned format;
size_t len;
diff --git a/source4/lib/ldb/ldb_tdb/ldb_search.c b/source4/lib/ldb/ldb_tdb/ldb_search.c
index 7e8cd8b289..18d51d1aa4 100644
--- a/source4/lib/ldb/ldb_tdb/ldb_search.c
+++ b/source4/lib/ldb/ldb_tdb/ldb_search.c
@@ -41,7 +41,7 @@
*/
static void msg_free_all_parts(struct ldb_context *ldb, struct ldb_message *msg)
{
- int i, j;
+ unsigned int i, j;
ldb_free(ldb, msg->dn);
for (i=0;i<msg->num_elements;i++) {
ldb_free(ldb, msg->elements[i].name);
@@ -89,7 +89,7 @@ struct ldb_val ldb_val_dup(struct ldb_context *ldb,
static int msg_add_element(struct ldb_context *ldb,
struct ldb_message *ret, const struct ldb_message_element *el)
{
- int i;
+ unsigned int i;
struct ldb_message_element *e2, *elnew;
e2 = ldb_realloc_p(ldb, ret->elements, struct ldb_message_element, ret->num_elements+1);
@@ -134,7 +134,7 @@ static int msg_add_element(struct ldb_context *ldb,
static int msg_add_all_elements(struct ldb_context *ldb, struct ldb_message *ret,
const struct ldb_message *msg)
{
- int i;
+ unsigned int i;
for (i=0;i<msg->num_elements;i++) {
int flags = ltdb_attribute_flags(ldb, msg->elements[i].name);
if ((msg->dn[0] != '@') && (flags & LTDB_FLAG_HIDDEN)) {
@@ -265,7 +265,7 @@ int ltdb_has_wildcard(struct ldb_context *ldb, const char *attr_name,
*/
void ltdb_search_dn1_free(struct ldb_context *ldb, struct ldb_message *msg)
{
- int i;
+ unsigned int i;
ldb_free(ldb, msg->private_data);
for (i=0;i<msg->num_elements;i++) {
ldb_free(ldb, msg->elements[i].values);
diff --git a/source4/lib/ldb/ldb_tdb/ldb_tdb.c b/source4/lib/ldb/ldb_tdb/ldb_tdb.c
index d60504ff17..b36c53e100 100644
--- a/source4/lib/ldb/ldb_tdb/ldb_tdb.c
+++ b/source4/lib/ldb/ldb_tdb/ldb_tdb.c
@@ -324,7 +324,7 @@ failed:
*/
static int find_element(const struct ldb_message *msg, const char *name)
{
- int i;
+ unsigned int i;
for (i=0;i<msg->num_elements;i++) {
if (ldb_attr_cmp(msg->elements[i].name, name) == 0) {
return i;
@@ -345,7 +345,7 @@ static int msg_add_element(struct ldb_context *ldb,
struct ldb_message *msg, struct ldb_message_element *el)
{
struct ldb_message_element *e2;
- int i;
+ unsigned int i;
e2 = ldb_realloc_p(ldb, msg->elements, struct ldb_message_element,
msg->num_elements+1);
@@ -385,7 +385,7 @@ static int msg_add_element(struct ldb_context *ldb,
static int msg_delete_attribute(struct ldb_context *ldb,
struct ldb_message *msg, const char *name)
{
- int i, count=0;
+ unsigned int i, count=0;
struct ldb_message_element *el2;
el2 = ldb_malloc_array_p(ldb, struct ldb_message_element, msg->num_elements);
@@ -419,15 +419,16 @@ static int msg_delete_element(struct ldb_context *ldb,
const char *name,
const struct ldb_val *val)
{
- int i;
+ unsigned int i;
+ int found;
struct ldb_message_element *el;
- i = find_element(msg, name);
- if (i == -1) {
+ found = find_element(msg, name);
+ if (found == -1) {
return -1;
}
- el = &msg->elements[i];
+ el = &msg->elements[found];
for (i=0;i<el->num_values;i++) {
if (ldb_val_equal(ldb, msg->elements[i].name, &el->values[i], val)) {
@@ -459,7 +460,8 @@ int ltdb_modify_internal(struct ldb_context *ldb, const struct ldb_message *msg)
struct ltdb_private *ltdb = ldb->private_data;
TDB_DATA tdb_key, tdb_data;
struct ldb_message msg2;
- int ret, i, j;
+ unsigned i, j;
+ int ret;
tdb_key = ltdb_key(ldb, msg->dn);
if (!tdb_key.dptr) {