Code cleanups after running with "sparse"
authorSteve McIntyre <steve.mcintyre@linaro.org>
Mon, 13 May 2013 17:22:54 +0000 (18:22 +0100)
committerSteve McIntyre <steve.mcintyre@linaro.org>
Mon, 13 May 2013 17:22:54 +0000 (18:22 +0100)
Lots of minor warnings picked up:

 * non-ANSI declarations
 * static variable declarations

and a real bug-fix in exit_if_enabled (use of "0" rather than "o");
could have caused a crash.

endian.c
extract-data.c
libjte/checksum.c
libjte/jte.c
libjte/libjte.c
libjte/sha256.c
libjte/sha512.c
libjte/test/demo.c
md5.c
mkimage.c
rsync.c

index a1f9895..ee372b1 100644 (file)
--- a/endian.c
+++ b/endian.c
@@ -11,9 +11,7 @@
 #include "endian.h"
 
 /* Write a 64-bit quantity out into memory in BIG ENDIAN order */
-void write_be64(in, out)
-    unsigned long long in;
-    unsigned char *out;
+void write_be64(unsigned long long in, unsigned char *out)
 {
     out[0] = (in >> 56) & 0xFF;
     out[1] = (in >> 48) & 0xFF;
@@ -26,8 +24,7 @@ void write_be64(in, out)
 }
 
 /* Read in a 64-bit BIG ENDIAN quantity */
-unsigned long long read_be64(in)
-    unsigned char *in;
+unsigned long long read_be64(unsigned char *in)
 {
     unsigned long long result = 0;
 
@@ -44,9 +41,7 @@ unsigned long long read_be64(in)
 }
 
 /* Write a 64-bit quantity out into memory in LITTLE ENDIAN order */
-void write_le64(in, out)
-    unsigned long long in;
-    unsigned char *out;
+void write_le64(unsigned long long in, unsigned char *out)
 {
     out[0] = in & 0xFF;
     out[1] = (in >> 8) & 0xFF;
@@ -59,8 +54,7 @@ void write_le64(in, out)
 }
 
 /* Read in a 64-bit LITTLE ENDIAN quantity */
-unsigned long long read_le64(in)
-    unsigned char *in;
+unsigned long long read_le64(unsigned char *in)
 {
     unsigned long long result = 0;
 
@@ -77,9 +71,7 @@ unsigned long long read_le64(in)
 }
 
 /* Write a 48-bit quantity out into memory in LITTLE ENDIAN order */
-void write_le48(in, out)
-    unsigned long long in;
-    unsigned char *out;
+void write_le48(unsigned long long in, unsigned char *out)
 {
     out[0] = in & 0xFF;
     out[1] = (in >> 8) & 0xFF;
@@ -90,8 +82,7 @@ void write_le48(in, out)
 }
 
 /* Read in a 48-bit LITTLE ENDIAN quantity */
-unsigned long long read_le48(in)
-    unsigned char *in;
+unsigned long long read_le48(unsigned char *in)
 {
     unsigned long long result = 0;
 
@@ -106,9 +97,7 @@ unsigned long long read_le48(in)
 }
 
 /* Write a 32-bit quantity out into memory in BIG ENDIAN order */
-void write_be32(in, out)
-    unsigned long in;
-    unsigned char *out;
+void write_be32(unsigned long in, unsigned char *out)
 {
     out[0] = (in >> 24) & 0xFF;
     out[1] = (in >> 16) & 0xFF;
@@ -117,8 +106,7 @@ void write_be32(in, out)
 }
 
 /* Read in a 32-bit BIG ENDIAN quantity */
-unsigned long read_be32(in)
-    unsigned char *in;
+unsigned long read_be32(unsigned char * in)
 {
     unsigned long result = 0;
 
@@ -131,9 +119,7 @@ unsigned long read_be32(in)
 }
 
 /* Write a 32-bit quantity out into memory in LITTLE ENDIAN order */
-void write_le32(in, out)
-    unsigned long in;
-    unsigned char *out;
+void write_le32(unsigned long in, unsigned char *out)
 {
     out[0] = in & 0xFF;
     out[1] = (in >> 8) & 0xFF;
@@ -142,8 +128,7 @@ void write_le32(in, out)
 }
 
 /* Read in a 32-bit LITTLE ENDIAN quantity */
-unsigned long read_le32(in)
-    unsigned char *in;
+unsigned long read_le32(unsigned char *in)
 {
     unsigned long result = 0;
 
@@ -156,17 +141,14 @@ unsigned long read_le32(in)
 }
 
 /* Write a 16-bit quantity out into memory in BIG ENDIAN order */
-void write_be16(in, out)
-    unsigned short in;
-    unsigned char *out;
+void write_be16(unsigned short in, unsigned char *out)
 {
     out[0] = (in >> 8) & 0xFF;
     out[1] = in & 0xFF;
 }
     
 /* Read in a 16-bit BIG ENDIAN quantity */
-unsigned short read_be16(in)
-    unsigned char *in;
+unsigned short read_be16(unsigned char *in)
 {
     unsigned short result = 0;
     
@@ -176,17 +158,14 @@ unsigned short read_be16(in)
 }
 
 /* Write a 16-bit quantity out into memory in LITTLE ENDIAN order */
-void write_le16(in, out)
-    unsigned short in;
-    unsigned char *out;
+void write_le16(unsigned short in, unsigned char *out)
 {
     out[0] = in & 0xFF;
     out[1] = in & 0xFF >> 8;
 }
     
 /* Read in a 16-bit LITTLE ENDIAN quantity */
-unsigned short read_le16(in)
-    unsigned char *in;
+unsigned short read_le16(unsigned char *in)
 {
     unsigned short result = 0;
     
index 2984bc7..f6d15c5 100644 (file)
@@ -15,8 +15,8 @@
 #include "md5.h"
 #include "jigdo.h"
 
-FILE *logfile;
-zip_state_t zip_state;
+static FILE *logfile;
+static zip_state_t zip_state;
 
 int main(int argc, char **argv)
 {
index e1580d5..fedbe18 100644 (file)
@@ -228,7 +228,7 @@ static void *checksum_thread(void *arg)
         pthread_mutex_unlock(&c->done_mutex);
     }
 
-    pthread_exit(0);
+    pthread_exit(NULL);
 }
 #endif
 
@@ -410,7 +410,7 @@ void checksum_final(checksum_context_t *context)
     {
         if (c->algo[i].enabled)
         {
-            void *ret = 0;
+            void *ret = NULL;
             struct algo_context *a = &c->algo[i];
 
             pthread_mutex_lock(&a->start_mutex);
index 5349c61..786c3c8 100644 (file)
@@ -88,7 +88,7 @@ static void exit_if_enabled(struct libjte_env *o, int value)
 {
     if (!(o->error_behavior & 2))
         return;
-    libjte_clear_msg_list(0, 1);
+    libjte_clear_msg_list(o, 1);
     exit(value);
 }
 
@@ -156,7 +156,7 @@ int libjte_destroy_path_match_list(struct libjte_env *o, int flag)
 }
 
 /* Build the list of exclusion regexps */
-extern int jte_add_exclude(struct libjte_env *o, char *pattern)
+int jte_add_exclude(struct libjte_env *o, char *pattern)
 {
     int ret;
     
@@ -187,7 +187,7 @@ static int check_exclude_by_name(struct libjte_env *o,
 }
 
 /* Build the list of required inclusion regexps */
-extern int jte_add_include(struct libjte_env *o, char *pattern)
+int jte_add_include(struct libjte_env *o, char *pattern)
 {
     int ret;
     
@@ -251,7 +251,7 @@ static int check_md5_file_match(struct libjte_env *o, char *filename)
       file, bail out with an error.
 
 */
-extern int list_file_in_jigdo(struct libjte_env *o,
+int list_file_in_jigdo(struct libjte_env *o,
             char *filename, off_t size, char **realname, unsigned char md5[16])
 {
     char *matched_rule;
@@ -332,7 +332,7 @@ extern int list_file_in_jigdo(struct libjte_env *o,
 
 /* Add a mapping of pathnames (e.g. Debian=/mirror/debian). We should
    be passed TO=FROM here */
-extern int jte_add_mapping(struct libjte_env *o, char *arg)
+int jte_add_mapping(struct libjte_env *o, char *arg)
 {
     struct path_mapping *new = NULL;
     struct path_mapping *entry = NULL;
index 0a1410d..aed9af0 100644 (file)
@@ -121,7 +121,7 @@ int libjte_new(struct libjte_env **jte_handle, int flag)
     o->include_in_jigdo = 0;
     memset(o->message_buffer, 0, sizeof(o->message_buffer));
     o->error_behavior = 1; /* Print to stderr, do not exit but return -1 */
-    o->msg_list = 0;
+    o->msg_list = NULL;
     o->uncomp_buf = NULL;
     o->uncomp_size = 0;
     o->uncomp_buf_used = 0;
index 9bca5ec..c92ddc5 100644 (file)
@@ -179,8 +179,7 @@ sha256_process_block (const void *buffer, size_t len, struct sha256_ctx *ctx)
 /* Initialize structure containing state of computation.
    (FIPS 180-2:5.3.2)  */
 void
-sha256_init_ctx (ctx)
-     struct sha256_ctx *ctx;
+sha256_init_ctx (struct sha256_ctx *ctx)
 {
   ctx->H[0] = 0x6a09e667;
   ctx->H[1] = 0xbb67ae85;
@@ -202,9 +201,7 @@ sha256_init_ctx (ctx)
    IMPORTANT: On some systems it is required that RESBUF is correctly
    aligned for a 32 bits value.  */
 void *
-sha256_finish_ctx (ctx, resbuf)
-     struct sha256_ctx *ctx;
-     void *resbuf;
+sha256_finish_ctx (struct sha256_ctx *ctx, void *resbuf)
 {
   /* Take yet unprocessed bytes into account.  */
   uint32_t bytes = ctx->buflen;
@@ -236,10 +233,7 @@ sha256_finish_ctx (ctx, resbuf)
 
 
 void
-sha256_process_bytes (buffer, len, ctx)
-     const void *buffer;
-     size_t len;
-     struct sha256_ctx *ctx;
+sha256_process_bytes (const void *buffer, size_t len, struct sha256_ctx *ctx)
 {
   /* When we already have some bits in our internal buffer concatenate
      both inputs first.  */
index 19e2bc4..89c9791 100644 (file)
@@ -270,8 +270,7 @@ sha512_process_block (const void *buffer, size_t len, struct sha512_ctx *ctx)
 /* Initialize structure containing state of computation.
    (FIPS 180-2:5.3.3)  */
 void
-sha512_init_ctx (ctx)
-     struct sha512_ctx *ctx;
+sha512_init_ctx (struct sha512_ctx *ctx)
 {
 
 #ifdef LIBJTE_INITIALIZE_32
@@ -317,9 +316,7 @@ sha512_init_ctx (ctx)
    IMPORTANT: On some systems it is required that RESBUF is correctly
    aligned for a 32 bits value.  */
 void *
-sha512_finish_ctx (ctx, resbuf)
-     struct sha512_ctx *ctx;
-     void *resbuf;
+sha512_finish_ctx (struct sha512_ctx *ctx, void *resbuf)
 {
   /* Take yet unprocessed bytes into account.  */
   uint64_t bytes = ctx->buflen;
@@ -351,10 +348,7 @@ sha512_finish_ctx (ctx, resbuf)
 
 
 void
-sha512_process_bytes (buffer, len, ctx)
-     const void *buffer;
-     size_t len;
-     struct sha512_ctx *ctx;
+sha512_process_bytes (const void *buffer, size_t len, struct sha512_ctx *ctx)
 {
   /* When we already have some bits in our internal buffer concatenate
      both inputs first.  */
index 980bbca..14177bf 100644 (file)
 #define LIBJTE_DEMO_MAX_FILES 1000
 #define LIBJTE_DEMO_BUFSIZE   4096
 
-struct libjte_env *handle = NULL;
-char *outfile = NULL;
-FILE *fp_out = NULL;
-int filec = 0;
-char *filev[LIBJTE_DEMO_MAX_FILES + 1];
+static struct libjte_env *handle = NULL;
+static char *outfile = NULL;
+static int filec = 0;
+static char *filev[LIBJTE_DEMO_MAX_FILES + 1];
 
 
 static int libjte_demo_setup(int argc, char **argv, int flag)
@@ -175,7 +174,7 @@ usage:;
 }
 
 
-int libjte_demo_write(char *buf, int size, FILE *fp_out)
+static int libjte_demo_write(char *buf, int size, FILE *fp_out)
 {
     int ret;
 
@@ -194,7 +193,7 @@ int libjte_demo_write(char *buf, int size, FILE *fp_out)
 }
 
 
-int libjte_demo_run(int flag)
+static int libjte_demo_run(int flag)
 {
     int ret, i, major, minor, micro, buf_fill;
     FILE *fp_out, *fp_in;
diff --git a/md5.c b/md5.c
index 7f6bd3e..40595ea 100644 (file)
--- a/md5.c
+++ b/md5.c
    surprised if they were a performance bottleneck for MD5.  */
 
 static mk_uint32
-getu32 (addr)
-     const unsigned char *addr;
+getu32 (const unsigned char *addr)
 {
        return (((((unsigned long)addr[3] << 8) | addr[2]) << 8)
                | addr[1]) << 8 | addr[0];
 }
 
 static void
-putu32 (data, addr)
-     mk_uint32 data;
-     unsigned char *addr;
+putu32 (mk_uint32 data, unsigned char *addr)
 {
        addr[0] = (unsigned char)data;
        addr[1] = (unsigned char)(data >> 8);
@@ -70,8 +67,7 @@ putu32 (data, addr)
  * initialization constants.
  */
 void
-mk_MD5Init (ctx)
-     struct mk_MD5Context *ctx;
+mk_MD5Init (struct mk_MD5Context *ctx)
 {
        ctx->buf[0] = 0x67452301;
        ctx->buf[1] = 0xefcdab89;
@@ -87,10 +83,7 @@ mk_MD5Init (ctx)
  * of bytes.
  */
 void
-mk_MD5Update (ctx, buf, len)
-     struct mk_MD5Context *ctx;
-     unsigned char const *buf;
-     unsigned len;
+mk_MD5Update(struct mk_MD5Context *ctx, unsigned char const *buf, unsigned len)
 {
        mk_uint32 t;
 
@@ -138,9 +131,7 @@ mk_MD5Update (ctx, buf, len)
  * 1 0* (64-bit count of bits processed, MSB-first)
  */
 void
-mk_MD5Final (digest, ctx)
-     unsigned char digest[16];
-     struct mk_MD5Context *ctx;
+mk_MD5Final (unsigned char digest[16], struct mk_MD5Context *ctx)
 {
        unsigned count;
        unsigned char *p;
@@ -201,9 +192,7 @@ mk_MD5Final (digest, ctx)
  * the data and converts bytes into longwords for this routine.
  */
 void
-mk_MD5Transform (buf, inraw)
-     mk_uint32 buf[4];
-     const unsigned char inraw[64];
+mk_MD5Transform (mk_uint32 buf[4], const unsigned char inraw[64])
 {
        register mk_uint32 a, b, c, d;
        mk_uint32 in[16];
index 50cb341..64e1a82 100644 (file)
--- a/mkimage.c
+++ b/mkimage.c
 #define MISSING -1
 #define UNKNOWN -2
 
-FILE *logfile = NULL;
-FILE *outfile = NULL;
-FILE *missing_file = NULL;
-long long start_offset = 0;
-long long end_offset = 0;
-int quick = 0;
-int verbose = 0;
-int check_jigdo_header = 1;
-UINT64 out_size = 0;
-char *missing_filename = NULL;
+static FILE *logfile = NULL;
+static FILE *outfile = NULL;
+static FILE *missing_file = NULL;
+static long long start_offset = 0;
+static long long end_offset = 0;
+static int quick = 0;
+static int verbose = 0;
+static int check_jigdo_header = 1;
+static UINT64 out_size = 0;
+static char *missing_filename = NULL;
 
 typedef struct match_list_
 {
@@ -46,8 +46,8 @@ typedef struct match_list_
     char *mirror_path;
 } match_list_t;
 
-match_list_t *match_list_head = NULL;
-match_list_t *match_list_tail = NULL;
+static match_list_t *match_list_head = NULL;
+static match_list_t *match_list_tail = NULL;
 
 typedef struct md5_list_
 {
@@ -57,10 +57,10 @@ typedef struct md5_list_
     char *full_path;
 } md5_list_t;
 
-md5_list_t *md5_list_head = NULL;
-md5_list_t *md5_list_tail = NULL;
+static md5_list_t *md5_list_head = NULL;
+static md5_list_t *md5_list_tail = NULL;
 
-zip_state_t zip_state;
+static zip_state_t zip_state;
 
 /* Grab the file component from a full path */
 static char *file_base_name(char *path)
diff --git a/rsync.c b/rsync.c
index 9b21232..be5f98a 100644 (file)
--- a/rsync.c
+++ b/rsync.c
@@ -85,7 +85,7 @@ static UInt32_t charTable[256] = {
   0xaf551d9a, 0x66222560, 0x059deeca, 0x535f99e2
 };
 
-unsigned long long rsync64(unsigned char *mem, size_t size)
+static unsigned long long rsync64(unsigned char *mem, size_t size)
 {
     UInt32_t a = 0;
     UInt32_t b = 0;