[prev in list] [next in list] [prev in thread] [next in thread] 

List:       libusb-devel
Subject:    Re: [libusb] [PATCH] Fix libusb_init for ENABLE_DEBUG_LOGGING
From:       John Sheu <sheu () google ! com>
Date:       2014-05-27 18:32:58
Message-ID: CAErgknD42UmwhWYFyNQ2H0jqCTirWNnisRQPGndRGV8zogeoQA () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


On Sat, May 24, 2014 at 3:23 AM, Hans de Goede <hdegoede@redhat.com> wrote:

> On 05/24/2014 02:18 AM, John Sheu wrote:
> > With ENABLE_DEBUG_LOGGING, usbi_dbg calls usbi_log_v with a NULL
> > context, which will fall back to the default context.  This will
> > segfault in the case that the default context does not exist yet --
> > which is always, since libusb_init calls usbi_dbg before it sets up the
> > default context.
> >
> > Signed-off-by: John Sheu <sheu@google.com>
>
> Thanks for the patch, but it seems that you're using an older version of
> libusb, this has been fixed for a while now.
>
> Regards,
>
> Hans


Oh, right you are.  Sorry for the fuss.  Carry on!

-John Sheu

[Attachment #5 (text/html)]

<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sat, May 24, 2014 \
at 3:23 AM, Hans de Goede <span dir="ltr">&lt;<a href="mailto:hdegoede@redhat.com" \
target="_blank">hdegoede@redhat.com</a>&gt;</span> wrote:<br> <blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex"><div class="">On 05/24/2014 02:18 AM, John Sheu wrote:<br> \
&gt; With ENABLE_DEBUG_LOGGING, usbi_dbg calls usbi_log_v with a NULL<br> &gt; \
context, which will fall back to the default context.   This will<br> &gt; segfault \
in the case that the default context does not exist yet --<br> &gt; which is always, \
since libusb_init calls usbi_dbg before it sets up the<br> &gt; default context.<br>
&gt;<br>
&gt; Signed-off-by: John Sheu &lt;<a \
href="mailto:sheu@google.com">sheu@google.com</a>&gt;<br> <br>
</div>Thanks for the patch, but it seems that you&#39;re using an older version \
of<br> libusb, this has been fixed for a while now.<br>
<br>
Regards,<br>
<br>
Hans</blockquote><div><br></div><div>Oh, right you are.   Sorry for the fuss.   Carry \
on!</div><div><br></div><div>-John Sheu  </div></div></div></div>



------------------------------------------------------------------------------
The best possible search technologies are now affordable for all companies.
Download your FREE open source Enterprise Search Engine today!
Our experts will assist you in its installation for $59/mo, no commitment.
Test it for FREE on our Cloud platform anytime!
http://pubads.g.doubleclick.net/gampad/clk?id=145328191&iu=/4140/ostg.clktrk

_______________________________________________
libusb-devel mailing list
libusb-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/libusb-devel


[prev in list] [next in list] [prev in thread] [next in thread]