Re: [Libevent-users] assert in http.c

2012-01-20 Thread Nick Mathewson
On Wed, Jan 18, 2012 at 12:56 PM, Myk Taylor wrote: > Hi all, > > I've done a bit more debugging on this, and after reproducing a few > segfaults (different from the original assert), it seems like the problem is > related to the issues Zhuang Yuyao was having a year and a half ago: > > http://arc

Re: [Libevent-users] [PATCH] Force strict validation of HTTP version in response

2012-01-20 Thread Nick Mathewson
On Mon, Jan 16, 2012 at 5:51 PM, Catalin Patulea wrote: > Hi, > > I ran into this while evhttp was trying to fetch Shoutcast streams. > The server responds with 'ICY 200 OK', which libevent intends to > reject. But the sscanf return value isn't checked properly and the > behaviour is undefined (in

[Libevent-users] Re: [Libevent-users] Re: [Libevent-users] Security question [was: Asynchronous writes…]

2012-01-20 Thread Nick Mathewson
On Fri, Jan 20, 2012 at 11:17 AM, Nick Mathewson wrote: > On Fri, Jan 20, 2012 at 4:27 AM, Frank Schoep wrote: >> On 19 jan 2012, at 22:00, MigueL DíaZ wrote: >>> … >>> I'd second that idea. Unless you intend to use the same code/protocol you >>> use for you inter-thread communication in a netwo

[Libevent-users] Re: [Libevent-users] Re: [Libevent-users] Security question [was: Asynchronous writes…]

2012-01-20 Thread Nick Mathewson
On Fri, Jan 20, 2012 at 4:27 AM, Frank Schoep wrote: > On 19 jan 2012, at 22:00, MigueL DíaZ wrote: >> … >> I'd second that idea. Unless you intend to use the same code/protocol you >> use for you inter-thread communication in a network environment, using >> evbuffer might be just an overkill. >

[Libevent-users] Re: [Libevent-users] Security question [was: Asynchronous writes…]

2012-01-20 Thread Frank Schoep
On 19 jan 2012, at 22:00, MigueL DíaZ wrote: > … > I'd second that idea. Unless you intend to use the same code/protocol you use > for you inter-thread communication in a network environment, using evbuffer > might be just an overkill. Thanks for the tips, Nick and Miguel – they made me rethink