Not sure why this was left out in the sun for so long... seems I ran
out of steam for the integers.  Anyway, thanks to Razvan for looking
at this - and yes, this should not really be needed ("the compiler knows
that 'len' is a positive integer at this point") but it's not too ugly,
and gets rid of one pair of #pragmas

Your patch has been applied to the master branch.

commit 8f2c2fac6cebe453166666fd73c0f58fac1f4a29
Author: Frank Lichtenheld
Date:   Thu Aug 13 20:08:29 2026 +0200

     tun: Avoid sign-compare warning in header_modify_read_write_return

     Signed-off-by: Frank Lichtenheld <[email protected]>
     Acked-by: Razvan Cojocaru <[email protected]>
     Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/1527
     Message-Id: <[email protected]>
     URL: 
https://www.mail-archive.com/[email protected]/msg38330.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