perf kmem: Fix compiler warning about may be accessing uninitialized variable
authorArnaldo Carvalho de Melo <acme@redhat.com>
Mon, 11 May 2015 14:41:17 +0000 (11:41 -0300)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Tue, 12 May 2015 12:59:47 +0000 (09:59 -0300)
The last argument to strtok_r doesn't need to be initialized, its just a
placeholder to make this routine reentrant, but gcc doesn't know about
that and complains, breaking the build, fix it by setting it to NULL.

Fixes: 0e11115644b3 ("perf kmem: Print gfp flags in human readable string")
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Borislav Petkov <bp@suse.de>
Cc: David Ahern <dsahern@gmail.com>
Cc: Don Zickus <dzickus@redhat.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Stephane Eranian <eranian@google.com>
Link: http://lkml.kernel.org/n/tip-iyyvkbnkrd9g19f6ta9zfkem@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/builtin-kmem.c

index e0173c7f17b1c1a9ef4b6b98350161272521bbaa..254614b10c4a3bdb33c01f2c0f87d6da22d0315e 100644 (file)
@@ -713,7 +713,7 @@ static int parse_gfp_flags(struct perf_evsel *evsel, struct perf_sample *sample,
                .size = sample->raw_size,
        };
        struct trace_seq seq;
-       char *str, *pos;
+       char *str, *pos = NULL;
 
        if (nr_gfps) {
                struct gfp_flag key = {