Remove warnings on VMS
authorWerner Koch <wk@gnupg.org>
Wed, 20 Oct 2010 06:51:11 +0000 (06:51 +0000)
committerWerner Koch <wk@gnupg.org>
Wed, 20 Oct 2010 06:51:11 +0000 (06:51 +0000)
g10/ChangeLog
g10/encr-data.c
g10/keyid.c
g10/parse-packet.c
g10/trustdb.c

index 7de067e..abf2084 100644 (file)
@@ -1,3 +1,10 @@
+2010-10-20  Werner Koch  <wk@g10code.com>
+
+       * encr-data.c (decode_filter): Cast -1 to size_t.  Reported by
+       Steven M. Schweda.
+       * parse-packet.c (parse_key): Ditto.
+       * trustdb.c (validate_keys): Likewise to unsigned int.
+
 2010-10-18  Werner Koch  <wk@g10code.com>
 
        * app-openpgp.c (parse_algorithm_attribute): Remove extra const in
index a3a9728..8347e2a 100644 (file)
@@ -298,7 +298,8 @@ decode_filter( void *opaque, int control, IOBUF a, byte *buf, size_t *ret_len)
     if( control == IOBUFCTRL_UNDERFLOW ) {
        assert(a);
        n = iobuf_read( a, buf, size );
-       if( n == -1 ) n = 0;
+       if (n == (size_t)(-1))
+          n = 0;
        if( n ) {
             if (fc->cipher_hd)
                 cipher_decrypt( fc->cipher_hd, buf, buf, n);
index 4e162dd..f2657f3 100644 (file)
@@ -444,6 +444,10 @@ mk_datestr (char *buffer, time_t atime)
 {
     struct tm *tp;
 
+    /* Note: VMS uses an unsigned time_t thus the compiler yields a
+       warning here.  You may ignore this warning or def out this test
+       for VMS.  The proper way to handle this would be a configure
+       test to a detect properly implemented unsigned time_t.  */
     if ( atime < 0 ) /* 32 bit time_t and after 2038-01-19 */
         strcpy (buffer, "????" "-??" "-??"); /* mark this as invalid */
     else {
index 4b0fd1c..2594477 100644 (file)
@@ -1842,7 +1842,7 @@ parse_key( IOBUF inp, int pkttype, unsigned long pktlen,
                    }
                    snlen = iobuf_get (inp);
                    pktlen--;
-                    if (pktlen < snlen || snlen == -1) {
+                    if (pktlen < snlen || snlen == (size_t)(-1)) {
                        rc = G10ERR_INVALID_PACKET;
                        goto leave;
                     }
index 55ed7d2..cffbc28 100644 (file)
@@ -2343,7 +2343,7 @@ validate_keys (int interactive)
            {
              k->ownertrust = ask_ownertrust (k->kid,min);
 
-             if (k->ownertrust == -1)
+             if (k->ownertrust == (unsigned int)(-1))
                {
                  quit=1;
                  goto leave;