On Thu, May 22, 2014 at 11:58:36PM +0200, Rickard Strandqvist wrote: > Cleaning up inconsistent NULL checks. > There is otherwise a risk of a possible null pointer dereference. > > Was largely found by using a static code analysis program called cppcheck. > > Signed-off-by: Rickard Strandqvist <rickard_strandqv...@spectrumdigital.se> > --- > tools/perf/util/session.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/tools/perf/util/session.c b/tools/perf/util/session.c > index 55960f2..8defd80 100644 > --- a/tools/perf/util/session.c > +++ b/tools/perf/util/session.c > @@ -1625,13 +1625,15 @@ out_delete_map: > void perf_session__fprintf_info(struct perf_session *session, FILE *fp, > bool full) > { > - int fd = perf_data_file__fd(session->file); > + int fd; > struct stat st; > int ret; > > if (session == NULL || fp == NULL) > return; > > + fd = perf_data_file__fd(session->file); > + > ret = fstat(fd, &st); > if (ret == -1) > return; > -- > 1.7.10.4 > hi, I merged same patch just a week ago or so ;-)
perf session: Fix possible null pointer dereference in session.c commit c5765ece8a050836c6255e1276fc8e0e867078da Author: Masanari Iida <standby2...@gmail.com> Date: Thu May 15 02:13:38 2014 +0900 thanks, jirka -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/