diff options
author | James Morris <jmorris@namei.org> | 2010-02-16 01:29:06 -0500 |
---|---|---|
committer | James Morris <jmorris@namei.org> | 2010-02-16 01:29:06 -0500 |
commit | 2da5d31bc72d0a36dc16af7f5d5baa4f86df9c76 (patch) | |
tree | 9d5bd3cc7d9e5b1beecc954bb5337af8454d352d | |
parent | 97d6931ead3e89a764cdaa3ad0924037367f0d34 (diff) |
security: fix a couple of sparse warnings
Fix a couple of sparse warnings for callers of
context_struct_to_string, which takes a *u32, not an *int.
These cases are harmless as the values are not used.
Signed-off-by: James Morris <jmorris@namei.org>
Acked-by: KaiGai Kohei <kaigai@ak.jp.nec.com>
-rw-r--r-- | security/selinux/ss/services.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/security/selinux/ss/services.c b/security/selinux/ss/services.c index 4e976f58b980..0e5c3a422a8e 100644 --- a/security/selinux/ss/services.c +++ b/security/selinux/ss/services.c | |||
@@ -455,7 +455,8 @@ static void security_dump_masked_av(struct context *scontext, | |||
455 | char *scontext_name = NULL; | 455 | char *scontext_name = NULL; |
456 | char *tcontext_name = NULL; | 456 | char *tcontext_name = NULL; |
457 | char *permission_names[32]; | 457 | char *permission_names[32]; |
458 | int index, length; | 458 | int index; |
459 | u32 length; | ||
459 | bool need_comma = false; | 460 | bool need_comma = false; |
460 | 461 | ||
461 | if (!permissions) | 462 | if (!permissions) |
@@ -807,7 +808,7 @@ int security_bounded_transition(u32 old_sid, u32 new_sid) | |||
807 | if (rc) { | 808 | if (rc) { |
808 | char *old_name = NULL; | 809 | char *old_name = NULL; |
809 | char *new_name = NULL; | 810 | char *new_name = NULL; |
810 | int length; | 811 | u32 length; |
811 | 812 | ||
812 | if (!context_struct_to_string(old_context, | 813 | if (!context_struct_to_string(old_context, |
813 | &old_name, &length) && | 814 | &old_name, &length) && |