free(vpninfo->urlpath)
@dlenski After examining the Fortinet code in the context of !418 (merged), I am wondering whether we should call free(vpninfo->urlpath) before each assignment to vpninfo->urlpath:
fortinet.c
Lines 643 to 656 in 910dfa4c
vpninfo->urlpath = strdup("remote/fortisslvpn_xml");
ret = do_https_request(vpninfo, "GET", NULL, NULL, &res_buf, NULL, HTTP_NO_FLAGS);
if (ret < 0) {
if (ret == -EPERM) {
[...]
vpninfo->urlpath = strdup("remote/fortisslvpn");
This is enforced everywhere else in the OpenConnect source code.