Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1569490

Re: [PATCH 1/2] perf tools: Use zfree() instead of free() in parse-events.c

From Jiri Olsa <jolsa@redhat.com>
Newsgroups linux.kernel
Subject Re: [PATCH 1/2] perf tools: Use zfree() instead of free() in parse-events.c
Date 2017-01-30 10:20 +0100
Message-ID <t5ghY-2bl-9@gated-at.bofh.it> (permalink)
References <t5cHo-8r7-3@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw

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

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 1/2] perf tools: Use zfree() instead of free() in parse-events.c Taeung Song <treeze.taeung@gmail.com> - 2017-01-30 06:30 +0100
  Re: [PATCH 1/2] perf tools: Use zfree() instead of free() in  parse-events.c Jiri Olsa <jolsa@redhat.com> - 2017-01-30 10:20 +0100
    Re: [PATCH 1/2] perf tools: Use zfree() instead of free() in  parse-events.c Taeung Song <treeze.taeung@gmail.com> - 2017-01-30 11:50 +0100
      Re: [PATCH 1/2] perf tools: Use zfree() instead of free() in  parse-events.c Jiri Olsa <jolsa@redhat.com> - 2017-01-30 13:50 +0100

csiph-web