So - stared at code, typical "we have a character here that we can not
handle right now, oops, where is did it disappear to?" thing, and the
peek() fix makes sense.  Also, tested against #1247, which fails without
this patch, and succeeds afterwards.  BB is happy as well, and we get
a new UT test...

Recorded the second +2 in gerrit here in the commit, the mail does not
show it due to "fingers too fast" timing :) - but gerrit has it.

Your patch has been applied to the master branch.

commit f5a2e2319bc7c3cce7f5cdccae44b9bae3e33375
Author: Frank Lichtenheld
Date:   Wed Oct 8 12:29:55 2025 +0200

     buffer: Fix buf_parse eating input

     Signed-off-by: Frank Lichtenheld <[email protected]>
     Acked-by: Gert Doering <[email protected]>
     Acked-by: Arne Schwabe <[email protected]>
     Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/1246
     Message-Id: <[email protected]>
     URL: https://sourceforge.net/p/openvpn/mailman/message/59243829/
     Signed-off-by: Gert Doering <[email protected]>


--
kind regards,

Gert Doering



_______________________________________________
Openvpn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to