From 7a84b96e7abe6251e217bd9c69a83230bda23bc5 Mon Sep 17 00:00:00 2001 From: Ken Dreyer Date: Mon, 26 Jan 2015 13:19:15 -0700 Subject: [PATCH] device-drivers/cpufreq: use safe format string arg Prior to this commit, the cpufreq_boost test failed to build if -Werror=format-security was used. The compiler could not examine the contents of the cdrv array. Adjust the SAFE_FILE_PRINTF call so that the format string is a string literal. Fedora recently enabled this compiler option in their RPM builds, so this fixes the build on Fedora 21. Signed-off-by: Ken Dreyer --- testcases/kernel/device-drivers/cpufreq/cpufreq_boost.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/testcases/kernel/device-drivers/cpufreq/cpufreq_boost.c b/testcases/kernel/device-drivers/cpufreq/cpufreq_boost.c index d1ba2cf..872b120 100644 --- a/testcases/kernel/device-drivers/cpufreq/cpufreq_boost.c +++ b/testcases/kernel/device-drivers/cpufreq/cpufreq_boost.c @@ -152,12 +152,12 @@ static void test_run(void) /* Enable boost */ if (boost_value == cdrv[id].off) - SAFE_FILE_PRINTF(cleanup, cdrv[id].file, cdrv[id].on_str); + SAFE_FILE_PRINTF(cleanup, cdrv[id].file, "%s", cdrv[id].on_str); tst_resm(TINFO, "load CPU0 with boost enabled"); boost_time = load_cpu(max_freq_khz); /* Disable boost */ - SAFE_FILE_PRINTF(cleanup, cdrv[id].file, cdrv[id].off_str); + SAFE_FILE_PRINTF(cleanup, cdrv[id].file, "%s", cdrv[id].off_str); tst_resm(TINFO, "load CPU0 with boost disabled"); boost_off_time = load_cpu(max_freq_khz); -- 1.9.3