On Tue, Sep 24, 2019 at 07:08:56AM -0700, Andi Kleen wrote:
> > >   expr__ctx_init(&pctx);
> > > + /* Must be first id entry */
> > > + expr__add_id(&pctx, name, avg);
> > 
> > hum, shouldn't u instead use strdup(name) instead of name?
> 
> The cleanup loop later skips freeing the first entry.

aaah, nice ;-)

Acked-by: Jiri Olsa <jo...@kernel.org>

thanks,
jirka

Reply via email to