Hi David, On Fri, 15 Apr 2005 15:45:02 +0100 David Howells <[email protected]> wrote: > > @@ -746,10 +747,79 @@ static void *do_smb_super_data_conv(void > return raw_data; > } > > +struct compat_nfs_string { > + compat_uint_t len; > + compat_uptr_t __user data; ^^^^^^ This __user (and the others later) add nothing (I think) as compat_uptr_t is generally just u32 and compat_ptr() does the right casting. Otherwise, the patch looks fine for a minimal fix. I agree with David and Bryan that we need to get the fs specific stuff out of compat.c in the longer term. -- Cheers, Stephen Rothwell [email protected] http://www.canb.auug.org.au/~sfr/
Attachment:
pgpEoBRfjVPBm.pgp
Description: PGP signature
- References:
- [PATCH] Add 32-bit compatibility for NFSv4 mount
- From: David Howells <[email protected]>
- [PATCH] Add 32-bit compatibility for NFSv4 mount
- Prev by Date: Poor I/O Performance with MegaRaid SATA 150-4; bug or feature?
- Next by Date: Re: touch screen driver --working?
- Previous by thread: [PATCH] Add 32-bit compatibility for NFSv4 mount
- Next by thread: Using __user with compat_uptr_t
- Index(es):