| Message ID | 20180319103444.GC8543@mwanda (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On 03/19/2018 03:34 AM, Dan Carpenter wrote: > 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 <dan.carpenter@oracle.com> > yep thanks I have pulled this in to apparmor-next > diff --git a/security/apparmor/domain.c b/security/apparmor/domain.c > index 57cc892e05a2..590b7e8cd21c 100644 > --- a/security/apparmor/domain.c > +++ b/security/apparmor/domain.c > @@ -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; > -- To unsubscribe from this list: send the line "unsubscribe linux-security-module" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/security/apparmor/domain.c b/security/apparmor/domain.c index 57cc892e05a2..590b7e8cd21c 100644 --- a/security/apparmor/domain.c +++ b/security/apparmor/domain.c @@ -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;
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 <dan.carpenter@oracle.com> -- To unsubscribe from this list: send the line "unsubscribe linux-security-module" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html