diff options
author | Werner Koch <[email protected]> | 2010-10-20 06:51:11 +0000 |
---|---|---|
committer | Werner Koch <[email protected]> | 2010-10-20 06:51:11 +0000 |
commit | 397f9cd6a110751ef3a43ef3d89fd9d6383cc4bc (patch) | |
tree | d625adf918dc283fa7d27cdda28831fff486e852 | |
parent | * armor.test: Tweak the bug 1179 test to only run if Twofish is (diff) | |
download | gnupg-397f9cd6a110751ef3a43ef3d89fd9d6383cc4bc.tar.gz gnupg-397f9cd6a110751ef3a43ef3d89fd9d6383cc4bc.zip |
Remove warnings on VMS
-rw-r--r-- | g10/ChangeLog | 7 | ||||
-rw-r--r-- | g10/encr-data.c | 3 | ||||
-rw-r--r-- | g10/keyid.c | 4 | ||||
-rw-r--r-- | g10/parse-packet.c | 2 | ||||
-rw-r--r-- | g10/trustdb.c | 2 |
5 files changed, 15 insertions, 3 deletions
diff --git a/g10/ChangeLog b/g10/ChangeLog index 7de067e62..abf2084c6 100644 --- a/g10/ChangeLog +++ b/g10/ChangeLog @@ -1,3 +1,10 @@ +2010-10-20 Werner Koch <[email protected]> + + * 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 <[email protected]> * app-openpgp.c (parse_algorithm_attribute): Remove extra const in diff --git a/g10/encr-data.c b/g10/encr-data.c index a3a9728cb..8347e2a35 100644 --- a/g10/encr-data.c +++ b/g10/encr-data.c @@ -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); diff --git a/g10/keyid.c b/g10/keyid.c index 4e162dd32..f2657f3a5 100644 --- a/g10/keyid.c +++ b/g10/keyid.c @@ -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 { diff --git a/g10/parse-packet.c b/g10/parse-packet.c index 4b0fd1c2b..259447738 100644 --- a/g10/parse-packet.c +++ b/g10/parse-packet.c @@ -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; } diff --git a/g10/trustdb.c b/g10/trustdb.c index 55ed7d210..cffbc28e6 100644 --- a/g10/trustdb.c +++ b/g10/trustdb.c @@ -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; |