Looks reasonable, BB compiles.  Tests needed (and not really for this
particular case, but for a more generic function "copy buffer list in
<format a> to a new buffer list in <format b>, with a skip list"
which would get used here... doing this in a single buffer just makes
everything much more complicated, for the gain of saving a single
malloc/free...).

Your patch has been applied to the master branch.

commit dc5baf8039b37095109f22632a2ca02da805b548
Author: Heiko Hund
Date:   Wed Nov 26 11:49:32 2025 +0100

     iservice: fix off by one error

     Signed-off-by: Heiko Hund <[email protected]>
     Acked-by: Gert Doering <[email protected]>
     Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/1395
     Message-Id: <[email protected]>
     URL: 
https://www.mail-archive.com/[email protected]/msg34678.html
     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