Messages in this thread | | | Date | Mon, 30 Jan 2017 10:01:59 +0100 | From | Jiri Olsa <> | Subject | Re: [PATCH 1/2] perf tools: Use zfree() instead of free() in parse-events.c |
| |
On Mon, Jan 30, 2017 at 02:23:38PM +0900, Taeung Song wrote: > Currently there are several parts not checking NULL > after allocating with zalloc() or asigning NULL value > to a pointer variable after doing free(). > > So I fill in code checking NULL and > use zfree() instead of free().
can't see directly reasons for zfree usage, but it looks reasonable.. do you have any crash reports due to missing zfree?
thanks, jirka
| |