[patch] Yama: remove an unused variable

[patch] Yama: remove an unused variable

[Dan Carpenter]

GCC complains that we don't use "one" any more after 389da25f93 "Yama:
add additional ptrace scopes".

security/yama/yama_lsm.c:322:12: warning: ‘one’ defined but not used
	[-Wunused-variable]

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>

diff --git a/security/yama/yama_lsm.c b/security/yama/yama_lsm.c
index afb04cb..c852f74 100644
--- a/security/yama/yama_lsm.c
+++ b/security/yama/yama_lsm.c
@@ -319,7 +319,6 @@ static int yama_dointvec_minmax(struct ctl_table *table, int write,
 }
 
 static int zero;
-static int one = 1;
 static int max_scope = YAMA_SCOPE_NO_ATTACH;
 
 struct ctl_path yama_sysctl_path[] = {

Re: [patch] Yama: remove an unused variable

[Kees Cook]

On Fri, Apr 20, 2012 at 6:35 AM, Dan Carpenter <dan.carpenter@oracle.com> wrote:
> GCC complains that we don't use "one" any more after 389da25f93 "Yama:
> add additional ptrace scopes".
>
> security/yama/yama_lsm.c:322:12: warning: ‘one’ defined but not used
>        [-Wunused-variable]
>
> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>

Acked-by: Kees Cook <keescook@chromium.org>

Thanks for catching that! I was still building with another set of
patches that used it. Whoops! :)

-Kees

-- 
Kees Cook
Chrome OS Security
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [patch] Yama: remove an unused variable

[James Morris]

[-- Attachment #1: Type: TEXT/PLAIN, Size: 729 bytes --]

On Fri, 20 Apr 2012, Kees Cook wrote:

> On Fri, Apr 20, 2012 at 6:35 AM, Dan Carpenter <dan.carpenter@oracle.com> wrote:
> > GCC complains that we don't use "one" any more after 389da25f93 "Yama:
> > add additional ptrace scopes".
> >
> > security/yama/yama_lsm.c:322:12: warning: ?one? defined but not used
> >        [-Wunused-variable]
> >
> > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
> 
> Acked-by: Kees Cook <keescook@chromium.org>
> 
> Thanks for catching that! I was still building with another set of
> patches that used it. Whoops! :)

Thanks, applied to
git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security.git next



- James
-- 
James Morris
<jmorris@namei.org>