apparmor: signedness bug in aa_xattrs_match()
Subsystems (3)
PATCH 1/1 apparmor: signedness bug in aa_xattrs_match()
The "size" variable needs to be signed ssize_t so that the error handling works correctly.
Fixes 8e51f9087f40 ("apparmor: Add support for attaching profiles via xattr, presence and value")
Signed-off-by Dan Carpenter <[email protected]>
security/apparmor/domain.c +1 -1
@@ -314,7 +314,7 @@ static int aa_xattrs_match(const struct linux_binprm *bprm, struct aa_profile *profile, unsigned int state) { int i; -size_t size; +ssize_t size;struct dentry *d;char *value =NULL ;int value_size =0 , ret = profile->xattr_count;
John Johansen
maintainerQuoted 6 lines · Dan Carpenter
The "size" variable needs to be signed
ssize_tso that the error handling works correctly.Fixes 8e51f9087f40 ("apparmor: Add support for attaching profiles via xattr, presence and value")Signed-off-by Dan Carpenter <[email protected]>
yep thanks I have pulled this in to apparmor-next
Quoted 14 lines
security/apparmor/domain.c +1
-1@@ -314,7 +314,7 @@ static int aa_xattrs_match(const struct linux_binprm *bprm, struct aa_profile *profile, unsigned int state) { int i; -size_t size; +ssize_t size;struct dentry *d;char *value =NULL ;int value_size =0 , ret = profile->xattr_count;