]> git.ipfire.org Git - thirdparty/gnutls.git/commitdiff
Fix warnings in opencdk.
authorSimon Josefsson <simon@josefsson.org>
Wed, 3 Dec 2008 11:54:02 +0000 (12:54 +0100)
committerSimon Josefsson <simon@josefsson.org>
Wed, 3 Dec 2008 11:54:02 +0000 (12:54 +0100)
lib/opencdk/Makefile.am
lib/opencdk/armor.c
lib/opencdk/keydb.c
lib/opencdk/literal.c
lib/opencdk/misc.c
lib/opencdk/new-packet.c
lib/opencdk/read-packet.c
lib/opencdk/sig-check.c
lib/opencdk/stream.c

index dbf536a6cc78afa8085286ac8631169d6882abac..3941d3baa0ffb84478a9cb4d68ca794e2ad16dab 100644 (file)
@@ -20,6 +20,7 @@
 # Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
 # MA 02110-1301, USA
 
+AM_CFLAGS = $(WARN_CFLAGS)
 AM_CPPFLAGS = \
        -I$(srcdir)/../gl \
        -I$(builddir)/../gl \
index b4e5a27eec65f43433df770baf796e0b4445f4aa..a188ef2124199209b1719a4fb587e13cdd5d7e02 100644 (file)
@@ -595,7 +595,8 @@ armor_decode (void *data, FILE * in, FILE * out)
   afx->crc_okay = (afx->crc == crc2) ? 1 : 0;
   if (!afx->crc_okay && !rc)
     {
-      _cdk_log_debug ("file crc=%08lX afx_crc=%08lX\n", crc2, afx->crc);
+      _cdk_log_debug ("file crc=%08lX afx_crc=%08lX\n",
+                     (unsigned long) crc2, (unsigned long) afx->crc);
       rc = CDK_Armor_CRC_Error;
     }
 
index 651a205bb08a9e477bec074d2303ba0b6849ac1c..c8218c48d7462437eda8d9afbc8ca7bede573eac 100644 (file)
@@ -48,13 +48,14 @@ static cdk_kbnode_t find_selfsig_node (cdk_kbnode_t key, cdk_pkt_pubkey_t pk);
 static char *
 keydb_idx_mkname (const char *file)
 {
-  char *fname, *fmt;
+  const char *add = ".idx";
+  char *fname;
 
-  fmt = "%s.idx";
-  fname = cdk_calloc (1, strlen (file) + strlen (fmt) + 1);
+  fname = cdk_calloc (1, strlen (file) + strlen (add) + 1);
   if (!fname)
     return NULL;
-  sprintf (fname, fmt, file);
+  strcpy (fname, file);
+  strcat (fname, add);
   return fname;
 }
 
@@ -661,7 +662,8 @@ keydb_cache_add (cdk_keydb_search_t dbs, off_t offset)
   k->next = dbs->cache;
   dbs->cache = k;
   dbs->ncache++;
-  _cdk_log_debug ("cache: add entry off=%d type=%d\n", offset, dbs->type);
+  _cdk_log_debug ("cache: add entry off=%ld type=%d\n", (signed long) offset,
+                 dbs->type);
   return 0;
 }
 
index fc4ee4fdc3407d57ff9e08bab345e8ffc28a1a84..4e680c6e258219471c26e2003643aae14f80bf62 100644 (file)
@@ -200,7 +200,7 @@ literal_encode (void *data, FILE * in, FILE * out)
   filelen = strlen (pfx->filename);
   cdk_pkt_new (&pkt);
   pt = pkt->pkt.literal = cdk_calloc (1, sizeof *pt + filelen);
-  pt->name = pt + sizeof(*pt);
+  pt->name = (char*) (pt + sizeof(*pt));
   if (!pt)
     {
       cdk_pkt_release (pkt);
index 13e7fd5275581fdc669d4ca973edc5131a00fceb..a8acdf8dfe7ceffbbe89efec3e13807ff7874c9d 100644 (file)
@@ -97,7 +97,7 @@ cdk_strlist_add (cdk_strlist_t * list, const char *string)
   sl = cdk_calloc (1, sizeof *sl + strlen (string) + 2);
   if (!sl)
     return NULL;
-  sl->d = sl + sizeof(*sl);
+  sl->d = (char*) (sl + sizeof(*sl));
   strcpy (sl->d, string);
   sl->next = *list;
   *list = sl;
index 501d65293f0018a706512f044ecfe4484c6d12a1..85322c5e1f7a5a36ec3f15adedd5a32c85dcf0e1 100644 (file)
@@ -388,7 +388,7 @@ _cdk_copy_userid (cdk_pkt_userid_t * dst, cdk_pkt_userid_t src)
   u = cdk_calloc (1, sizeof *u + strlen (src->name) + 2);
   if (!u)
     return CDK_Out_Of_Core;
-  u->name = (void*)u + sizeof(*u);
+  u->name = (char*) (u + sizeof(*u));
 
   memcpy (u, src, sizeof *u);
   memcpy (u->name, src->name, strlen (src->name));
@@ -627,7 +627,7 @@ cdk_subpkt_new (size_t size)
   s = cdk_calloc (1, sizeof *s + size + 2);
   if (!s)
     return NULL;
-  s->d = (void*)s + sizeof(*s);
+  s->d = (char*) (s + sizeof(*s));
 
   return s;
 }
index 2869c2ce053080629025f5b126d1764975d9480e..4808eb166ce33a2ef52829a254845e9b858add56 100644 (file)
@@ -553,7 +553,7 @@ read_user_id (cdk_stream_t inp, size_t pktlen, cdk_pkt_userid_t user_id)
     return CDK_Inv_Packet;
 
   if (DEBUG_PKT)
-    _cdk_log_debug ("read_user_id: %lu octets\n", pktlen);
+    _cdk_log_debug ("read_user_id: %lu octets\n", (unsigned long) pktlen);
 
   user_id->len = pktlen;
   rc = stream_read (inp, user_id->name, pktlen, &nread);
@@ -836,7 +836,7 @@ read_literal (cdk_stream_t inp, size_t pktlen,
       *ret_pt = pt = cdk_realloc (pt, sizeof *pt + pt->namelen + 2);
       if (!pt)
        return CDK_Out_Of_Core;
-      pt->name = (void*)pt + sizeof(*pt);
+      pt->name = (char*) (pt + sizeof(*pt));
       rc = stream_read (inp, pt->name, pt->namelen, &nread);
       if (rc)
        return rc;
@@ -958,7 +958,8 @@ cdk_pkt_read (cdk_stream_t inp, cdk_packet_t pkt)
   if (!(ctb & 0x80))
     {
       _cdk_log_info ("cdk_pkt_read: no openpgp data found. "
-                    "(ctb=%02X; fpos=%02X)\n", ctb, cdk_stream_tell (inp));
+                    "(ctb=%02X; fpos=%02lX)\n", ctb,
+                    (unsigned long) cdk_stream_tell (inp));
       return CDK_Inv_Packet;
     }
 
@@ -998,7 +999,8 @@ cdk_pkt_read (cdk_stream_t inp, cdk_packet_t pkt)
                                     + pkt->pktlen + 16 + 1);
       if (!pkt->pkt.user_id)
        return CDK_Out_Of_Core;
-      pkt->pkt.user_id->name = (void*)pkt->pkt.user_id + sizeof(*pkt->pkt.user_id);
+      pkt->pkt.user_id->name = (char*) (pkt->pkt.user_id
+                                       + sizeof(*pkt->pkt.user_id));
 
       rc = read_attribute (inp, pktlen, pkt->pkt.user_id);
       pkt->pkttype = CDK_PKT_ATTRIBUTE;
@@ -1009,7 +1011,8 @@ cdk_pkt_read (cdk_stream_t inp, cdk_packet_t pkt)
                                     + pkt->pktlen + 1);
       if (!pkt->pkt.user_id)
        return CDK_Out_Of_Core;
-      pkt->pkt.user_id->name = (void*)pkt->pkt.user_id + sizeof(*pkt->pkt.user_id);
+      pkt->pkt.user_id->name = (char*) (pkt->pkt.user_id
+                                       + sizeof(*pkt->pkt.user_id));
       rc = read_user_id (inp, pktlen, pkt->pkt.user_id);
       break;
 
index 38007299faa3e96c5a7fea2497ca039e8b891114..48d407f6ec9647d47b8ff8a69fc395cec40223fc 100644 (file)
@@ -542,8 +542,9 @@ cdk_pk_check_sigs (cdk_kbnode_t key, cdk_keydb_hd_t keydb, int *r_status)
        }
 
       _cdk_log_debug ("signature %s: signer %08lX keyid %08lX\n",
-                     rc == CDK_Bad_Sig ? "BAD" : "good", sig->keyid[1],
-                     keyid);
+                     rc == CDK_Bad_Sig ? "BAD" : "good",
+                     (unsigned long) sig->keyid[1],
+                     (unsigned long) keyid);
 
       if (IS_UID_SIG (sig) && uid_name != NULL)
        {
index d1061925b84cee41ae37cb8761e8d2a17a533f26..dcf66c5c329a06ee091cfc04e785c08a17d778d3 100644 (file)
@@ -1477,7 +1477,8 @@ cdk_stream_mmap_part (cdk_stream_t s, off_t off, size_t len,
     len = cdk_stream_get_length (s);
   if (!len)
     {
-      _cdk_log_debug ("cdk_stream_mmap_part: invalid file size %lu\n", len);
+      _cdk_log_debug ("cdk_stream_mmap_part: invalid file size %lu\n",
+                     (unsigned long) len);
       gnutls_assert ();
       return s->error;
     }