Terminar a lo bruto con un thread

[[escafandra]]

Tengo una aplicación en la que una parte está colocada en un Thread. Esta parte se encarga de conexiones a Internet y en ocasiones provoca fallos en a conexión. Para controlar los fallos, el Thread se autocontrola con un bucle temporal y un valor TimeOut. Y para evitar cuelgues, la aplicación dispone de un Timer. Este Timmer debe terminar con el Thread pero no mediante Treminate(), si no de forma drástica.

1) La API TerminateThread el peligrosa, y no la controlo bien.
2) La API ExitThread termina con el Thread y con la ventana principal de la aplicación, dejando un residuo que no se si es el mismo Thread.
3) La API ExitProcess Termina con todo correctamente pero cancela la aplicación.

Mi intención es terminar sólo con el Thread problemático sin esperar más y dar el código de error correspondiente.

Si alguien tiene más ideas se lo agradezco.

Saludos.

[[cHackAll]]

Empezado por escafandra

1) La API TerminateThread el peligrosa, y no la controlo bien.

Pues la API CreateThread cual crea un hilo nuevo retorna dos valores, el valor retornado de la función en si es la dwThread, y el último parametro [out] es dwThreadId. Entonces debes utilizar una variable GLOBAL para almacenar el valor retornado de la función y luego utilizar la API TerminateThread para finalizarla;

Empezado por Pseudo código

para crear el hilo;
(entero global de 32 bits)dwThread = (API)CreateThread(parámetros);

para finalizarlo;
(API)TerminateThread(dwThread, 0);

Saludos

[[escafandra]]

La API TerminateThread:

BOOL TerminateThread(
    HANDLE hThread,    // handle to the thread 
    DWORD dwExitCode     // exit code for the thread 
   );

La he probado y funciona. Lo he hecho de esta manera:

bool __fastcall MyThread::Exit()
{
   DWORD ExitCode;
   bool R = GetExitCodeThread((HANDLE)Handle, &ExitCode); 
   if(R){
      TerminateThread((HANDLE)Handle, ExitCode);
   } 
   return R;    
}

El problema está en la advertencia que hace Microshoft y algunos foros, sobre la peligrosidad de usarla, y recomiendan terminar de forma más controlada:

TerminateThread is used to cause a thread to exit. When this occurs, the target thread has no chance to execute any user-mode code. DLLs attached to the thread are not notified that the thread is terminating. The system frees the thread's initial stack.

Windows Server 2003 and Windows XP/2000: The target thread's initial stack is not freed, causing a resource leak.

TerminateThread is a dangerous function that should only be used in the most extreme cases. You should call TerminateThread only if you know exactly what the target thread is doing, and you control all of the code that the target thread could possibly be running at the time of the termination. For example, TerminateThread can result in the following problems:

* If the target thread owns a critical section, the critical section will not be released.
* If the target thread is allocating memory from the heap, the heap lock will not be released.
* If the target thread is executing certain kernel32 calls when it is terminated, the kernel32 state for the thread's process could be inconsistent.
* If the target thread is manipulating the global state of a shared DLL, the state of the DLL could be destroyed, affecting other users of the DLL.

A thread cannot protect itself against TerminateThread, other than by controlling access to its handles. The thread handle returned by the CreateThread and CreateProcess functions has THREAD_TERMINATE access, so any caller holding one of these handles can terminate your thread.

If the target thread is the last thread of a process when this function is called, the thread's process is also terminated.

The state of the thread object becomes signaled, releasing any other threads that had been waiting for the thread to terminate. The thread's termination status changes from STILL_ACTIVE to the value of the dwExitCode parameter.

Terminating a thread does not necessarily remove the thread object from the system. A thread object is deleted when the last thread handle is closed.

Otra Cita: Why you should never call Suspend/TerminateThread

No se si terminando como he expuesto mas arriba, puedo tener problemas mas adelante en el transcurso de la aplicación. Las pruebas que he realizado han sido satisfactorias, no he tenido problemas, pero tras la advertencia, dudo, y no se si es necesario tomar alguna medida adicional.

Saludos.

[[escafandra]]

¿Nadie sabe como terminar drásticamente con un thread pero de forma controlada?

Saludos

[[cHackAll]]

No entiendo lo que esperas; talvez la idea de que el hilo se cierre solo luego de obtener una peticion;

Código Delphi [-]
     repeat
        try
          HandleMessage;
        except
          HandleException(Self);
        end;
      until Terminated;

Acá tenemos el "control" que la VCL de Delphi usa en su hilo principal; una variable booleana global (FTerminated), o parte del objeto que el hilo toma como accion de cierre y la misma finaliza "de forma controlada".

Saludos

[[escafandra]]

Claro, ese es un sistema similar al utilizado por la clase TThread. La variable Terminated se usa para el control de la terminación del bucle del thread.

Lo que espero es hacerlo de otra manera. Si el thread hace conexiones a internet, puede ocurrir que fallen o que se demoren demasiado, o que caiga la conexión... Entonces, además del método controlado por Terminated, quiero un control externo para la finalización. Este control, que realizaría un Timer con una variable TimeOut, Haría una llamada a la función de terminación drástica. Esa función puede ser miembro del thread o no, según la solución más eficiente.


Espero haberme explicado bien.

Saludos.

[[escafandra]]

Por lo que he investigado, no es posible terminar de forma segura con un thread desde fuera del mismo....

Saludos.