[prev in list] [next in list] [prev in thread] [next in thread] 

List:       linux-nfsv4
Subject:    Problem on Itanium platform with Kerberos
From:       kwc () citi ! umich ! edu (Kevin Coffman)
Date:       2005-04-20 15:10:37
Message-ID: 20050420191021.79F4D1BAF3 () citi ! umich ! edu
[Download RAW message or body]

I believe this patch should fix the mount problems with Kerberos on 
64-bit machines.  Let me know what you find!

----

diff -puN utils/gssd/write_bytes.h~64_bit_serialize 
utils/gssd/write_bytes.h
--- nfs-utils-1.0.7/utils/gssd/write_bytes.h~64_bit_serialize   
2005-04-20 15:02:53.228121000 -0400
+++ nfs-utils-1.0.7-kwc/utils/gssd/write_bytes.h        2005-04-20 
15:02:53.318031000 -0400
@@ -53,12 +53,13 @@ write_bytes(char **ptr, const char *end,
 inline static int
 write_buffer(char **p, char *end, gss_buffer_desc *arg)
 {
-       if (WRITE_BYTES(p, end, arg->length))
+       int len = (int)arg->length;             /* make an int out of 
size_t */
+       if (WRITE_BYTES(p, end, len))
                return -1;
        if (*p + arg->length > end)
                return -1;
-       memcpy(*p, arg->value, arg->length);
-       *p += arg->length;
+       memcpy(*p, arg->value, len);
+       *p += len;
        return 0;
 }
 
@@ -80,8 +81,10 @@ get_buffer(char **ptr, const char *end, 
 {
        char *p, *q;
        p = *ptr;
-       if (get_bytes(&p, end, &res->length, sizeof(res->length)))
+       int len;
+       if (get_bytes(&p, end, &len, sizeof(len)))
                return -1;
+       res->length = len;              /* promote to size_t if 
necessary */
        q = p + res->length;
        if (q > end || q < p)
                return -1;

_



> Hello Philippe,
> 
> I now have access to a Itanium machine and have been able to reproduce the pr
> oblem.  A small patch to the nfs-utils code which passes the context informat
> ion from user-space to the kernel got me past the "unspported algorithm" prob
> lem, however the mount still did not work.  I hope to get back to this today 
> after fighting fires here the last few days.
> 
> Sorry for the delay and any inconvenience.
> 
> Kevin



[prev in list] [next in list] [prev in thread] [next in thread]