Am 13.08.20 um 12:13 schrieb Gert Doering:
> Commit 5fde831c5807 fixed NEXTADDR() for all *BSDs and MacOS.
>
> OpenSolaris has to use a slightly different macro due to lack of
> sockaddr->sa_len - but it has the same problem, first rounding up,
> then memmove()'ing. Switch order.
>
> Signed-off-by: Gert Doering <g...@greenie.muc.de>
> ---
> src/openvpn/route.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/openvpn/route.c b/src/openvpn/route.c
> index 24563ed6..f127a90a 100644
> --- a/src/openvpn/route.c
> +++ b/src/openvpn/route.c
> @@ -3429,7 +3429,7 @@ struct rtmsg {
> #if defined(TARGET_SOLARIS)
> #define NEXTADDR(w, u) \
> if (rtm_addrs & (w)) { \
> - l = ROUNDUP(sizeof(u)); memmove(cp, &(u), l); cp += l; \
> + l = sizeof(u); memmove(cp, &(u), l); cp += ROUNDUP(l); \
> }
>
> #define ADVANCE(x, n) (x += ROUNDUP(sizeof(struct sockaddr_in)))
> Acked-By: Arne Schwabe <a...@rfc2549.org>
signature.asc
Description: OpenPGP digital signature
_______________________________________________ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel
