krb5 commit: Fix t_marshal on big-endian platforms

Greg Hudson ghudson at MIT.EDU
Mon May 19 10:17:21 EDT 2014


https://github.com/krb5/krb5/commit/cacdcf8ebe184326579fabef3ae3f86b16dade81
commit cacdcf8ebe184326579fabef3ae3f86b16dade81
Author: Greg Hudson <ghudson at mit.edu>
Date:   Sun May 18 16:40:35 2014 -0400

    Fix t_marshal on big-endian platforms
    
    t_marshal.c attempts to skip the version 1 and 2 tests on big-endian
    platforms, but didn't do so correctly.  Correctly start at version 3
    on big-endian platforms, and change the way we do it to avoid
    preprocessor conditionals inside a function body.

 src/lib/krb5/ccache/t_marshal.c |   18 ++++++++++++------
 1 files changed, 12 insertions(+), 6 deletions(-)

diff --git a/src/lib/krb5/ccache/t_marshal.c b/src/lib/krb5/ccache/t_marshal.c
index 07685d6..d6908da 100644
--- a/src/lib/krb5/ccache/t_marshal.c
+++ b/src/lib/krb5/ccache/t_marshal.c
@@ -36,6 +36,17 @@
 #include <arpa/inet.h>
 #include <fcntl.h>
 
+/*
+ * Versions 1 and 2 of the ccache format use native byte order representations
+ * of integers.  The test data below is from a little-endian platform.  Skip
+ * those tests on big-endian platforms by starting at version 3.
+ */
+#ifdef K5_BE
+#define FIRST_VERSION 3
+#else
+#define FIRST_VERSION 1
+#endif
+
 /* Each test contains the expected binary representation of a credential cache
  * divided into the header, the default principal, and two credentials. */
 const struct test {
@@ -275,13 +286,8 @@ main(int argc, char **argv)
     if (krb5_init_context(&context) != 0)
         abort();
 
-    for (version = 1; version <= 4; version++) {
+    for (version = FIRST_VERSION; version <= 4; version++) {
         t = &tests[version - 1];
-#ifdef K5_BE
-        /* The version 1 and 2 test data use little-endian integers. */
-        if (version == 0 || version == 1)
-            continue;
-#endif
 
         /* Test principal unmarshalling and marshalling. */
         if (k5_unmarshal_princ(t->princ, t->princlen, version, &princ) != 0)


More information about the cvs-krb5 mailing list