- don't set password via remoting if field is empty

Marius Cramer

2015-08-06 37b29231e47a0c4458dc1c15d98588f16f07e1e2

 interface/web/admin/remote_action_osupdate.php

@@ -32,6 +32,7 @@

//* Check permissions for module
$app->auth->check_module_permissions('admin');
$app->auth->check_security_permissions('admin_allow_osupdate');

//* This is only allowed for administrators
if(!$app->auth->is_admin()) die('only allowed for administrators.');
@@ -61,6 +62,10 @@
 * If the user wants to do the action, write this to our db
*/
if (isset($_POST['server_select'])) {
	
   //* CSRF Check
   $app->auth->csrf_token_check();
	
   $server = $_POST['server_select'];
   $servers = array();
   if ($server == '*') {
@@ -75,21 +80,19 @@
   }
   foreach ($servers as $serverId) {
      $sql = "INSERT INTO sys_remoteaction (server_id, tstamp, action_type, action_param, action_state, response) " .
         "VALUES (".
         (int)$serverId . ", " .
         time() . ", " .
         "'os_update', " .
         "'', " .
         "'pending', " .
         "''" .
         ")";
      $app->db->query($sql);
         "VALUES (?, UNIX_TIMESTAMP(), 'os_update', '', 'pending', '')";
      $app->db->query($sql, $serverId);
   }
   $msg = $wb['action_scheduled'];
}

$app->tpl->setVar('msg', $msg);

//* SET csrf token
$csrf_token = $app->auth->csrf_token_get('osupdate');
$app->tpl->setVar('_csrf_id',$csrf_token['csrf_id']);
$app->tpl->setVar('_csrf_key',$csrf_token['csrf_key']);

$app->tpl->setVar($wb);

$app->tpl_defaults();