Re: [PATCH] tools: perf: util: session.c: Cleaning up inconsistent NULL checks

2014-05-23 Thread Rickard Strandqvist
Okay, I wrote a patch with many errors in relation to how you want your patches on 3 of May. Maybe from there. But good, then this fix is included :) Best regards Rickard Strandqvist 2014-05-23 9:05 GMT+02:00 Jiri Olsa : > On Thu, May 22, 2014 at 11:58:36PM +0200, Rickard Strandqvist wrote: >>

Re: [PATCH] tools: perf: util: session.c: Cleaning up inconsistent NULL checks

2014-05-23 Thread Jiri Olsa
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

Re: [PATCH] tools: perf: util: session.c: Cleaning up inconsistent NULL checks

2014-05-23 Thread Jiri Olsa
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

Re: [PATCH] tools: perf: util: session.c: Cleaning up inconsistent NULL checks

2014-05-23 Thread Rickard Strandqvist
Okay, I wrote a patch with many errors in relation to how you want your patches on 3 of May. Maybe from there. But good, then this fix is included :) Best regards Rickard Strandqvist 2014-05-23 9:05 GMT+02:00 Jiri Olsa jo...@redhat.com: On Thu, May 22, 2014 at 11:58:36PM +0200, Rickard

[PATCH] tools: perf: util: session.c: Cleaning up inconsistent NULL checks

2014-05-22 Thread Rickard Strandqvist
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 --- tools/perf/util/session.c |4 +++- 1 file changed, 3 insertions(+), 1

[PATCH] tools: perf: util: session.c: Cleaning up inconsistent NULL checks

2014-05-22 Thread Rickard Strandqvist
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 +++-