thomascube
2009-07-13 a366a323b5d78f453b4988be576e6520957c9488
Prevent from endless loops in render_page hook

2 files modified
23 ■■■■■ changed files
program/include/rcube_plugin_api.php 18 ●●●●● patch | view | raw | blame | history
program/include/rcube_template.php 5 ●●●●● patch | view | raw | blame | history
program/include/rcube_plugin_api.php
@@ -39,7 +39,8 @@
  private $objectsmap = array();
  private $template_contents = array();
  
  private  $required_plugins = array('filesystem_attachments');
  private $required_plugins = array('filesystem_attachments');
  private $active_hook = false;
  /**
   * This implements the 'singleton' design pattern
@@ -179,6 +180,7 @@
  public function exec_hook($hook, $args = array())
  {
    $args += array('abort' => false);
    $this->active_hook = $hook;
    
    foreach ((array)$this->handlers[$hook] as $callback) {
      $ret = call_user_func($callback, $args);
@@ -189,6 +191,7 @@
        break;
    }
    
    $this->active_hook = false;
    return $args;
  }
@@ -257,6 +260,19 @@
    }
  }
  
  /**
   * Check if a plugin hook is currently processing.
   * Mainly used to prevent loops and recursion.
   *
   * @param string Hook to check (optional)
   * @return boolean True if any/the given hook is currently processed, otherwise false
   */
  public function is_processing($hook = null)
  {
    return $this->active_hook && (!$hook || $this->active_hook == $hook);
  }
  /**
   * Include a plugin script file in the current HTML page
   */
program/include/rcube_template.php
@@ -287,6 +287,11 @@
    public function send($templ = null, $exit = true)
    {
        if ($templ != 'iframe') {
            // prevent from endless loops
            if ($this->app->plugins->is_processing('render_page')) {
                raise_error(array('code' => 505, 'type' => 'php', 'message' => 'Recursion alert: ignoring output->send()'), true, false);
                return;
            }
            $this->parse($templ, false);
        }
        else {