diff options
| author | Dave Chinner <david@fromorbit.com> | 2010-01-20 00:47:48 +0100 |
|---|---|---|
| committer | Dave Chinner <david@fromorbit.com> | 2010-01-20 00:47:48 +0100 |
| commit | a9273ca5c6814f393e18ed66645f817b2b71e9ad (patch) | |
| tree | 3714b7de8b233a31ea6995754f80349327738a88 /fs/xfs/linux-2.6/xfs_ioctl32.c | |
| parent | xfs: xfs_buf_iomove() doesn't care about signedness (diff) | |
| download | linux-a9273ca5c6814f393e18ed66645f817b2b71e9ad.tar.xz linux-a9273ca5c6814f393e18ed66645f817b2b71e9ad.zip | |
xfs: convert attr to use unsigned names
To be consistent with the directory code, the attr code should use
unsigned names. Convert the names from the vfs at the highest level
to unsigned, and ænsure they are consistenly used as unsigned down
to disk.
Signed-off-by: Dave Chinner <david@fromorbit.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'fs/xfs/linux-2.6/xfs_ioctl32.c')
| -rw-r--r-- | fs/xfs/linux-2.6/xfs_ioctl32.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/xfs/linux-2.6/xfs_ioctl32.c b/fs/xfs/linux-2.6/xfs_ioctl32.c index be1527b1670c..0bf6d61f0528 100644 --- a/fs/xfs/linux-2.6/xfs_ioctl32.c +++ b/fs/xfs/linux-2.6/xfs_ioctl32.c @@ -411,7 +411,7 @@ xfs_compat_attrmulti_by_handle( compat_xfs_fsop_attrmulti_handlereq_t am_hreq; struct dentry *dentry; unsigned int i, size; - char *attr_name; + unsigned char *attr_name; if (!capable(CAP_SYS_ADMIN)) return -XFS_ERROR(EPERM); @@ -440,7 +440,7 @@ xfs_compat_attrmulti_by_handle( error = 0; for (i = 0; i < am_hreq.opcount; i++) { - ops[i].am_error = strncpy_from_user(attr_name, + ops[i].am_error = strncpy_from_user((char *)attr_name, compat_ptr(ops[i].am_attrname), MAXNAMELEN); if (ops[i].am_error == 0 || ops[i].am_error == MAXNAMELEN) |