- (dtucker) [auth-krb5.c] Fix -Wsign-compare warning in non-Heimdal path.

   Patch from djm@.

diff --git a/ChangeLog b/ChangeLog
index 468ae72a736907cdaacf978fd91774eb1146a5cd..839e7b912661ba282c538eeedd1145d4a7c08373 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -6,6 +6,8 @@
    unnecessary prototype.
  - (dtucker) [openbsd-compat/setenv.c] Sync changes from OpenBSD setenv.c
    revs 1.7 - 1.9.
+ - (dtucker) [auth-krb5.c] Fix -Wsign-compare warning in non-Heimdal path.
+   Patch from djm@.
 
 20051105
  - (djm) OpenBSD CVS Sync

diff --git a/auth-krb5.c b/auth-krb5.c
index c7367b49a2772e4d8aacb14de39165cbdda22af8..a84e5401c7d448b1fddfde3e346d803108671291 100644 (file)
--- a/auth-krb5.c
+++ b/auth-krb5.c
@@ -218,7 +218,7 @@ ssh_krb5_cc_gen(krb5_context ctx, krb5_ccache *ccache) {
 
        ret = snprintf(ccname, sizeof(ccname),
            "FILE:/tmp/krb5cc_%d_XXXXXXXXXX", geteuid());
-       if (ret == -1 || ret >= sizeof(ccname))
+       if (ret < 0 || (size_t)ret >= sizeof(ccname))
                return ENOMEM;
 
        old_umask = umask(0177);