Changeset 1657

Timestamp:

06/10/2007 05:57:27 PM (19 months ago)

Author:

robertb

Message:

SQL-Injektion und XSS behoben

Files:

• trunk/plugins/CommentCloser.jplug.php (modified) (4 diffs)

trunk/plugins/CommentCloser.jplug.php

@@ -3 +3 @@
  * @name:        CommentCloser <jeenaparadies.net/projects/jlog/>
  * @author:      Robert Bienert <robertbienert@gmx.net>
- * @version:     1.1
- * @date:        2007-05-26
+ * @version:     1.2
+ * @date:        2007-06-10
  *
  * Dieses Plugin schliesst nach einer bestimmten, festzulegenden
@@ -38 +38 @@
                         $lifeTime = $_POST['commclose_lifetime'];
                         $timeUnit = $_POST['commclose_unit'];
-                        # TODO Einheiten ueberpruefen
+                        // Einheiten pruefen
+                        if (! in_array($timeUnit, $this->_units))
+                                $timeUnit = '';
 
-                        // keine negative Zeiten
-                        if ($lifeTime < 0)
+                        // keine (negative) Zeit XXX implicit cast
+                        if ($lifeTime + 0 <= 0)
                                 $lifeTime = 0;
 
@@ -80 +82 @@
                         $output .= $this->_units[$i];
 
-                        if ($this->_units[$i] == COMMENT_CLOSER_TIMEUNIT)
+                        if ($this->_units[$i] == $timeUnit)
                                 $output .= '" selected="selected';
 
@@ -99 +101 @@
         // Pruefen auf zu schliessende Beitraege
         function hook_onUpdate($data) {
+                // Kein Schliessen gewuenscht, Plugin nicht konfiguriert
+                // oder ungueltige Werte gesetzt:
                 if (!defined('COMMENT_CLOSER_LIFETIME') ||
-                        COMMENT_CLOSER_LIFETIME == 0 ||
-                        !defined('COMMENT_CLOSER_TIMEUNIT'))
+                        COMMENT_CLOSER_LIFETIME+0 <= 0 ||
+                        !defined('COMMENT_CLOSER_TIMEUNIT') ||
+                        !in_array(COMMENT_CLOSER_TIMEUNIT,
+                                $this->_units))
                 {
                         return $data;