Re: Review Request 41026: libevent ssl: Added check for buffer length before swapping request.

2015-12-22 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41026/#review111657 --- Ship it! Ship It! - Joris Van Remoortere On Dec. 18, 2015,

Re: Review Request 41026: libevent ssl: Added check for buffer length before swapping request.

2015-12-18 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41026/ --- (Updated Dec. 18, 2015, 11:53 p.m.) Review request for mesos and Joris Van

Re: Review Request 41026: libevent ssl: Added check for buffer length before swapping request.

2015-12-18 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41026/ --- (Updated Dec. 18, 2015, 11:51 p.m.) Review request for mesos and Joris Van

Re: Review Request 41026: libevent ssl: Added check for buffer length before swapping request.

2015-12-18 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41026/#review111295 --- 3rdparty/libprocess/src/libevent_ssl_socket.cpp (lines 225 - 226)

Re: Review Request 41026: libevent ssl: Added check for buffer length before swapping request.

2015-12-07 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41026/#review109294 --- Patch looks great! Reviews applied: [41026] Passed command:

Re: Review Request 41026: libevent ssl: Added check for buffer length before swapping request.

2015-12-07 Thread Jojy Varghese
> On Dec. 8, 2015, 1:43 a.m., Joseph Wu wrote: > > Would it be plausible to write a repro/test (in the libprocess level) for > > this? > > > > Presumably, we should be able to write a process that does a "long running > > streaming download" (which causes the bug, according to your

Re: Review Request 41026: libevent ssl: Added check for buffer length before swapping request.

2015-12-07 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41026/#review109226 --- Patch looks great! Reviews applied: [41026] Passed command:

Re: Review Request 41026: libevent ssl: Added check for buffer length before swapping request.

2015-12-07 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41026/ --- (Updated Dec. 8, 2015, 1:22 a.m.) Review request for mesos and Joris Van

Re: Review Request 41026: libevent ssl: Added check for buffer length before swapping request.

2015-12-07 Thread Joseph Wu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41026/#review109246 --- Would it be plausible to write a repro/test (in the libprocess