summaryrefslogtreecommitdiffstats
path: root/sources
diff options
context:
space:
mode:
authorOpenGnSys Support Team <soporte-og@soleta.eu>2019-11-28 17:11:05 +0100
committerOpenGnSys Support Team <soporte-og@soleta.eu>2020-03-04 14:54:39 +0100
commitd3239f70eb4f25c6c894fff1884f5a7cbeb04585 (patch)
tree902670890608086570731e6cf0070bfe2db8f973 /sources
parentad70969c2fb1cd686e7bfa0280b8667b591bec96 (diff)
#941 Consolidate RESPUESTA_Configurar and RESPUESTA_EjecutarScript
These function are almost identical, remove duplicated code.
Diffstat (limited to 'sources')
-rw-r--r--sources/ogAdmServer.cpp64
1 files changed, 8 insertions, 56 deletions
diff --git a/sources/ogAdmServer.cpp b/sources/ogAdmServer.cpp
index 16eb289..e161ffd 100644
--- a/sources/ogAdmServer.cpp
+++ b/sources/ogAdmServer.cpp
@@ -2053,59 +2053,6 @@ static bool Configurar(TRAMA* ptrTrama, struct og_client *cli)
return true;
}
// ________________________________________________________________________________________________________
-// Función: RESPUESTA_Configurar
-//
-// Descripción:
-// Respuesta del cliente al comando Configurar
-// Parámetros:
-// - socket_c: Socket del cliente que envió el mensaje
-// - ptrTrama: Trama recibida por el servidor con el contenido y los parámetros
-// Devuelve:
-// true: Si el proceso es correcto
-// false: En caso de ocurrir algún error
-// ________________________________________________________________________________________________________
-//
-static bool RESPUESTA_Configurar(TRAMA* ptrTrama, struct og_client *ci)
-{
- char msglog[LONSTD];
- Database db;
- Table tbl;
- bool res;
- char *iph, *ido,*cfg;
-
- if (!db.Open(usuario, pasguor, datasource, catalog)) {
- db.GetErrorErrStr(msglog);
- syslog(LOG_ERR, "cannot open connection database (%s:%d) %s\n",
- __func__, __LINE__, msglog);
- return false;
- }
-
- iph = copiaParametro("iph",ptrTrama); // Toma dirección ip
- ido = copiaParametro("ido",ptrTrama); // Toma identificador del ordenador
-
- if (!respuestaEstandar(ptrTrama, iph, ido, db, tbl)) {
- liberaMemoria(iph);
- liberaMemoria(ido);
- syslog(LOG_ERR, "failed to register notification\n");
- return false;
- }
-
- cfg = copiaParametro("cfg",ptrTrama); // Toma configuración de particiones
- res=actualizaConfiguracion(db, tbl, cfg, atoi(ido)); // Actualiza la configuración del ordenador
-
- liberaMemoria(iph);
- liberaMemoria(ido);
- liberaMemoria(cfg);
-
- if(!res){
- syslog(LOG_ERR, "Problem updating client configuration\n");
- return false;
- }
-
- db.Close(); // Cierra conexión
- return true;
-}
-// ________________________________________________________________________________________________________
// Función: EjecutarScript
//
// Descripción:
@@ -2144,6 +2091,7 @@ static bool RESPUESTA_EjecutarScript(TRAMA* ptrTrama, struct og_client *cli)
Database db;
Table tbl;
char *iph, *ido,*cfg;
+ bool res = true;
if (!db.Open(usuario, pasguor, datasource, catalog)) {
db.GetErrorErrStr(msglog);
@@ -2164,14 +2112,18 @@ static bool RESPUESTA_EjecutarScript(TRAMA* ptrTrama, struct og_client *cli)
cfg = copiaParametro("cfg",ptrTrama); // Toma configuración de particiones
if(cfg){
- actualizaConfiguracion(db, tbl, cfg, atoi(ido)); // Actualiza la configuración del ordenador
+ res = actualizaConfiguracion(db, tbl, cfg, atoi(ido)); // Actualiza la configuración del ordenador
liberaMemoria(cfg);
}
liberaMemoria(iph);
liberaMemoria(ido);
-
+ if (!res) {
+ syslog(LOG_ERR, "Problem updating client configuration\n");
+ return false;
+ }
+
db.Close(); // Cierra conexión
return true;
}
@@ -3064,7 +3016,7 @@ static struct {
{ "RestaurarSoftIncremental", RestaurarSoftIncremental, },
{ "RESPUESTA_RestaurarSoftIncremental", RESPUESTA_RestaurarSoftIncremental, },
{ "Configurar", Configurar, },
- { "RESPUESTA_Configurar", RESPUESTA_Configurar, },
+ { "RESPUESTA_Configurar", RESPUESTA_EjecutarScript, },
{ "EjecutarScript", EjecutarScript, },
{ "RESPUESTA_EjecutarScript", RESPUESTA_EjecutarScript, },
{ "RESPUESTA_InventarioHardware", RESPUESTA_InventarioHardware, },