BugFixes varios.

diff --git a/lib/YATTA/Controlador.php b/lib/YATTA/Controlador.php
index 4d9a8cdab24d0809a1f6656761197842c1a215b9..d33ed815dd0410a100c5984cbb5bbc80c7be9280 100644 (file)
--- 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 2f502c47da741fdbbc0ae2964772c993504db240..4ccd109e217b7fa164a73a0ea1d5c917eef45cda 100755 (executable)
--- 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);
     }
     // }}}