ftrace/scripts: Fix incorrect use of sprintf in recordmcount
authorColin Ian King <colin.king@canonical.com>
Wed, 30 Dec 2015 23:06:41 +0000 (23:06 +0000)
committerSteven Rostedt <rostedt@goodmis.org>
Mon, 4 Jan 2016 16:13:16 +0000 (11:13 -0500)
Fix build warning:

scripts/recordmcount.c:589:4: warning: format not a string
literal and no format arguments [-Wformat-security]
    sprintf("%s: failed\n", file);

Fixes: a50bd43935586 ("ftrace/scripts: Have recordmcount copy the object file")
Link: http://lkml.kernel.org/r/1451516801-16951-1-git-send-email-colin.king@canonical.com
Cc: Li Bin <huawei.libin@huawei.com>
Cc: Russell King <rmk+kernel@arm.linux.org.uk>
Cc: Will Deacon <will.deacon@arm.com>
Cc: stable@vger.kernel.org # 2.6.37+
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
scripts/recordmcount.c

index 301d70b0174f3b3ff3aaea7a721ffdd905a576f9..e167592793a70e8a1bac057d58ca7456e064db8e 100644 (file)
@@ -586,7 +586,7 @@ main(int argc, char *argv[])
                        do_file(file);
                        break;
                case SJ_FAIL:    /* error in do_file or below */
-                       sprintf("%s: failed\n", file);
+                       fprintf(stderr, "%s: failed\n", file);
                        ++n_error;
                        break;
                case SJ_SUCCEED:    /* premature success */