106b816cb4
At the end of process_filter(), collapse_tree() was changed to update the parg parameter, but the reassignment after the call wasn't removed. What happens is that the "current_op" gets modified and freed and parg is assigned to the new allocated argument. But after the call to collapse_tree(), parg is assigned again to the just freed "current_op", and this causes the tool to crash. The current_op variable must also be assigned to NULL in case of error, otherwise it will cause it to be free()ed twice. Signed-off-by: Steven Rostedt <rostedt@goodmis.org> Acked-by: Namhyung Kim <namhyung@kernel.org> Cc: stable@vger.kernel.org # 3.14+ Fixes: 42d6194d133c ("tools lib traceevent: Refactor process_filter()") Link: http://lkml.kernel.org/r/20160511150936.678c18a1@gandalf.local.home Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com> |
||
|---|---|---|
| .. | ||
| .gitignore | ||
| Build | ||
| event-parse.c | ||
| event-parse.h | ||
| event-plugin.c | ||
| event-utils.h | ||
| kbuffer-parse.c | ||
| kbuffer.h | ||
| Makefile | ||
| parse-filter.c | ||
| parse-utils.c | ||
| plugin_cfg80211.c | ||
| plugin_function.c | ||
| plugin_hrtimer.c | ||
| plugin_jbd2.c | ||
| plugin_kmem.c | ||
| plugin_kvm.c | ||
| plugin_mac80211.c | ||
| plugin_sched_switch.c | ||
| plugin_scsi.c | ||
| plugin_xen.c | ||
| trace-seq.c | ||