[libvirt] [Klaus at Ethgen.de: [Pkg-libvirt-maintainers] Bug#579208: USB Class/Number/... is base 16, not base 10 (Failed to convert 'ff' to unsigned int)]

Dave Allan dallan at redhat.com
Mon Apr 26 17:46:37 UTC 2010


On Mon, Apr 26, 2010 at 11:26:51AM -0600, Eric Blake wrote:
> On 04/26/2010 11:11 AM, Guido Günther wrote:
> > Hi,
> > this patch from Klau Ethgen looks right to me. O.k. to apply? I'll
> > remove the comment with this patch then.
> 
> Is there documentation somewhere that mentions what base the udev
> strings are encoded in?  At any rate, I agree with you that given the
> XXX comment, that this patch looks right.

That's my comment in the code.  I was uncertain at the time I wrote it
what the correct base was.  If you're certain that the correct base is
16, then ACK, and please do remove the comment.

Dave


> >      if (udevGetUintSysfsAttr(device,
> >                               "bInterfaceNumber",
> >                               &data->usb_if.number,
> > -                             10) == PROPERTY_ERROR) {
> > +                             16) == PROPERTY_ERROR) {
> 
> ACK.
> 
> -- 
> Eric Blake   eblake at redhat.com    +1-801-349-2682
> Libvirt virtualization library http://libvirt.org
> 



> --
> libvir-list mailing list
> libvir-list at redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list




More information about the libvir-list mailing list