From 85bf75618ce08e9d9c419a63f2a08e3f321bf12c Mon Sep 17 00:00:00 2001
From: Leandro Lucarella <llucax@gmail.com>
Date: Thu, 15 Apr 2004 16:07:56 +0000
Subject: [PATCH] BugFixes varios.

---
 lib/YATTA/Controlador.php          |  5 +--
 script/servidores/YATTA_Server.php | 51 +++++++++++++++++++++++-------
 2 files changed, 43 insertions(+), 13 deletions(-)

diff --git a/lib/YATTA/Controlador.php b/lib/YATTA/Controlador.php
index 4d9a8cd..d33ed81 100644
--- a/lib/YATTA/Controlador.php
+++ b/lib/YATTA/Controlador.php
@@ -160,7 +160,8 @@ class YATTA_Controlador {
             null, $archivo = null, $notificar = 0) {
 
         //Actualizo la base procesos
-        $sql = 'UPDATE yatta.procesos SET status = '. $status;
+        $sql = 'UPDATE yatta.procesos SET status = '. 
+            $db->quoteSmart($status);
         if (@$nota) {
             $sql.= ', nota = '. $db->quoteSmart($nota);
         }
@@ -170,7 +171,7 @@ class YATTA_Controlador {
         if (@$notificar) {
             $sql.= ', notificar = '. $db->quoteSmart($notificar);
         }
-        $sql.= ' WHERE id = '. $id_proceso;
+        $sql.= ' WHERE id = '. $db->quoteSmart($id_proceso);
         
         $res = $db->query($sql);
         if (DB::isError($res)) {
diff --git a/script/servidores/YATTA_Server.php b/script/servidores/YATTA_Server.php
index 2f502c4..4ccd109 100755
--- a/script/servidores/YATTA_Server.php
+++ b/script/servidores/YATTA_Server.php
@@ -27,6 +27,15 @@ $Id$
 
 //XXX LANZAR ESTE SCRIPT CON NICE -19
 
+//TODO No mantener abierto el archivo de log durante toda la ejecucion, porque
+//se afecta con el logrotate. Ver de capturar alguna señal por esto.
+
+//TODO Ver si es conveniente que la conexion al mysql esta abierta desde el
+//principio. Hay problemas con eso?
+
+//TODO Que el logueo sea selectivo segun un parametro en el archivo de
+//configuracion (tipo modo histerico, normal, tranquilo, nada)
+
 //YATTA LOG {{{
 function yatta_log ($texto) {
     GLOBAL $YATTA_SERVER;
@@ -37,7 +46,7 @@ function yatta_log ($texto) {
 }
 //}}}
 
-//Leo el archivo de configuracion {{{
+//LEO EL ARCHIVO DE CONFIGURACION {{{
 $CONF = parse_ini_file(dirname(__FILE__) . '/configuracion.ini', true);
 //}}}
 
@@ -88,20 +97,19 @@ yatta_log ('**************************************************');
 //}}}
 
 //WHILEO LAS OPCIONES {{{
+yatta_log ('Se busca un nuevo proceso.');
 while (1) {
    
     //Busco un nuevo proceso {{{
     $proceso = $CONTROLADOR->obtenerNuevoProceso($db,
             $YATTA_SERVER['id']);
-    yatta_log ('Se busca un nuevo proceso.');
     //}}}
 
     //Si hay un proceso lo lanzo {{{
     if (@$proceso) {
         
         yatta_log ('Hay un nuevo proceso. Se forkea');
-//        $pid = pcntl_fork();
-        $pid = -1;
+        $pid = pcntl_fork();
 
         //FORK ERROR {{{
         if ($pid == -1) {
@@ -119,9 +127,13 @@ while (1) {
         elseif ($pid) {
             yatta_log ('Fork Padre.');
             //Espero a que termine el proceso.
-            yatta_log ('Se espera a que termine el proceso en ejecución. Se'.
-                    ' verificara si hay que abortar el proceso cada 5 segundo.');
-            while (($wapid = pcntl_waitpid ($pid, $status, WNOHANG)) != $pid) {
+            yatta_log ('Se espera a que termine el proceso en ejecución. '.
+                    ' Verifica aborto en 5 segundos.');
+
+            //Pregunto si el hijo que se forkeo arriba termino, pero pido que no
+            //corte la ejecucion (WNOHANG) para poder entrar en el while.
+            //$res sera el pid del hijo, en este caso $res = $pid.
+            while (!($res = pcntl_waitpid($pid, $status, WNOHANG))) {
                 if ($CONTROLADOR->abortar($db, $proceso['id'])) {
                     yatta_log ('Hay que abortar el proceso en ejecución.');
                     posix_kill($pid, 15); //SIGTERM
@@ -129,7 +141,7 @@ while (1) {
                             $YATTA_SERVER['id'], 4);
                     yatta_log ('Se aborto el proceso en ejecución.');
                 }
-                sleep(10);
+                sleep(5);
             }
             yatta_log ('Fin fork Padre.');
         }
@@ -139,9 +151,12 @@ while (1) {
         else {
             yatta_log ('Fork Hijo.');
 
+            //Ejecuto el proceso si existe el archivo {{{
             if (is_readable($proceso['script'])) {
                 require_once $proceso['script'];
-                yatta_log ('Se crea el proceso..');
+                yatta_log ('Se llama a create_process.');
+                
+                //Llamo a la funcion create_process {{{
                 if (function_exists('create_process')) {
                     $SCRIPT =& create_process();
                     yatta_log ('Se instancio el proceso. Se ejecuta el metodo run.');
@@ -158,32 +173,46 @@ while (1) {
                         yatta_log ('Finaliza la ejecución del proceso.');
                     }
                 }
+                //}}}
+
+                //No existe la funcion {{{
                 else {
                     require_once 'YATTA/Proceso.php';
                     $SCRIPT =& new YATTA_Proceso;
                     $SCRIPT->error = 'No existe la funcion create_process en'.
                         $proceso['script'];
                 }
+                //}}}
 
+                //Finalizo el script {{{
                 if (@$SCRIPT->error) {
                     $estado = 3;
                 }
                 else {
                     $estado = 2;
                 }
-                $CONROLADOR->terminarProceso($db, $proceso['id'],
+                $CONTROLADOR->terminarProceso($db, $proceso['id'],
                         $YATTA_SERVER['id'], $estado, @$SCRIPT->error,
                         @$SCRIPT->archivo, @$SCRIPT->notificar);
+                //}}}
+            
             }
+            //}}}
+           
+            //No existe el archivo {{{
             else {
                 yatta_log('No se puede alcanzar el archivo.');
+                $res = $CONTROLADOR->terminarProceso($db, $proceso['id'],
+                        $YATTA_SERVER['id'], 3, 'No se puede alcanzar el archivo.');
             }
-           
+            //}}}
             
             yatta_log ('Fin fork Hijo.');
+            exit;
         }
         //}}}
 
+        unset($proceso);
     }
     // }}}
 
-- 
2.43.0