mirrors/apparmor
1
0
Fork 0
mirror of https://gitlab.com/apparmor/apparmor.git synced 2025-03-04 08:24:42 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix compiler warnings about format specifiers with DEBUG set

Browse source 
As for %s when the pointer is null: even if gcc prints (null) this is still undefined behavior, so we should do this explicitly

Signed-off-by: Ryan Lee <ryan.lee@canonical.com>
This commit is contained in:
Ryan Lee 2024-10-16 12:26:54 -07:00
parent 6d20a10606
commit 776c56bd7e
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
parser/parser_interface.c
View file

@ -245,7 +245,7 @@ static inline void sd_write_uint64(std::ostringstream &buf, u64 b)
static inline void sd_write_name(std::ostringstream &buf, const char *name) static inline void sd_write_name(std::ostringstream &buf, const char *name)
{ {
PDEBUG("Writing name '%s'\n", name); PDEBUG("Writing name '%s'\n", name ? name : "(null)");
if (name) { if (name) {
sd_write8(buf, SD_NAME); sd_write8(buf, SD_NAME);
sd_write16(buf, strlen(name) + 1); sd_write16(buf, strlen(name) + 1);

2
parser/parser_yacc.y
View file

@ -704,7 +704,7 @@ rules: rules opt_prefix block
if (($2).priority != 0) { if (($2).priority != 0) {
yyerror(_("priority is not allowed on rule blocks")); yyerror(_("priority is not allowed on rule blocks"));
} }
PDEBUG("matched: %s%s%sblock\n", PDEBUG("matched: %s%s%s%sblock\n",
$2.audit == AUDIT_FORCE ? "audit " : "", $2.audit == AUDIT_FORCE ? "audit " : "",
$2.rule_mode == RULE_DENY ? "deny " : "", $2.rule_mode == RULE_DENY ? "deny " : "",
$2.rule_mode == RULE_PROMPT ? "prompt " : "", $2.rule_mode == RULE_PROMPT ? "prompt " : "",