From d370cbfc85280220745f7ca2b9a3d27432f34abf Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Christian=20G=C3=B6ttsche?= Date: Mon, 29 Apr 2024 18:31:20 +0200 Subject: [PATCH] libselinux/man: add format attribute for set_matchpathcon_printf(3) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The parameter for set_matchpathcon_printf(3) is a function pointer that takes a format string with variable arguments. Add a GNU printf function attribute, if supported, to enable compilers checking for format string inconsistencies. Signed-off-by: Christian Göttsche Acked-by: James Carter --- libselinux/include/selinux/selinux.h | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/libselinux/include/selinux/selinux.h b/libselinux/include/selinux/selinux.h index a0948853..83ea871c 100644 --- a/libselinux/include/selinux/selinux.h +++ b/libselinux/include/selinux/selinux.h @@ -443,7 +443,11 @@ extern void selinux_flush_class_cache(void); /* Set the function used by matchpathcon_init when displaying errors about the file_contexts configuration. If not set, then this defaults to fprintf(stderr, fmt, ...). */ -extern void set_matchpathcon_printf(void (*f) (const char *fmt, ...)); +extern void set_matchpathcon_printf(void +#ifdef __GNUC__ + __attribute__ ((format(printf, 1, 2))) +#endif + (*f) (const char *fmt, ...)); /* Set the function used by matchpathcon_init when checking the validity of a context in the file contexts configuration. If not set,