Le 18/02/2015 19:36, Jens Axboe a écrit :
http://git.kernel.dk/?p=fio.git;a=blob;f=iolog.c;h=99f8bc18d8694cca0c141c51d116aced1b4130f2;hb=HEAD#l863
In that function, if we do return 1 we do leak ic.buf, we shall free it
before the return
Yep, there's a potential leak or two there, should be fixed up.
http://git.kernel.dk/?p=fio.git;a=blob;f=lib/axmap.c;h=164300f254014b10ed6e04e3e06b7263aa917aac;hb=HEAD#l127
Here, we surely miss the free of the axmap. We did free its internal
structure but not axmap itself.
Yes, that should free 'axmap' too of course.
Hey Jens,
I don't know your workflow : do you accept pull request directly on github ?
At least, please find the two commits for that issues in my repo :
https://github.com/enovance/fio/tree/erwan/memory-leak
Cheers,
--
Erwan Velu
eNovance from Redhat
--
To unsubscribe from this list: send the line "unsubscribe fio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
