Change state to connecting/resuming before return from StartOpenVPN

- Early state change from the main thread makes it synchronous and
  thus easier to wait on the connection to complete when started
  programmatically.
  Made use of in Connect() in the PLAP implementation that follows.

Does not affect on the current mode of operation.

Signed-off-by: Selva Nair <selva.nair@gmail.com>
pull/529/head
Selva Nair 2022-07-28 22:55:59 -04:00
parent 47f950eca7
commit b2f60c239a
1 changed files with 2 additions and 2 deletions

View File

@ -2105,8 +2105,6 @@ ThreadOpenVPNStatus(void *p)
_tcsncpy(conn_name, c->config_file, _countof(conn_name));
conn_name[_tcslen(conn_name) - _tcslen(o.ext_string) - 1] = _T('\0');
c->state = (c->state == suspended || c->state == detached) ? resuming : connecting;
/* Create and Show Status Dialog */
c->hwndStatus = CreateLocalizedDialogParam(ID_DLG_STATUS, StatusDialogFunc, (LPARAM) c);
if (!c->hwndStatus)
@ -2339,6 +2337,8 @@ StartOpenVPN(connection_t *c)
return false;
}
c->state = (c->state == suspended || c->state == detached) ? resuming : connecting;
/* Start the status dialog thread */
ResumeThread(hThread);