On 11/27/2015 12:28 PM, Fam Zheng wrote: > I think when write_dump_header() returns a failure, it does call > dump_cleanup(), in create_header{32,64}. > > But the changes of code in this patch look sane to me, and the new error > handling is definitely looking better.
Yes, you are right. I will modify the commit message to correct mentioning about memory leak, and keep the codes itself. Thanks. Peter > > Fam