[Trusty][PATCH] UBUNTU: SAUCE: apparmor: delete extra variable dev_path

John Johansen john.johansen at canonical.com
Wed Nov 23 23:10:17 UTC 2016


On 11/23/2016 02:55 PM, Luis Henriques wrote:
> BugLink: https://bugs.launchpad.net/bugs/1644302
> 
> Commit 74a2b3a08798 ("UBUNTU: SAUCE: apparmor: fix sleep in critical section")
> introduced a regression by not deleting a local variable dev_path, shadowing
> another variable with broader scope and preventing it from being initialised.
> 
> This patch simply deletes the extra struct path dev_path variable.
> 
> Signed-off-by: Luis Henriques <luis.henriques at canonical.com>

yep
Acked-by: John Johansen <john.johansen at canonical.com>

> ---
>  security/apparmor/mount.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/security/apparmor/mount.c b/security/apparmor/mount.c
> index 076200ece0f5..f59937529ba4 100644
> --- a/security/apparmor/mount.c
> +++ b/security/apparmor/mount.c
> @@ -517,8 +517,6 @@ int aa_new_mount(struct aa_label *label, const char *orig_dev_name,
>  		put_filesystem(fstype);
>  
>  		if (requires_dev) {
> -			struct path dev_path;
> -
>  			if (!dev_name || !*dev_name) {
>  				error = -ENOENT;
>  				goto out;
> 





More information about the kernel-team mailing list