Messages in this thread Patch in this message | | | From | Namhyung Kim <> | Subject | [PATCH] perf evsel: Fix format string in perf_evsel__open_strerror() | Date | Fri, 22 Mar 2013 11:54:11 +0900 |
| |
From: Namhyung Kim <namhyung.kim@lge.com>
In case of EPERM or EACCESS, there was a debious "%s" before actual string so that an user would see "You may not have permission to collect %sstats." instead of "You may not ... system-wide stats.".
Signed-off-by: Namhyung Kim <namhyung@kernel.org> --- tools/perf/util/evsel.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/perf/util/evsel.c b/tools/perf/util/evsel.c index 1adb824610f0..b190d57e926e 100644 --- a/tools/perf/util/evsel.c +++ b/tools/perf/util/evsel.c @@ -1495,7 +1495,7 @@ int perf_evsel__open_strerror(struct perf_evsel *evsel, switch (err) { case EPERM: case EACCES: - return scnprintf(msg, size, "%s", + return scnprintf(msg, size, "You may not have permission to collect %sstats.\n" "Consider tweaking /proc/sys/kernel/perf_event_paranoid:\n" " -1 - Not paranoid at all\n" -- 1.7.11.7
| |