int/ptr bug in gssapi code
authorKen Raeburn <raeburn@mit.edu>
Thu, 5 Feb 2009 01:49:21 +0000 (01:49 +0000)
committerKen Raeburn <raeburn@mit.edu>
Thu, 5 Feb 2009 01:49:21 +0000 (01:49 +0000)
Fix a pointer argument passed where an integer is needed.
Update Sun compiler options to make that an error.  (The options we're
currently using make it an error for assignment but not for argument passing.)

ticket: 6363
target_version: 1.7
tags: pullup

git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21887 dc483132-0cff-0310-8789-dd5450dbe970

src/aclocal.m4
src/lib/gssapi/krb5/k5seal.c

index c494750ad129d45708c9179d26e3ead5a6bb0cdf..4f1c9d6e2f154b3bd28d2070b836f902f8180e9f 100644 (file)
@@ -664,7 +664,7 @@ else
     # works, but it also means that declaration-in-code warnings won't
     # be issued.
     # -v -fd -errwarn=E_DECLARATION_IN_CODE ...
-    WARN_CFLAGS="-errtags=yes -errwarn=E_BAD_PTR_INT_COMBINATION -errwarn=E_PTR_TO_VOID_IN_ARITHMETIC"
+    WARN_CFLAGS="-errtags=yes -errwarn=E_BAD_PTR_INT_COMBINATION -errwarn=E_BAD_PTR_INT_COMB_ARG -errwarn=E_PTR_TO_VOID_IN_ARITHMETIC"
     WARN_CXXFLAGS="-errtags=yes +w +w2 -xport64"
   fi
 fi
index bff1ab18581ebff3aada543a780d8e886835b389..1949020ab19802ecd94c3c06d96de170139e871b 100644 (file)
@@ -258,7 +258,7 @@ make_seal_token_v1 (krb5_context context,
             unsigned char bigend_seqnum[4];
             krb5_keyblock *enc_key;
             int i;
-            store_32_be(seqnum, bigend_seqnum);
+            store_32_be(*seqnum, bigend_seqnum);
             code = krb5_copy_keyblock (context, enc, &enc_key);
             if (code)
             {