On Tue, Jan 10, 2017 at 3:18 AM, Ilya Shipitsin <chipits...@gmail.com>
wrote:
> diff --git a/src/openvpn/init.c b/src/openvpn/init.c
> index b2547f4..5b39929 100644
> --- a/src/openvpn/init.c
> +++ b/src/openvpn/init.c
> @@ -266,7 +266,6 @@ ce_management_query_remote(struct context *c)
> management_event_loop_n_seconds(management, 1);
> if (IS_SIG(c))
> {
> - ret = false;
> break;
> }
> }
>
This looks like an arbitrary fix to silence the analyzer. Something else is
likely wrong here. Why "ret" is over-written later (line 276) as
ret = (flags != CE_MAN_QUERY_REMOTE_SKIP);
is the real question. Looks like a bug that causes signal to be ignored.
Selva
------------------------------------------------------------------------------
Developer Access Program for Intel Xeon Phi Processors
Access to Intel Xeon Phi processor-based developer platforms.
With one year of Intel Parallel Studio XE.
Training and support from Colfax.
Order your platform today. http://sdm.link/xeonphi
_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
