Skip to content

Commit f009918

Browse files
Anton Blancharddavem330
authored andcommitted
RxRPC: Fix v1 keys
commit 3394128 (RxRPC: Allow key payloads to be passed in XDR form) broke klog for me. I notice the v1 key struct had a kif_version field added: -struct rxkad_key { - u16 security_index; /* RxRPC header security index */ - u16 ticket_len; /* length of ticket[] */ - u32 expiry; /* time at which expires */ - u32 kvno; /* key version number */ - u8 session_key[8]; /* DES session key */ - u8 ticket[0]; /* the encrypted ticket */ -}; +struct rxrpc_key_data_v1 { + u32 kif_version; /* 1 */ + u16 security_index; + u16 ticket_length; + u32 expiry; /* time_t */ + u32 kvno; + u8 session_key[8]; + u8 ticket[0]; +}; However the code in rxrpc_instantiate strips it away: data += sizeof(kver); datalen -= sizeof(kver); Removing kif_version fixes my problem. Signed-off-by: Anton Blanchard <anton@samba.org> Signed-off-by: David Howells <dhowells@redhat.com> Signed-off-by: David S. Miller <davem@davemloft.net>
1 parent 1000345 commit f009918

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

include/keys/rxrpc-type.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,6 @@ struct rxrpc_key_token {
9999
* structure of raw payloads passed to add_key() or instantiate key
100100
*/
101101
struct rxrpc_key_data_v1 {
102-
u32 kif_version; /* 1 */
103102
u16 security_index;
104103
u16 ticket_length;
105104
u32 expiry; /* time_t */

0 commit comments

Comments
 (0)