Searched hist:"04 ccd53f09741c4bc54ab36db000bc1383e4812e" (Results 1 – 1 of 1) sorted by relevance
| /openbmc/linux/security/apparmor/ |
| H A D | lib.c | diff 04ccd53f09741c4bc54ab36db000bc1383e4812e Fri Aug 27 20:33:28 CDT 2010 John Johansen <john.johansen@canonical.com> AppArmor: Fix splitting an fqname into separate namespace and profile names
As per Dan Carpenter <error27@gmail.com>
If we have a ns name without a following profile then in the original
code it did "*ns_name = &name[1];". "name" is NULL so "*ns_name" is
0x1. That isn't useful and could cause an oops when this function is
called from aa_remove_profiles().
Beyond this the assignment of the namespace name was wrong in the case
where the profile name was provided as it was being set to &name[1]
after name = skip_spaces(split + 1);
Move the ns_name assignment before updating name for the split and
also add skip_spaces, making the interface more robust.
Signed-off-by: John Johansen <john.johansen@canonical.com>
Signed-off-by: James Morris <jmorris@namei.org>
|