summary refs log tree commit diff
path: root/fs/afs
diff options
context:
space:
mode:
authorDavid Howells <dhowells@redhat.com>2017-11-02 15:27:49 +0000
committerDavid Howells <dhowells@redhat.com>2017-11-13 15:38:18 +0000
commit03dc2cfca536df89f4b1747caad9324c9be482fa (patch)
tree7bb0cedf49ccd5f056edc2950ece6428413f9581 /fs/afs
parentf4b3526d83c40dd8bf5948b9d7a1b2c340f0dcc8 (diff)
downloadlinux-03dc2cfca536df89f4b1747caad9324c9be482fa.tar.gz
afs: Fix the afs_uuid struct to make the char-sized fields signed
In AFS's encoding of a UUID, the eight 'char' fields are all signed, so
represent them with __s8 rather than __u8.  This makes the compiler
sign-extend them correctly when XDR-encoding them.

Signed-off-by: David Howells <dhowells@redhat.com>
Diffstat (limited to 'fs/afs')
-rw-r--r--fs/afs/afs.h6
1 files changed, 3 insertions, 3 deletions
diff --git a/fs/afs/afs.h b/fs/afs/afs.h
index 0d837bddbf7d..2e2887a7d331 100644
--- a/fs/afs/afs.h
+++ b/fs/afs/afs.h
@@ -79,9 +79,9 @@ struct afs_uuid {
 	__be32		time_low;			/* low part of timestamp */
 	__be16		time_mid;			/* mid part of timestamp */
 	__be16		time_hi_and_version;		/* high part of timestamp and version  */
-	__u8		clock_seq_hi_and_reserved;	/* clock seq hi and variant */
-	__u8		clock_seq_low;			/* clock seq low */
-	__u8		node[6];			/* spatially unique node ID (MAC addr) */
+	__s8		clock_seq_hi_and_reserved;	/* clock seq hi and variant */
+	__s8		clock_seq_low;			/* clock seq low */
+	__s8		node[6];			/* spatially unique node ID (MAC addr) */
 };
 
 /*