From: Marco Baffo <ma...@mandelbit.com> Change-Id: I73f8ad9bf105920f4bde357ea9dcf0e485f3f3cb Signed-off-by: Marco Baffo <ma...@mandelbit.com> Acked-by: Frank Lichtenheld <fr...@lichtenheld.com> ---
This change was reviewed on Gerrit and approved by at least one developer. I request to merge it to master. Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/1177 This mail reflects revision 3 of this Change. Acked-by according to Gerrit (reflected above): Frank Lichtenheld <fr...@lichtenheld.com> diff --git a/src/openvpn/push_util.c b/src/openvpn/push_util.c index bd3992b..f7a4fca 100644 --- a/src/openvpn/push_util.c +++ b/src/openvpn/push_util.c @@ -161,6 +161,10 @@ * will not be routed towards the client. * For the same reason we later update the vhash too in * `send_push_update()` function. + * Using `buf_string_compare_advance()` we mimic the behavior + * inside `process_incoming_push_msg()`. However, we don't need + * to check the return value here because we just want to `advance`, + * meaning we skip the `push_update_cmd' we added earlier. */ buf_string_compare_advance(&msgs[i], push_update_cmd); if (process_incoming_push_update(c, pull_permission_mask(c), option_types_found, &msgs[i], true) == PUSH_MSG_ERROR) _______________________________________________ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel
