Re: [Qemu-devel] [PATCH 1/5] Fix segmentation fault when qemu_signal_init fails

2018-09-19 Thread Markus Armbruster
Fam Zheng writes: > On Wed, 09/05 19:20, Fei Li wrote: >> >> >> On 09/05/2018 04:36 PM, Daniel P. Berrangé wrote: >> > On Wed, Sep 05, 2018 at 12:17:24PM +0800, Fei Li wrote: >> > > Thanks for the review! :) >> > > >> > > >> > > On 09/04/2018 07:26 PM, Daniel P. Berrangé wrote: >> > > > On Tu

Re: [Qemu-devel] [PATCH 1/5] Fix segmentation fault when qemu_signal_init fails

2018-09-05 Thread Fei Li
On 09/05/2018 10:38 PM, Fam Zheng wrote: On Wed, 09/05 19:20, Fei Li wrote: On 09/05/2018 04:36 PM, Daniel P. Berrangé wrote: On Wed, Sep 05, 2018 at 12:17:24PM +0800, Fei Li wrote: Thanks for the review! :) On 09/04/2018 07:26 PM, Daniel P. Berrangé wrote: On Tue, Sep 04, 2018 at 07:08

Re: [Qemu-devel] [PATCH 1/5] Fix segmentation fault when qemu_signal_init fails

2018-09-05 Thread Fam Zheng
On Wed, 09/05 19:20, Fei Li wrote: > > > On 09/05/2018 04:36 PM, Daniel P. Berrangé wrote: > > On Wed, Sep 05, 2018 at 12:17:24PM +0800, Fei Li wrote: > > > Thanks for the review! :) > > > > > > > > > On 09/04/2018 07:26 PM, Daniel P. Berrangé wrote: > > > > On Tue, Sep 04, 2018 at 07:08:18PM +

Re: [Qemu-devel] [PATCH 1/5] Fix segmentation fault when qemu_signal_init fails

2018-09-05 Thread Fei Li
On 09/05/2018 04:36 PM, Daniel P. Berrangé wrote: On Wed, Sep 05, 2018 at 12:17:24PM +0800, Fei Li wrote: Thanks for the review! :) On 09/04/2018 07:26 PM, Daniel P. Berrangé wrote: On Tue, Sep 04, 2018 at 07:08:18PM +0800, Fei Li wrote: ... snip ... free(info); re

Re: [Qemu-devel] [PATCH 1/5] Fix segmentation fault when qemu_signal_init fails

2018-09-05 Thread Daniel P . Berrangé
On Wed, Sep 05, 2018 at 12:17:24PM +0800, Fei Li wrote: > Thanks for the review! :) > > > On 09/04/2018 07:26 PM, Daniel P. Berrangé wrote: > > On Tue, Sep 04, 2018 at 07:08:18PM +0800, Fei Li wrote: > > > Currently, when qemu_signal_init() fails it only returns a non-zero > > > value but without

Re: [Qemu-devel] [PATCH 1/5] Fix segmentation fault when qemu_signal_init fails

2018-09-04 Thread Fei Li
Thanks for the review! :) On 09/04/2018 07:26 PM, Daniel P. Berrangé wrote: On Tue, Sep 04, 2018 at 07:08:18PM +0800, Fei Li wrote: Currently, when qemu_signal_init() fails it only returns a non-zero value but without propagating any Error. But its callers need a non-null err when runs error_r

Re: [Qemu-devel] [PATCH 1/5] Fix segmentation fault when qemu_signal_init fails

2018-09-04 Thread Daniel P . Berrangé
On Tue, Sep 04, 2018 at 07:08:18PM +0800, Fei Li wrote: > Currently, when qemu_signal_init() fails it only returns a non-zero > value but without propagating any Error. But its callers need a > non-null err when runs error_report_err(err), or else 0->msg occurs. > > To avoid such segmentation faul

[Qemu-devel] [PATCH 1/5] Fix segmentation fault when qemu_signal_init fails

2018-09-04 Thread Fei Li
Currently, when qemu_signal_init() fails it only returns a non-zero value but without propagating any Error. But its callers need a non-null err when runs error_report_err(err), or else 0->msg occurs. To avoid such segmentation fault, add a new Error parameter to make the call trace to propagate t