Invalid access to memory location using GlobalFree

[Paul McKenzie]

What's this critical section for?

EnterCriticalSection(&l_statcheck);
if (l_State == TRUE) {
	exit(ShowError(ERR_HEARTBEAT));
}
LeaveCriticalSection(&l_statcheck);

No variable is being directly being assigned to. A comparison is done, and then a call to exit().

If ShowError for some reason needs to be synchronized, then that is where the critical section would be placed, and not at this level. Otherwise I see no need for the critical section -- this is paranoid programming by someone who didn't understand how and when to use synchronization objects.

Regards,

Paul McKenzie