escape all users input strings
authorArun Persaud <arun@nubati.net>
Sun, 27 Nov 2011 06:20:22 +0000 (22:20 -0800)
committerArun Persaud <apersaud@lbl.gov>
Sun, 27 Nov 2011 06:20:22 +0000 (22:20 -0800)
use intval for integers or sqlite_escape_string for strings

getjson.php
index.php

index cc93c7c..0288646 100644 (file)
@@ -19,7 +19,7 @@ else
 if (isset($_REQUEST["S"]))
   {
     /* single tag or part of tag */
-    $tag = $_REQUEST["S"];
+    $tag = sqlite_escape_string($_REQUEST["S"]);
     /* individual tags are separated by '+' */
     $result = $DB->query("SELECT name FROM tags where name like \"%$tag%\"");
     $count = $DB->query("SELECT 1");
@@ -27,14 +27,14 @@ if (isset($_REQUEST["S"]))
 else
   {
     if (isset($_REQUEST["P"]))
-      $OFFSET = "".($_REQUEST["P"]*$N-$N);
+      $OFFSET = "".(intval($_REQUEST["P"])*$N-$N);
     else
       $OFFSET = "0";
 
     if (isset($_REQUEST["T"]))
       {
        /* single tag or part of tag */
-       $tags = $_REQUEST["T"];
+       $tags = sqlite_escape_string($_REQUEST["T"]);
        $tags = explode("+",$tags);
        $tags = "'".implode("','",$tags)."'";
 
index 822df79..a6112ab 100644 (file)
--- a/index.php
+++ b/index.php
@@ -15,7 +15,7 @@ else
   $page = 1;
 
 if(isset($_REQUEST["tag"]))
-  $tags = $_REQUEST["tag"];
+  $tags = htmlentities($_REQUEST["tag"]);
 else
   $tags = "";