[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [PATCH] dump.c: Fix memory leak issue in cleanup proc
Re: [Qemu-trivial] [PATCH] dump.c: Fix memory leak issue in cleanup processing for dump_init()
Wed, 13 Aug 2014 06:19:40 +0800
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130625 Thunderbird/17.0.7
On 08/12/2014 11:43 PM, Laszlo Ersek wrote:
> On 08/03/14 17:28, Chen Gang wrote:
>> In dump_init(), when failure occurs, need notice about 'fd' and memory
>> mapping. So call dump_cleanup() for it (need let all initializations at
>> Also simplify dump_cleanup(): remove redundant 'ret' and redundant 'fd'
>> Signed-off-by: Chen Gang <address@hidden>
> The patch is not trivial at all, because the lifecycles of the affected
> resources are not trivial.
For me, next, I shall split it into 2 pieces, one for resource leak, and
the other for improving current code, which may let reviewers easier
(maybe 2 patches belong to 2 different maintainers).
> The commit message is an abomination. If you want to contribute to open
> source, please learn proper English, and make a *serious* effort to
> document your changes. Don't expect earlier contributors to have any
> working knowledge about a file they have *maybe* touched several months
> ago. People have to swap out a whole load of crap from their minds, and
> swap in the old crap, to understand your patch. Help them by writing
> good commit messages.
OK, thanks, And I am just trying to be improving.
- always communicate with open source (qemu/kvm/kernel/binutils/gcc) in
English (it must be).
- I read/listen to Holy Bible in English every day: morning and evening
in subway (I spend 2 hours from home to office, so 4 hours total),
so at least, I spend about 1-1.5 hours for reading/listening per day.
> That said, no matter how annoying this submission is, my conscience
> didn't allow me to let it go ignored, so I spent the time and made an
> effort to track the lifetimes of "s->fd" and "s->list" in, and around,
OK, thanks, next, you can notify me for it, and I shall try to give some
related test (or we can test together for it), although it is not quite
easy for me.
> The patch seems correct. That's your only excuse for the loss of gray
> matter I've suffered while parsing your commit message.
> Reviewed-by: Laszlo Ersek <address@hidden>
OK, thank you very much.
Open share and attitude like air water and life which God blessed
Re: [Qemu-trivial] [PATCH] dump.c: Fix memory leak issue in cleanup processing for dump_init(), Laszlo Ersek, 2014/08/12
- [Qemu-trivial] Cc'ing emails [was: [PATCH] dump.c: Fix memory leak issue in cleanup processing for dump_init()], (continued)
Re: [Qemu-trivial] [PATCH] dump.c: Fix memory leak issue in cleanup processing for dump_init(), Luiz Capitulino, 2014/08/14
- Re: [Qemu-trivial] [PATCH] dump.c: Fix memory leak issue in cleanup processing for dump_init(),
Chen Gang <=