diff options
author | rwatson <rwatson@FreeBSD.org> | 2003-06-05 04:58:58 +0000 |
---|---|---|
committer | rwatson <rwatson@FreeBSD.org> | 2003-06-05 04:58:58 +0000 |
commit | 5fd55dbf851e2109d785f953b69000ce3b5b6ce3 (patch) | |
tree | 28440a873d3ab2ad069985d77e596df7b97ba5d1 /sys/ufs | |
parent | 8d5437ca29f38633b7ffcaae709f015bddc8f45e (diff) | |
download | FreeBSD-src-5fd55dbf851e2109d785f953b69000ce3b5b6ce3.zip FreeBSD-src-5fd55dbf851e2109d785f953b69000ce3b5b6ce3.tar.gz |
Don't special-case handling of the empty string in the UFS1
extended attribute retrieval code: it's no longer special-cased,
and is caught by the normal UFS1 EA validity checks (and, in
fact, returns the same error, EINVAL).
Obtained from: TrustedBSD Project
Sponsored by: DARPA, Network Associates Laboratories
Diffstat (limited to 'sys/ufs')
-rw-r--r-- | sys/ufs/ufs/ufs_extattr.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/sys/ufs/ufs/ufs_extattr.c b/sys/ufs/ufs/ufs_extattr.c index cc5a9cc..55fc2f6 100644 --- a/sys/ufs/ufs/ufs_extattr.c +++ b/sys/ufs/ufs/ufs_extattr.c @@ -839,11 +839,8 @@ ufs_extattr_get(struct vnode *vp, int attrnamespace, const char *name, if (!(ump->um_extattr.uepm_flags & UFS_EXTATTR_UEPM_STARTED)) return (EOPNOTSUPP); - if (strlen(name) == 0) { - /* XXX retrieve attribute lists. */ - /* XXX should probably be checking for name == NULL? */ + if (strlen(name) == 0) return (EINVAL); - } error = extattr_check_cred(vp, attrnamespace, cred, td, IREAD); if (error) |