1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
|
From 7a84b96e7abe6251e217bd9c69a83230bda23bc5 Mon Sep 17 00:00:00 2001
From: Ken Dreyer <kdreyer@redhat.com>
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 <kdreyer@redhat.com>
---
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
|