[levent-tracker] [ levent-Bugs-3410115 ] bufferevent_peek does the Wrong Thing with len=-1 & n_vec=0

[SourceForge.net <no...@so...>]

Bugs item #3410115, was opened at 2011-09-15 18:31
Message generated for change (Tracker Item Submitted) made by nobody
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=461322&aid=3410115&group_id=50884

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: libevent-core
Group: For 2.0
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Nobody/Anonymous (nobody)
Summary: bufferevent_peek does the Wrong Thing with len=-1 & n_vec=0

Initial Comment:
The documentation comment for bufferevent_peek implies that you can find out how many extents you need to inspect the entire buffer by doing

    int nextents = bufferevent_peek(buf, -1, NULL, NULL, 0);

but this does not work; if len<0, bufferevent_peek will return a value no larger than the number of extents provided.

Patch attached.  (I have another patch in my git tree below this one, but there should be no conflict.)

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=461322&aid=3410115&group_id=50884