diff mbox

Compilation warning when CONFIG_SECURITY is not set

Message ID 1283590159-10244-1-git-send-email-daniel.lezcano@free.fr
State Rejected
Delegated to: Leann Ogasawara
Headers show

Commit Message

Daniel Lezcano Sept. 4, 2010, 8:49 a.m. UTC
Function prototype differs when CONFIG_SECURITY is not set leading to
a warning.

Signed-off-by: Daniel Lezcano <daniel.lezcano@free.fr>
---
 include/linux/security.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

Comments

Tim Gardner Sept. 7, 2010, 2:11 p.m. UTC | #1
On 09/04/2010 02:49 AM, Daniel Lezcano wrote:
> Function prototype differs when CONFIG_SECURITY is not set leading to
> a warning.
>
> Signed-off-by: Daniel Lezcano<daniel.lezcano@free.fr>
> ---
>   include/linux/security.h |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/include/linux/security.h b/include/linux/security.h
> index d7d65e0..a380eed 100644
> --- a/include/linux/security.h
> +++ b/include/linux/security.h
> @@ -2240,7 +2240,7 @@ static inline int security_task_create(unsigned long clone_flags)
>   	return 0;
>   }
>
> -static inline int security_task_free(struct task_struct *task)
> +static inline void security_task_free(struct task_struct *task)
>   { }
>
>   static inline int security_cred_alloc_blank(struct cred *cred, gfp_t gfp)

Ack after changing the commit log to "UBUNTU: SAUCE: ...". The patch 
that introduced this warning was Maverick commit 
2aec770a575b15cb1e43c2e16dff4f00a02a8c9a.

Acked-by: Tim Gardner <tim.gardner@canonical.com>
diff mbox

Patch

diff --git a/include/linux/security.h b/include/linux/security.h
index d7d65e0..a380eed 100644
--- a/include/linux/security.h
+++ b/include/linux/security.h
@@ -2240,7 +2240,7 @@  static inline int security_task_create(unsigned long clone_flags)
 	return 0;
 }
 
-static inline int security_task_free(struct task_struct *task)
+static inline void security_task_free(struct task_struct *task)
 { }
 
 static inline int security_cred_alloc_blank(struct cred *cred, gfp_t gfp)