diff options
| author | Markus Elfring <elfring@users.sourceforge.net> | 2019-07-02 20:27:32 +0200 |
|---|---|---|
| committer | John Johansen <john.johansen@canonical.com> | 2020-01-18 15:35:23 -0800 |
| commit | 278de07ef84a4356a3b166dc17751abc03abca67 (patch) | |
| tree | f4fd0552b4be6101c94fd657af2ad58240ac1767 | |
| parent | c79f46a282390e0f5b306007bf7b11a46d529538 (diff) | |
apparmor: Replace two seq_printf() calls by seq_puts() in aa_label_seq_xprint()
Two strings which did not contain a data format specification should be put
into a sequence. Thus use the corresponding function “seq_puts”.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: John Johansen <john.johansen@canonical.com>
| -rw-r--r-- | security/apparmor/label.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/security/apparmor/label.c b/security/apparmor/label.c index 470693239e64..bb34094421c4 100644 --- a/security/apparmor/label.c +++ b/security/apparmor/label.c @@ -1749,13 +1749,13 @@ void aa_label_seq_xprint(struct seq_file *f, struct aa_ns *ns, AA_DEBUG("label print error"); return; } - seq_printf(f, "%s", str); + seq_puts(f, str); kfree(str); } else if (display_mode(ns, label, flags)) seq_printf(f, "%s (%s)", label->hname, label_modename(ns, label, flags)); else - seq_printf(f, "%s", label->hname); + seq_puts(f, label->hname); } void aa_label_xprintk(struct aa_ns *ns, struct aa_label *label, int flags, |