diff options
| author | Joseph Qi <joseph.qi@huawei.com> | 2013-09-11 14:19:51 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-09-11 15:56:39 -0700 |
| commit | 3d94ea51c1d8db6f41268a9d2aea5f5771e9a8d3 (patch) | |
| tree | 28907cc30d1d8502c0af318aba031fce1a7cc9aa /fs/attr.c | |
| parent | df53cd3b70712cd136f10ef79457623c5c3764a4 (diff) | |
ocfs2: clean up dead code in ocfs2_acl_from_xattr()
In ocfs2_acl_from_xattr(), if size is less than sizeof(struct
posix_acl_entry), it returns ERR_PTR(-EINVAL) directly. Then assign (size
/ sizeof(struct posix_acl_entry)) to count which will be at least 1, that
means the following branch (count < 0) and (count == 0) will never be
true.
Signed-off-by: Joseph Qi <joseph.qi@huawei.com>
Cc: Mark Fasheh <mfasheh@suse.com>
Acked-by: Joel Becker <jlbec@evilplan.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/attr.c')
0 files changed, 0 insertions, 0 deletions