Re: [Xen-devel] [PATCH 1/2] libxl: Set rc on failure of usbdev_busaddr_to_busid

2016-04-08 Thread Ian Jackson
Chun Yan Liu writes ("Re: [PATCH 1/2] libxl: Set rc on failure of usbdev_busaddr_to_busid"): > On 4/8/2016 at 01:04 AM, in message > <22278.37677.975595.101...@mariner.uk.xensource.com>, Ian Jackson > wrote: > > Chun Yan Liu writes ("Re: [PATCH 1/2] libxl: Set rc on failure of > > usbdev_busad

Re: [Xen-devel] [PATCH 1/2] libxl: Set rc on failure of usbdev_busaddr_to_busid

2016-04-07 Thread Chun Yan Liu
>>> On 4/8/2016 at 01:04 AM, in message <22278.37677.975595.101...@mariner.uk.xensource.com>, Ian Jackson wrote: > Chun Yan Liu writes ("Re: [PATCH 1/2] libxl: Set rc on failure of > usbdev_busaddr_to_busid"): > > Thanks, Ian! > > Should I take that as a Reviewed-by ? Ah, yes. Acked. :-

Re: [Xen-devel] [PATCH 1/2] libxl: Set rc on failure of usbdev_busaddr_to_busid

2016-04-07 Thread Ian Jackson
Chun Yan Liu writes ("Re: [PATCH 1/2] libxl: Set rc on failure of usbdev_busaddr_to_busid"): > Thanks, Ian! Should I take that as a Reviewed-by ? Thanks, Ian. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH 1/2] libxl: Set rc on failure of usbdev_busaddr_to_busid

2016-04-07 Thread Chun Yan Liu
Thanks, Ian! >>> On 4/4/2016 at 11:09 PM, in message <1459782600-16073-1-git-send-email-ian.jack...@eu.citrix.com>, Ian Jackson wrote: > We must set rc before using `goto out'. > > Bug introduced in bf7628f0 "libxl: add pvusb API". > > CID: 1358113 > Signed-off-by: Ian Jackson > CC: cov

[Xen-devel] [PATCH 1/2] libxl: Set rc on failure of usbdev_busaddr_to_busid

2016-04-04 Thread Ian Jackson
We must set rc before using `goto out'. Bug introduced in bf7628f0 "libxl: add pvusb API". CID: 1358113 Signed-off-by: Ian Jackson CC: cover...@xenproject.org CC: Simon Cao CC: George Dunlap CC: Chunyan Liu --- tools/libxl/libxl_pvusb.c |1 + 1 file changed, 1 insertion(+) diff --git a/