The branch, master has been updated
       via  e551f43b4b690d92cbf00a510e6c79a497d91464 (commit)
      from  74ea79cb8fa42b1bc13ac2bd9450beb13ad1cd5e (commit)

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit e551f43b4b690d92cbf00a510e6c79a497d91464
Author: Volker Lendecke <v...@samba.org>
Date:   Thu Jan 1 18:16:42 2009 +0100

    Remove two pointless globals

-----------------------------------------------------------------------

Summary of changes:
 source3/lib/ldb/tools/ldbadd.c    |   13 ++++++-------
 source3/lib/ldb/tools/ldbmodify.c |   12 ++++++------
 2 files changed, 12 insertions(+), 13 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/lib/ldb/tools/ldbadd.c b/source3/lib/ldb/tools/ldbadd.c
index 4dde2a1..155395e 100644
--- a/source3/lib/ldb/tools/ldbadd.c
+++ b/source3/lib/ldb/tools/ldbadd.c
@@ -35,8 +35,6 @@
 #include "ldb/include/includes.h"
 #include "ldb/tools/cmdline.h"
 
-static int failures;
-
 static void usage(void)
 {
        printf("Usage: ldbadd <options> <ldif...>\n");
@@ -53,7 +51,8 @@ static void usage(void)
 /*
   add records from an opened file
 */
-static int process_file(struct ldb_context *ldb, FILE *f, int *count)
+static int process_file(struct ldb_context *ldb, FILE *f, int *count,
+                       int *failures)
 {
        struct ldb_ldif *ldif;
        int ret = LDB_SUCCESS;
@@ -71,7 +70,7 @@ static int process_file(struct ldb_context *ldb, FILE *f, int 
*count)
                if (ret != LDB_SUCCESS) {
                        fprintf(stderr, "ERR: \"%s\" on DN %s\n", 
                                ldb_errstring(ldb), ldb_dn_linearize(ldb, 
ldif->msg->dn));
-                       failures++;
+                       (*failures)++;
                } else {
                        (*count)++;
                }
@@ -86,7 +85,7 @@ static int process_file(struct ldb_context *ldb, FILE *f, int 
*count)
 int main(int argc, const char **argv)
 {
        struct ldb_context *ldb;
-       int i, ret=0, count=0;
+       int i, ret=0, count=0, failures=0;
        struct ldb_cmdline *options;
 
        ldb_global_init();
@@ -96,7 +95,7 @@ int main(int argc, const char **argv)
        options = ldb_cmdline_process(ldb, argc, argv, usage);
 
        if (options->argc == 0) {
-               ret = process_file(ldb, stdin, &count);
+               ret = process_file(ldb, stdin, &count, &failures);
        } else {
                for (i=0;i<options->argc;i++) {
                        const char *fname = options->argv[i];
@@ -106,7 +105,7 @@ int main(int argc, const char **argv)
                                perror(fname);
                                exit(1);
                        }
-                       ret = process_file(ldb, f, &count);
+                       ret = process_file(ldb, f, &count, &failures);
                        fclose(f);
                }
        }
diff --git a/source3/lib/ldb/tools/ldbmodify.c 
b/source3/lib/ldb/tools/ldbmodify.c
index 368b4cf..f12387a 100644
--- a/source3/lib/ldb/tools/ldbmodify.c
+++ b/source3/lib/ldb/tools/ldbmodify.c
@@ -35,8 +35,6 @@
 #include "ldb/include/includes.h"
 #include "ldb/tools/cmdline.h"
 
-static int failures;
-
 static void usage(void)
 {
        printf("Usage: ldbmodify <options> <ldif...>\n");
@@ -52,7 +50,8 @@ static void usage(void)
 /*
   process modifies for one file
 */
-static int process_file(struct ldb_context *ldb, FILE *f, int *count)
+static int process_file(struct ldb_context *ldb, FILE *f, int *count,
+                       int *failures)
 {
        struct ldb_ldif *ldif;
        int ret = LDB_SUCCESS;
@@ -73,7 +72,7 @@ static int process_file(struct ldb_context *ldb, FILE *f, int 
*count)
                if (ret != LDB_SUCCESS) {
                        fprintf(stderr, "ERR: \"%s\" on DN %s\n", 
                                ldb_errstring(ldb), ldb_dn_linearize(ldb, 
ldif->msg->dn));
-                       failures++;
+                       (*failures)++;
                } else {
                        (*count)++;
                }
@@ -87,6 +86,7 @@ int main(int argc, const char **argv)
 {
        struct ldb_context *ldb;
        int count=0;
+       int failures=0;
        int i, ret=LDB_SUCCESS;
        struct ldb_cmdline *options;
 
@@ -97,7 +97,7 @@ int main(int argc, const char **argv)
        options = ldb_cmdline_process(ldb, argc, argv, usage);
 
        if (options->argc == 0) {
-               ret = process_file(ldb, stdin, &count);
+               ret = process_file(ldb, stdin, &count, &failures);
        } else {
                for (i=0;i<options->argc;i++) {
                        const char *fname = options->argv[i];
@@ -107,7 +107,7 @@ int main(int argc, const char **argv)
                                perror(fname);
                                exit(1);
                        }
-                       ret = process_file(ldb, f, &count);
+                       ret = process_file(ldb, f, &count, &failures);
                }
        }
 


-- 
Samba Shared Repository

Reply via email to