apparmorfs: Use seq_putc() in two functions
authorMarkus Elfring <elfring@users.sourceforge.net>
Sun, 7 May 2017 11:50:28 +0000 (13:50 +0200)
committerJohn Johansen <john.johansen@canonical.com>
Thu, 8 Jun 2017 18:21:02 +0000 (11:21 -0700)
Two single characters (line breaks) should be put into a sequence.
Thus use the corresponding function "seq_putc".

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>
security/apparmor/apparmorfs.c

index b4d83e0..41e427a 100644 (file)
@@ -494,7 +494,7 @@ static int aa_fs_seq_hash_show(struct seq_file *seq, void *v)
        if (profile->hash) {
                for (i = 0; i < size; i++)
                        seq_printf(seq, "%.2x", profile->hash[i]);
-               seq_puts(seq, "\n");
+               seq_putc(seq, '\n');
        }
        aa_put_profile(profile);
 
@@ -602,7 +602,7 @@ static int aa_fs_seq_raw_hash_show(struct seq_file *seq, void *v)
        if (profile->rawdata->hash) {
                for (i = 0; i < size; i++)
                        seq_printf(seq, "%.2x", profile->rawdata->hash[i]);
-               seq_puts(seq, "\n");
+               seq_putc(seq, '\n');
        }
        aa_put_profile(profile);