[libvirt] [PATCH v4 2/2] daemon: Remove unnecessary goto error

2017-04-23 Thread Wang King
Free the dst is unnecessary if the VIR_STRDUP fails, and therefore we need to remove the error label. --- daemon/remote.c | 10 ++ 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/daemon/remote.c b/daemon/remote.c index a8c21fd..2709c3c 100644 --- a/daemon/remote.c +++ b/daemo

Re: [libvirt] [PATCH v4 2/2] daemon: Remove unnecessary goto error

2017-04-25 Thread Martin Kletzander
On Mon, Apr 24, 2017 at 12:05:25PM +0800, Wang King wrote: Free the dst is unnecessary if the VIR_STRDUP fails, and therefore we need to remove the error label. --- daemon/remote.c | 10 ++ 1 file changed, 2 insertions(+), 8 deletions(-) Nice cleanup; I'll tweak the commit message a tin