Fix size_t vs int issues.
authorMarcus Brinkmann <marcus@neumann.local>
Wed, 1 Jun 2011 19:43:30 +0000 (21:43 +0200)
committerMarcus Brinkmann <marcus@neumann.local>
Wed, 1 Jun 2011 19:43:30 +0000 (21:43 +0200)
12 files changed:
agent/ChangeLog
agent/cvt-openpgp.c
common/ChangeLog
common/util.h
g10/ChangeLog
g10/build-packet.c
g10/import.c
g10/parse-packet.c
g10/seskey.c
g13/mountinfo.c
sm/ChangeLog
sm/certreqgen.c

index 78ddf8d..156fbdb 100644 (file)
@@ -1,3 +1,8 @@
+2011-06-01  Marcus Brinkmann  <mb@g10code.com>
+
+       * cvt-openpgp.c (convert_to_openpgp): Change type of N to unsigned
+       int.
+
 2011-04-26  Werner Koch  <wk@g10code.com>
 
        * cvt-openpgp.c (convert_to_openpgp): Use rfc4880 encoded S2K count.
index 0f31728..ec0fd0a 100644 (file)
@@ -1062,7 +1062,7 @@ convert_to_openpgp (ctrl_t ctrl, gcry_sexp_t s_key, const char *passphrase,
       void *format_args_buf_ptr[1];
       int   format_args_buf_int[1];
       void *format_args[10+2];
-      size_t n;
+      unsigned int n;
       gcry_sexp_t tmpkey, tmpsexp = NULL;
 
       snprintf (countbuf, sizeof countbuf, "%lu", s2k_count);
index d77fad5..4f8b541 100644 (file)
@@ -1,3 +1,7 @@
+2011-06-01  Marcus Brinkmann  <mb@g10code.com>
+
+       * util.h: Undef snprintf before redefining it.
+
 2011-05-20  Werner Koch  <wk@g10code.com>
 
        * util.h: Remove some error code substitutes.
index 45bccec..31c2caa 100644 (file)
@@ -62,6 +62,7 @@
 
 /* Due to a bug in mingw32's snprintf related to the 'l' modifier and
    for increased portability we use our snprintf on all systems. */
+#undef snprintf
 #define snprintf estream_snprintf
 
 
index b8d3232..91d49b4 100644 (file)
@@ -1,3 +1,12 @@
+2011-06-01  Marcus Brinkmann  <mb@g10code.com>
+
+       * parse-packet.c (parse_pubkeyenc): Change type of N to size_t.
+       (parse_key): Likewise.
+       * seskey.c (encode_session_key): Convert nframe to int for
+       debugging.
+       * build-packet.c (gpg_mpi_write): Change type of N to unsigned int.
+       * import.c (transfer_secret_keys): Likewise.
+
 2011-04-29  Werner Koch  <wk@g10code.com>
 
        * keydb.c (keydb_get_keyblock, keydb_add_resource): Use gpg_error.
index 1cdf961..f2ef691 100644 (file)
@@ -164,7 +164,7 @@ gpg_mpi_write (iobuf_t out, gcry_mpi_t a)
 
   if (gcry_mpi_get_flag (a, GCRYMPI_FLAG_OPAQUE))
     {
-      size_t nbits;
+      unsigned int nbits;
       const void *p;
 
       p = gcry_mpi_get_opaque (a, &nbits);
index 05dfd1d..ffbcf16 100644 (file)
@@ -1153,7 +1153,7 @@ transfer_secret_keys (ctrl_t ctrl, struct stats_s *stats, kbnode_t sec_keyblock)
   int nskey;
   membuf_t mbuf;
   int i, j;
-  size_t n;
+  unsigned int n;
   void *format_args_buf_ptr[PUBKEY_MAX_NSKEY];
   int   format_args_buf_int[PUBKEY_MAX_NSKEY];
   void *format_args[2*PUBKEY_MAX_NSKEY];
index 1171443..50786eb 100644 (file)
@@ -962,7 +962,6 @@ static int
 parse_pubkeyenc (IOBUF inp, int pkttype, unsigned long pktlen,
                 PACKET * packet)
 {
-  unsigned int n;
   int rc = 0;
   int i, ndata;
   PKT_pubkey_enc *k;
@@ -1009,12 +1008,13 @@ parse_pubkeyenc (IOBUF inp, int pkttype, unsigned long pktlen,
         {
           if (k->pubkey_algo == PUBKEY_ALGO_ECDH && i == 1)
             {
-              rc = read_size_body (inp, pktlen, &n, k->data+i);
+              size_t n;
+             rc = read_size_body (inp, pktlen, &n, k->data+i);
               pktlen -= n;
             }
           else
             {
-              n = pktlen;
+             int n = pktlen;
               k->data[i] = mpi_read (inp, &n, 0);
               pktlen -= n;
               if (!k->data[i])
@@ -1890,7 +1890,6 @@ parse_key (IOBUF inp, int pkttype, unsigned long pktlen,
 {
   gpg_error_t err = 0;
   int i, version, algorithm;
-  unsigned n;
   unsigned long timestamp, expiredate, max_expiredate;
   int npkey, nskey;
   int is_v4 = 0;
@@ -2003,12 +2002,13 @@ parse_key (IOBUF inp, int pkttype, unsigned long pktlen,
           if ((algorithm == PUBKEY_ALGO_ECDSA
                || algorithm == PUBKEY_ALGO_ECDH) && (i==0 || i == 2))
             {
-              err = read_size_body (inp, pktlen, &n, pk->pkey+i);
+              size_t n;
+             err = read_size_body (inp, pktlen, &n, pk->pkey+i);
               pktlen -= n;
             }
           else
             {
-              n = pktlen;
+              unsigned int n = pktlen;
               pk->pkey[i] = mpi_read (inp, &n, 0);
               pktlen -= n;
               if (!pk->pkey[i])
@@ -2255,7 +2255,7 @@ parse_key (IOBUF inp, int pkttype, unsigned long pktlen,
                }
              else
                {
-                 n = pktlen;
+                 unsigned int n = pktlen;
                  pk->pkey[i] = mpi_read (inp, &n, 0);
                  pktlen -= n;
                  if (list_mode)
index b210ae0..93a809d 100644 (file)
@@ -120,7 +120,7 @@ encode_session_key (int openpgp_pk_algo, DEK *dek, unsigned int nbits)
       if (DBG_CIPHER)
         log_debug ("encode_session_key: "
                    "[%d] %02x  %02x %02x ...  %02x %02x %02x\n",
-                   nframe, frame[0], frame[1], frame[2],
+                   (int) nframe, frame[0], frame[1], frame[2],
                    frame[nframe-3], frame[nframe-2], frame[nframe-1]);
 
       if (gcry_mpi_scan (&a, GCRYMPI_FMT_USG, frame, nframe, &nframe))
index 90b205e..be83de7 100644 (file)
@@ -193,6 +193,6 @@ mountinfo_dump_all (void)
   for (idx=0, m = mounttable; idx < mounttable_size; idx++, m++)
     if (m->in_use)
       log_info ("mtab[%d] %s on %s type %d rid %u%s\n",
-                idx, m->container, m->mountpoint, m->conttype, m->rid,
+                (int)idx, m->container, m->mountpoint, m->conttype, m->rid,
                 m->flags.remove?" [remove]":"");
 }
index 7127fb5..a450ebd 100644 (file)
@@ -1,3 +1,7 @@
+2011-06-01  Marcus Brinkmann  <mb@g10code.com>
+
+       * certreqgen.c (proc_parameters): Initialize RC.
+
 2011-04-25  Werner Koch  <wk@g10code.com>
 
        * certlist.c (gpgsm_add_to_certlist): Mark classify_user_id for
index e854474..a5c122b 100644 (file)
@@ -409,7 +409,7 @@ proc_parameters (ctrl_t ctrl, struct para_data_s *para,
   unsigned int nbits;
   char numbuf[20];
   unsigned char keyparms[100];
-  int rc;
+  int rc = 0;
   ksba_sexp_t public;
   ksba_sexp_t sigkey = NULL;
   int seq;