[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [Qemu-devel] [PATCH] numa: fix qerror_report_err not
Re: [Qemu-trivial] [Qemu-devel] [PATCH] numa: fix qerror_report_err not free issue
Sat, 30 Aug 2014 05:28:56 +0000
> -----Original Message-----
> From: address@hidden
> [mailto:address@hidden On
> Behalf Of Michael Tokarev
> Sent: Saturday, August 30, 2014 1:26 PM
> To: Liuli (I)
> Cc: address@hidden; address@hidden
> Subject: Re: [Qemu-devel] [PATCH] numa: fix qerror_report_err not free issue
> 30.08.2014 07:36, john.liuli wrote:
> > From: Li Liu <address@hidden>
> > All qerror_report_err returned none NULL pointers need to
> > be freed, otherwise will cause memory leaking.
> > Although this place did not cause real memory leaking by exit,
> > obviously it's not correct to use qerror_report_err
> > without error_free it.
> I don't thing there's any good reason to free resources like
> this (freeing memory, closing files, etc) right before exit()
> (esp. in error path). The OS will do that for us in one go
> much faster.