A rather large and important update:

revised most of the SQL queries and tried to make drupal as secure as possible (while trying to avoid redundant/duplicate checks).  For drupal's sake, try to screw something up.  See the mail about PHPNuke being hacked appr. 6 days ago.  The one who finds a problem is rewarded a beer (and I'm willing to ship it to Norway if required).  I beg you to be evil.  Try dumping a table a la "http://localhost/index.php?date=77778;DROP TABLE users" or something. ;)

1 files changed, 7 insertions, 16 deletions

diff --git a/modules/backend.class b/modules/backend.class
index 593b6904f..679c149a6 100644
--- a/modules/backend.class
+++ b/modules/backend.class
@@ -32,7 +32,7 @@ class backend {
       if (time() - $this->timestamp > $timout) $this->url2sql();
 
       // Read headlines:
-      $result = db_query("SELECT * FROM headlines WHERE id = $this->id ORDER BY number");
+      $result = db_query("SELECT * FROM headlines WHERE id = '$this->id' ORDER BY number");
       while ($headline = db_fetch_object($result)) {
         array_push($this->headlines, "<A HREF=\"$headline->link\">$headline->title</A>");
       }
@@ -91,7 +91,7 @@ class backend {
           }
 
           // Mark channels as being updated:
-          $result = db_query("UPDATE channel SET timestamp = '". time() ."' WHERE id = $this->id");
+          $result = db_query("UPDATE channel SET timestamp = '". time() ."' WHERE id = '$this->id'");
           $this->timestamp = time();
         }
         else {
@@ -113,7 +113,7 @@ class backend {
       if (time() - $this->timestamp > $timout) $this->url2sql();
 
       // Grab headlines from database:
-      $result = db_query("SELECT * FROM headlines WHERE id = $this->id ORDER BY number");
+      $result = db_query("SELECT * FROM headlines WHERE id = '$this->id' ORDER BY number");
       while ($headline = db_fetch_object($result)) {
         $content .= "<LI><A HREF=\"$headline->link\">$headline->title</A></LI>";
       }
@@ -133,27 +133,18 @@ class backend {
 
   function delete() {
     // Delete channel:
-    $result = db_query("DELETE FROM channel WHERE id = $this->id");
+    $result = db_query("DELETE FROM channel WHERE id = '$this->id'");
 
     // Delete headlines:
-    $result = db_query("DELETE FROM headlines WHERE id = $this->id");
+    $result = db_query("DELETE FROM headlines WHERE id = '$this->id'");
   }
 
   function refresh() {
     // Delete headlines:
-    $result = db_query("DELETE FROM headlines WHERE id = $this->id");
+    $result = db_query("DELETE FROM headlines WHERE id = '$this->id'");
 
     // Mark channel as invalid to enforce an update:
-    $result = db_query("UPDATE channel SET timestamp = 1 WHERE id = $this->id");
-  }
-
-  function dump() {
-    print "<B>Dump backend:</B><BR>";
-    print "Id: $this->id<BR>";
-    print "Site: $this->site<BR>";
-    print "URL: $this->url<BR>";
-    print "File: $this->file<BR>";
-    print "Contact: $this->contact<BR>";
+    $result = db_query("UPDATE channel SET timestamp = 1 WHERE id = '$this->id'");
   }
 }