diff --git a/libselinux-rhat.patch b/libselinux-rhat.patch index 5edcb6f..c55f204 100644 --- a/libselinux-rhat.patch +++ b/libselinux-rhat.patch @@ -1,3 +1,16 @@ +diff --git a/libselinux/include/selinux/avc.h b/libselinux/include/selinux/avc.h +index da18e41..9655957 100644 +--- a/libselinux/include/selinux/avc.h ++++ b/libselinux/include/selinux/avc.h +@@ -130,7 +130,7 @@ struct avc_memory_callback { + + struct avc_log_callback { + /* log the printf-style format and arguments. */ +- void (*func_log) (const char *fmt, ...); ++ void (*func_log) (const char *fmt, ...) __attribute__((__format__(printf, 1, 2))); + /* store a string representation of auditdata (corresponding + to the given security class) into msgbuf. */ + void (*func_audit) (void *auditdata, security_class_t cls, diff --git a/libselinux/include/selinux/label.h b/libselinux/include/selinux/label.h index 1a54307..f6eeb21 100644 --- a/libselinux/include/selinux/label.h @@ -15,7 +28,7 @@ index 1a54307..f6eeb21 100644 /* * Label operations diff --git a/libselinux/include/selinux/selinux.h b/libselinux/include/selinux/selinux.h -index 2985f6f..826ed71 100644 +index 2985f6f..e0b2dd4 100644 --- a/libselinux/include/selinux/selinux.h +++ b/libselinux/include/selinux/selinux.h @@ -139,7 +139,10 @@ struct av_decision { @@ -42,6 +55,15 @@ index 2985f6f..826ed71 100644 /* Free the memory allocated by matchpathcon_init. */ extern void matchpathcon_fini(void); +@@ -537,7 +545,7 @@ extern int selinux_check_securetty_context(const security_context_t tty_context) + Normally, this is determined automatically during libselinux + initialization, but this is not always possible, e.g. for /sbin/init + which performs the initial mount of selinuxfs. */ +-void set_selinuxmnt(char *mnt); ++void set_selinuxmnt(const char *mnt); + + /* Check if selinuxfs exists as a kernel filesystem */ + int selinuxfs_exists(void); diff --git a/libselinux/man/man3/matchpathcon.3 b/libselinux/man/man3/matchpathcon.3 index cdbb252..b6814ed 100644 --- a/libselinux/man/man3/matchpathcon.3 @@ -109,6 +131,67 @@ index 8674e37..fc5b120 100644 .BR selinux_set_callback (3), .BR selinux (8) - +diff --git a/libselinux/man/man8/selinux.8 b/libselinux/man/man8/selinux.8 +index 1fc5b95..9f16f77 100644 +--- a/libselinux/man/man8/selinux.8 ++++ b/libselinux/man/man8/selinux.8 +@@ -12,7 +12,7 @@ enforcement of many kinds of mandatory access control policies, + including those based on the concepts of Type Enforcement®, Role- + Based Access Control, and Multi-Level Security. Background + information and technical documentation about SELinux can be found at +-http://www.nsa.gov/selinux. ++http://www.nsa.gov/research/selinux. + + The + .I /etc/selinux/config +diff --git a/libselinux/src/Makefile b/libselinux/src/Makefile +index 1ddddb0..985842d 100644 +--- a/libselinux/src/Makefile ++++ b/libselinux/src/Makefile +@@ -51,9 +51,29 @@ endif + GENERATED=$(SWIGCOUT) $(SWIGRUBYCOUT) selinuxswig_python_exception.i + SRCS= $(filter-out $(UNUSED_SRCS) $(GENERATED) audit2why.c, $(wildcard *.c)) + ++MAX_STACK_SIZE=32768 ++ + OBJS= $(patsubst %.c,%.o,$(SRCS)) + LOBJS= $(patsubst %.c,%.lo,$(SRCS)) +-CFLAGS ?= -Werror -Wall -W -Wundef -Wshadow -Wmissing-noreturn -Wmissing-format-attribute ++CFLAGS ?= -Wall -W -Wundef -Wformat-y2k -Wformat-security -Winit-self -Wmissing-include-dirs \ ++ -Wunused -Wunknown-pragmas -Wstrict-aliasing -Wshadow -Wpointer-arith \ ++ -Wbad-function-cast -Wcast-align -Wwrite-strings -Wlogical-op -Waggregate-return \ ++ -Wstrict-prototypes -Wold-style-definition -Wmissing-prototypes \ ++ -Wmissing-declarations -Wmissing-noreturn -Wmissing-format-attribute \ ++ -Wredundant-decls -Wnested-externs -Winline -Winvalid-pch -Wvolatile-register-var \ ++ -Wdisabled-optimization -Wbuiltin-macro-redefined -Wmudflap -Wpacked-bitfield-compat \ ++ -Wsync-nand -Wattributes -Wcoverage-mismatch -Wmultichar -Wcpp \ ++ -Wdeprecated-declarations -Wdiv-by-zero -Wdouble-promotion -Wendif-labels -Wextra \ ++ -Wformat-contains-nul -Wformat-extra-args -Wformat-zero-length -Wformat=2 -Wmultichar \ ++ -Wnormalized=nfc -Woverflow -Wpointer-to-int-cast -Wpragmas -Wsuggest-attribute=const \ ++ -Wsuggest-attribute=noreturn -Wsuggest-attribute=pure -Wtrampolines \ ++ -Wno-missing-field-initializers -Wno-sign-compare -Wjump-misses-init \ ++ -Wno-format-nonliteral -Wframe-larger-than=$(MAX_STACK_SIZE) -Wp,-D_FORTIFY_SOURCE=2 \ ++ -fstack-protector-all --param=ssp-buffer-size=4 -fexceptions \ ++ -fasynchronous-unwind-tables -fdiagnostics-show-option -funit-at-a-time \ ++ -fipa-pure-const -Wno-suggest-attribute=pure -Wno-suggest-attribute=const \ ++ -Werror -Wno-aggregate-return -Wno-redundant-decls ++ + override CFLAGS += -I../include -I$(INCLUDEDIR) -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 $(EMFLAGS) + RANLIB=ranlib + +diff --git a/libselinux/src/avc_internal.h b/libselinux/src/avc_internal.h +index 53610e8..f851659 100644 +--- a/libselinux/src/avc_internal.h ++++ b/libselinux/src/avc_internal.h +@@ -20,7 +20,7 @@ + extern void *(*avc_func_malloc) (size_t) hidden; + extern void (*avc_func_free) (void *)hidden; + +-extern void (*avc_func_log) (const char *, ...)hidden; ++extern void (*avc_func_log) (const char *, ...) __attribute__((__format__(printf,1,2))) hidden; + extern void (*avc_func_audit) (void *, security_class_t, char *, size_t)hidden; + + extern int avc_using_threads hidden; diff --git a/libselinux/src/callbacks.c b/libselinux/src/callbacks.c index b245364..7c47222 100644 --- a/libselinux/src/callbacks.c @@ -121,6 +204,42 @@ index b245364..7c47222 100644 va_start(ap, fmt); rc = vfprintf(stderr, fmt, ap); va_end(ap); +diff --git a/libselinux/src/get_default_type.c b/libselinux/src/get_default_type.c +index ca3d291..27f2ae5 100644 +--- a/libselinux/src/get_default_type.c ++++ b/libselinux/src/get_default_type.c +@@ -27,7 +27,8 @@ int get_default_type(const char *role, char **type) + static int find_default_type(FILE * fp, const char *role, char **type) + { + char buf[250]; +- char *ptr = "", *end, *t; ++ const char *ptr = "", *end; ++ char *t; + size_t len; + int found = 0; + +diff --git a/libselinux/src/init.c b/libselinux/src/init.c +index 00afde7..6d1ef33 100644 +--- a/libselinux/src/init.c ++++ b/libselinux/src/init.c +@@ -28,7 +28,7 @@ int obj_class_compat = 1; + * The file system is read/write + * then set this as the default file system. + */ +-static int verify_selinuxmnt(char *mnt) ++static int verify_selinuxmnt(const char *mnt) + { + struct statfs sfbuf; + int rc; +@@ -139,7 +139,7 @@ void fini_selinuxmnt(void) + + hidden_def(fini_selinuxmnt) + +-void set_selinuxmnt(char *mnt) ++void set_selinuxmnt(const char *mnt) + { + selinux_mnt = strdup(mnt); + } diff --git a/libselinux/src/label_file.c b/libselinux/src/label_file.c index 7bc46cc..82a608c 100644 --- a/libselinux/src/label_file.c @@ -222,8 +341,21 @@ index 7bc46cc..82a608c 100644 if (status) goto finish; } +diff --git a/libselinux/src/load_policy.c b/libselinux/src/load_policy.c +index f569664..10e29b9 100644 +--- a/libselinux/src/load_policy.c ++++ b/libselinux/src/load_policy.c +@@ -369,7 +369,7 @@ int selinux_init_load_policy(int *enforce) + * Check for the existence of SELinux via selinuxfs, and + * mount it if present for use in the calls below. + */ +- char *mntpoint = NULL; ++ const char *mntpoint = NULL; + if (mount(SELINUXFS, SELINUXMNT, SELINUXFS, 0, 0) == 0 || errno == EBUSY) { + mntpoint = SELINUXMNT; + } else { diff --git a/libselinux/src/matchpathcon.c b/libselinux/src/matchpathcon.c -index 48f7a11..c625f55 100644 +index 48f7a11..ca87bd2 100644 --- a/libselinux/src/matchpathcon.c +++ b/libselinux/src/matchpathcon.c @@ -2,6 +2,7 @@ @@ -243,7 +375,16 @@ index 48f7a11..c625f55 100644 va_end(ap); } -@@ -304,7 +305,7 @@ static void matchpathcon_init_once(void) +@@ -292,6 +293,8 @@ static void matchpathcon_thread_destructor(void __attribute__((unused)) *ptr) + matchpathcon_fini(); + } + ++void __attribute__((destructor)) matchpathcon_lib_destructor(void); ++ + void __attribute__((destructor)) matchpathcon_lib_destructor(void) + { + if (destructor_key_initialized) +@@ -304,7 +307,7 @@ static void matchpathcon_init_once(void) destructor_key_initialized = 1; } @@ -252,7 +393,7 @@ index 48f7a11..c625f55 100644 { if (!mycanoncon) mycanoncon = default_canoncon; -@@ -312,15 +313,22 @@ int matchpathcon_init_prefix(const char *path, const char *subset) +@@ -312,15 +315,22 @@ int matchpathcon_init_prefix(const char *path, const char *subset) __selinux_once(once, matchpathcon_init_once); __selinux_setspecific(destructor_key, (void *)1); @@ -267,16 +408,227 @@ index 48f7a11..c625f55 100644 return hnd ? 0 : -1; } +hidden_def(matchpathcon_init_prefixes) - ++ +int matchpathcon_init_prefix(const char *path, const char *prefix) +{ + const char *prefixes[2] = { prefix, NULL }; -+ + + return matchpathcon_init_prefixes(path, prefixes); +} hidden_def(matchpathcon_init_prefix) int matchpathcon_init(const char *path) +diff --git a/libselinux/src/selinux_config.c b/libselinux/src/selinux_config.c +index f4c33df..f42cb7c 100644 +--- a/libselinux/src/selinux_config.c ++++ b/libselinux/src/selinux_config.c +@@ -246,172 +246,172 @@ static const char *get_path(int idx) + return file_paths[idx]; + } + +-const char *selinux_default_type_path() ++const char *selinux_default_type_path(void) + { + return get_path(DEFAULT_TYPE); + } + + hidden_def(selinux_default_type_path) + +-const char *selinux_policy_root() ++const char *selinux_policy_root(void) + { + __selinux_once(once, init_selinux_config); + return selinux_policyroot; + } + +-const char *selinux_path() ++const char *selinux_path(void) + { + return selinux_rootpath; + } + + hidden_def(selinux_path) + +-const char *selinux_default_context_path() ++const char *selinux_default_context_path(void) + { + return get_path(DEFAULT_CONTEXTS); + } + + hidden_def(selinux_default_context_path) + +-const char *selinux_securetty_types_path() ++const char *selinux_securetty_types_path(void) + { + return get_path(SECURETTY_TYPES); + } + + hidden_def(selinux_securetty_types_path) + +-const char *selinux_failsafe_context_path() ++const char *selinux_failsafe_context_path(void) + { + return get_path(FAILSAFE_CONTEXT); + } + + hidden_def(selinux_failsafe_context_path) + +-const char *selinux_removable_context_path() ++const char *selinux_removable_context_path(void) + { + return get_path(REMOVABLE_CONTEXT); + } + + hidden_def(selinux_removable_context_path) + +-const char *selinux_binary_policy_path() ++const char *selinux_binary_policy_path(void) + { + return get_path(BINPOLICY); + } + + hidden_def(selinux_binary_policy_path) + +-const char *selinux_file_context_path() ++const char *selinux_file_context_path(void) + { + return get_path(FILE_CONTEXTS); + } + + hidden_def(selinux_file_context_path) + +-const char *selinux_homedir_context_path() ++const char *selinux_homedir_context_path(void) + { + return get_path(HOMEDIR_CONTEXTS); + } + + hidden_def(selinux_homedir_context_path) + +-const char *selinux_media_context_path() ++const char *selinux_media_context_path(void) + { + return get_path(MEDIA_CONTEXTS); + } + + hidden_def(selinux_media_context_path) + +-const char *selinux_customizable_types_path() ++const char *selinux_customizable_types_path(void) + { + return get_path(CUSTOMIZABLE_TYPES); + } + + hidden_def(selinux_customizable_types_path) + +-const char *selinux_contexts_path() ++const char *selinux_contexts_path(void) + { + return get_path(CONTEXTS_DIR); + } + +-const char *selinux_user_contexts_path() ++const char *selinux_user_contexts_path(void) + { + return get_path(USER_CONTEXTS); + } + + hidden_def(selinux_user_contexts_path) + +-const char *selinux_booleans_path() ++const char *selinux_booleans_path(void) + { + return get_path(BOOLEANS); + } + + hidden_def(selinux_booleans_path) + +-const char *selinux_users_path() ++const char *selinux_users_path(void) + { + return get_path(USERS_DIR); + } + + hidden_def(selinux_users_path) + +-const char *selinux_usersconf_path() ++const char *selinux_usersconf_path(void) + { + return get_path(SEUSERS); + } + + hidden_def(selinux_usersconf_path) + +-const char *selinux_translations_path() ++const char *selinux_translations_path(void) + { + return get_path(TRANSLATIONS); + } + + hidden_def(selinux_translations_path) + +-const char *selinux_colors_path() ++const char *selinux_colors_path(void) + { + return get_path(COLORS); + } + + hidden_def(selinux_colors_path) + +-const char *selinux_netfilter_context_path() ++const char *selinux_netfilter_context_path(void) + { + return get_path(NETFILTER_CONTEXTS); + } + + hidden_def(selinux_netfilter_context_path) + +-const char *selinux_file_context_homedir_path() ++const char *selinux_file_context_homedir_path(void) + { + return get_path(FILE_CONTEXTS_HOMEDIR); + } + + hidden_def(selinux_file_context_homedir_path) + +-const char *selinux_file_context_local_path() ++const char *selinux_file_context_local_path(void) + { + return get_path(FILE_CONTEXTS_LOCAL); + } + + hidden_def(selinux_file_context_local_path) + +-const char *selinux_x_context_path() ++const char *selinux_x_context_path(void) + { + return get_path(X_CONTEXTS); + } + + hidden_def(selinux_x_context_path) + +-const char *selinux_virtual_domain_context_path() ++const char *selinux_virtual_domain_context_path(void) + { + return get_path(VIRTUAL_DOMAIN); + } + + hidden_def(selinux_virtual_domain_context_path) + +-const char *selinux_virtual_image_context_path() ++const char *selinux_virtual_image_context_path(void) + { + return get_path(VIRTUAL_IMAGE); + } +@@ -430,7 +430,7 @@ const char * selinux_file_context_subs_dist_path(void) { + + hidden_def(selinux_file_context_subs_dist_path) + +-const char *selinux_sepgsql_context_path() ++const char *selinux_sepgsql_context_path(void) + { + return get_path(SEPGSQL_CONTEXTS); + } diff --git a/libselinux/src/selinux_internal.h b/libselinux/src/selinux_internal.h index 710396a..9a3fc14 100644 --- a/libselinux/src/selinux_internal.h @@ -289,3 +641,231 @@ index 710396a..9a3fc14 100644 hidden_proto(matchpathcon_init_prefix) hidden_proto(selinux_users_path) hidden_proto(selinux_usersconf_path); +diff --git a/libselinux/src/setrans_client.c b/libselinux/src/setrans_client.c +index e074142..9432f49 100644 +--- a/libselinux/src/setrans_client.c ++++ b/libselinux/src/setrans_client.c +@@ -253,6 +253,8 @@ static void setrans_thread_destructor(void __attribute__((unused)) *unused) + free(prev_r2c_raw); + } + ++void __attribute__((destructor)) setrans_lib_destructor(void); ++ + void __attribute__((destructor)) setrans_lib_destructor(void) + { + if (destructor_key_initialized) +diff --git a/libselinux/src/seusers.c b/libselinux/src/seusers.c +index b653cad..5cdf6c0 100644 +--- a/libselinux/src/seusers.c ++++ b/libselinux/src/seusers.c +@@ -269,9 +269,10 @@ int getseuser(const char *username, const char *service, + size_t lineno = 0; + char *rec = NULL; + char *path=NULL; ++ FILE *fp = NULL; + if (asprintf(&path,"%s/logins/%s", selinux_policy_root(), username) < 0) + goto err; +- FILE *fp = fopen(path, "r"); ++ fp = fopen(path, "r"); + free(path); + if (fp == NULL) goto err; + __fsetlocking(fp, FSETLOCKING_BYCALLER); +diff --git a/libselinux/src/stringrep.c b/libselinux/src/stringrep.c +index f0167e7..176ac34 100644 +--- a/libselinux/src/stringrep.c ++++ b/libselinux/src/stringrep.c +@@ -305,28 +305,6 @@ err1: + return NULL; + } + +-void flush_class_cache(void) +-{ +- struct discover_class_node *cur = discover_class_cache, *prev = NULL; +- size_t i; +- +- while (cur != NULL) { +- free(cur->name); +- +- for (i=0 ; iperms[i]); +- +- free(cur->perms); +- +- prev = cur; +- cur = cur->next; +- +- free(prev); +- } +- +- discover_class_cache = NULL; +-} +- + static security_class_t string_to_security_class_compat(const char *s) + { + unsigned int val; +diff --git a/libselinux/utils/Makefile b/libselinux/utils/Makefile +index 6f5aa52..4b2f348 100644 +--- a/libselinux/utils/Makefile ++++ b/libselinux/utils/Makefile +@@ -4,7 +4,25 @@ LIBDIR ?= $(PREFIX)/lib + BINDIR ?= $(PREFIX)/sbin + _BINDIR ?= $(DESTDIR)/sbin + +-CFLAGS ?= -Werror -Wall -W ++MAX_STACK_SIZE=8192 ++CFLAGS ?= -Wall -W -Wundef -Wformat-y2k -Wformat-security -Winit-self -Wmissing-include-dirs \ ++ -Wunused -Wunknown-pragmas -Wstrict-aliasing -Wshadow -Wpointer-arith \ ++ -Wbad-function-cast -Wcast-align -Wwrite-strings -Wlogical-op -Waggregate-return \ ++ -Wstrict-prototypes -Wold-style-definition -Wmissing-prototypes \ ++ -Wmissing-declarations -Wmissing-noreturn -Wmissing-format-attribute \ ++ -Wredundant-decls -Wnested-externs -Winline -Winvalid-pch -Wvolatile-register-var \ ++ -Wdisabled-optimization -Wbuiltin-macro-redefined -Wmudflap -Wpacked-bitfield-compat \ ++ -Wsync-nand -Wattributes -Wcoverage-mismatch -Wmultichar -Wcpp \ ++ -Wdeprecated-declarations -Wdiv-by-zero -Wdouble-promotion -Wendif-labels -Wextra \ ++ -Wformat-contains-nul -Wformat-extra-args -Wformat-zero-length -Wformat=2 -Wmultichar \ ++ -Wnormalized=nfc -Woverflow -Wpointer-to-int-cast -Wpragmas -Wsuggest-attribute=const \ ++ -Wsuggest-attribute=noreturn -Wsuggest-attribute=pure -Wtrampolines \ ++ -Wno-missing-field-initializers -Wno-sign-compare -Wjump-misses-init \ ++ -Wno-format-nonliteral -Wframe-larger-than=$(MAX_STACK_SIZE) -Wp,-D_FORTIFY_SOURCE=2 \ ++ -fstack-protector-all --param=ssp-buffer-size=4 -fexceptions \ ++ -fasynchronous-unwind-tables -fdiagnostics-show-option -funit-at-a-time \ ++ -fipa-pure-const -Wno-suggest-attribute=pure -Wno-suggest-attribute=const \ ++ -Werror -Wno-aggregate-return -Wno-redundant-decls + override CFLAGS += -I../include -D_GNU_SOURCE $(EMFLAGS) + LDLIBS += -L../src -lselinux -L$(LIBDIR) + +diff --git a/libselinux/utils/avcstat.c b/libselinux/utils/avcstat.c +index 772118a..7239ef2 100644 +--- a/libselinux/utils/avcstat.c ++++ b/libselinux/utils/avcstat.c +@@ -43,7 +43,7 @@ static char buf[DEF_BUF_SIZE]; + /* selinuxfs mount point */ + extern char *selinux_mnt; + +-static void die(const char *msg, ...) ++static __attribute__((__format__(printf,1,2))) void die(const char *msg, ...) + { + va_list args; + +@@ -118,7 +118,7 @@ int main(int argc, char **argv) + exit(0); + default: + usage(); +- die("unrecognized parameter", i); ++ die("unrecognized parameter '%c'", i); + } + } + +diff --git a/libselinux/utils/getconlist.c b/libselinux/utils/getconlist.c +index 4f473e4..94c9bff 100644 +--- a/libselinux/utils/getconlist.c ++++ b/libselinux/utils/getconlist.c +@@ -9,7 +9,7 @@ + #include + #include + +-void usage(char *name, char *detail, int rc) ++static void usage(const char *name, const char *detail, int rc) + { + fprintf(stderr, "usage: %s [-l level] user [context]\n", name); + if (detail) +diff --git a/libselinux/utils/getdefaultcon.c b/libselinux/utils/getdefaultcon.c +index e6eb98b..049e75c 100644 +--- a/libselinux/utils/getdefaultcon.c ++++ b/libselinux/utils/getdefaultcon.c +@@ -9,7 +9,7 @@ + #include + #include + +-void usage(char *name, char *detail, int rc) ++static void usage(const char *name, const char *detail, int rc) + { + fprintf(stderr, "usage: %s [-l level] user fromcon\n", name); + if (detail) +diff --git a/libselinux/utils/getsebool.c b/libselinux/utils/getsebool.c +index cab2bb9..3a90449 100644 +--- a/libselinux/utils/getsebool.c ++++ b/libselinux/utils/getsebool.c +@@ -6,7 +6,7 @@ + #include + #include + +-void usage(const char *progname) ++static void usage(const char *progname) + { + fprintf(stderr, "usage: %s -a or %s boolean...\n", progname, progname); + exit(1); +diff --git a/libselinux/utils/matchpathcon.c b/libselinux/utils/matchpathcon.c +index 5f0a4c2..b1adadd 100644 +--- a/libselinux/utils/matchpathcon.c ++++ b/libselinux/utils/matchpathcon.c +@@ -13,7 +13,7 @@ + #include + + +-void usage(const char *progname) ++static void usage(const char *progname) + { + fprintf(stderr, + "usage: %s [-N] [-n] [-f file_contexts] [-p prefix] [-Vq] path...\n", +@@ -21,7 +21,7 @@ void usage(const char *progname) + exit(1); + } + +-int printmatchpathcon(char *path, int header, int mode) ++static int printmatchpathcon(const char *path, int header, int mode) + { + char *buf; + int rc = matchpathcon(path, mode, &buf); +diff --git a/libselinux/utils/selinux_check_securetty_context.c b/libselinux/utils/selinux_check_securetty_context.c +index 95bfb7f..b158eb3 100644 +--- a/libselinux/utils/selinux_check_securetty_context.c ++++ b/libselinux/utils/selinux_check_securetty_context.c +@@ -9,7 +9,7 @@ + #include + #include + +-void usage(const char *progname) ++static void usage(const char *progname) + { + fprintf(stderr, "usage: %s tty_context...\n", progname); + exit(1); +diff --git a/libselinux/utils/selinuxexeccon.c b/libselinux/utils/selinuxexeccon.c +index c55fde9..476f564 100644 +--- a/libselinux/utils/selinuxexeccon.c ++++ b/libselinux/utils/selinuxexeccon.c +@@ -9,7 +9,7 @@ + #include + #include + +-void usage(char *name, char *detail, int rc) ++static void usage(const char *name, const char *detail, int rc) + { + fprintf(stderr, "usage: %s command [ fromcon ]\n", name); + if (detail) +diff --git a/libselinux/utils/setenforce.c b/libselinux/utils/setenforce.c +index e45b804..df58597 100644 +--- a/libselinux/utils/setenforce.c ++++ b/libselinux/utils/setenforce.c +@@ -6,7 +6,7 @@ + #include + #include + +-void usage(const char *progname) ++static void usage(const char *progname) + { + fprintf(stderr, "usage: %s [ Enforcing | Permissive | 1 | 0 ]\n", + progname); +diff --git a/libselinux/utils/togglesebool.c b/libselinux/utils/togglesebool.c +index 680ed8d..ad0d2a2 100644 +--- a/libselinux/utils/togglesebool.c ++++ b/libselinux/utils/togglesebool.c +@@ -10,7 +10,7 @@ + + /* Attempt to rollback the transaction. No need to check error + codes since this is rolling back something that blew up. */ +-void rollback(int argc, char **argv) ++static void rollback(int argc, char **argv) + { + int i; +