better patch for the XSS search issue

Since the query string could be output when displaying the results too

diff --git a/docs/search.php.code.in b/docs/search.php.code.in
index 84f87591d5b1baa5856d56eb71f9cc9b09517351..c7511c3782553ba3082a17c2e3e0ded4e6fc8566 100644 (file)
--- a/docs/search.php.code.in
+++ b/docs/search.php.code.in
@@ -9,11 +9,12 @@
     $scope = ltrim ($scope);
     if ($scope == "")
         $scope = "any";
+    $querystr = htmlspecialchars($query, ENT_QUOTES, 'UTF-8');
 ?>
 
 <form action="<?php echo $_SERVER['PHP_SELF'], "?query=", rawurlencode($query) ?>"
       enctype="application/x-www-form-urlencoded" method="get">
-  <input name="query" type="text" size="50" value="<?php echo htmlspecialchars($query, ENT_QUOTES, 'UTF-8')?>"/>
+  <input name="query" type="text" size="50" value="<?php echo $querystr ?>"/>
   <select name="scope">
     <option value="any">Search All</option>
     <option value="API" <?php if ($scope == 'API') print "selected='selected'"?>>Only the APIs</option>
@@ -200,7 +201,7 @@
            }
            mysql_close($link);
            $nb = count($results);
-           echo "<h3 align='center'>Found $nb results for query $query</h3>\n";
+           echo "<h3 align='center'>Found $nb results for query $querystr</h3>\n";
            usort($results, "resSort");
 
             if ($nb > 0) {