[PATCH] ima: Replace two seq_printf() calls by seq_puts() in ima_show_template_data_ascii()
From: Markus Elfring <hidden>
Date: 2019-07-02 19:00:59
Also in:
kernel-janitors, linux-security-module, lkml
Subsystem:
extended verification module (evm), integrity measurement architecture (ima), security subsystem, the rest · Maintainers:
Mimi Zohar, Roberto Sassu, Dmitry Kasatkin, Paul Moore, James Morris, "Serge E. Hallyn", Linus Torvalds
From: Markus Elfring <redacted> Date: Tue, 2 Jul 2019 20:52:21 +0200 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 <redacted> --- security/integrity/ima/ima_template_lib.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/security/integrity/ima/ima_template_lib.c b/security/integrity/ima/ima_template_lib.c
index 9fe0ef7f91e2..05636e9b19b1 100644
--- a/security/integrity/ima/ima_template_lib.c
+++ b/security/integrity/ima/ima_template_lib.c@@ -74,7 +74,7 @@ static void ima_show_template_data_ascii(struct seq_file *m, case DATA_FMT_DIGEST_WITH_ALGO: buf_ptr = strnchr(field_data->data, buflen, ':'); if (buf_ptr != field_data->data) - seq_printf(m, "%s", field_data->data); + seq_puts(m, field_data->data); /* skip ':' and '\0' */ buf_ptr += 2;
@@ -87,7 +87,7 @@ static void ima_show_template_data_ascii(struct seq_file *m, ima_print_digest(m, buf_ptr, buflen); break; case DATA_FMT_STRING: - seq_printf(m, "%s", buf_ptr); + seq_puts(m, buf_ptr); break; default: break; --
2.22.0