summaryrefslogtreecommitdiffstats
path: root/web/lib/aurjson.class.php
diff options
context:
space:
mode:
authorcanyonknight <canyonknight@gmail.com>2012-10-21 21:53:16 +0200
committerLukas Fleischer <archlinux@cryptocrack.de>2012-10-22 12:33:13 +0200
commit43a69e7127c65f2651a61300d3e32bf6f89a2595 (patch)
treecb0869c747d12fa8418a536878d7b6aaf5eeb3f9 /web/lib/aurjson.class.php
parent5222bf09320ebc1cd5da799970fcc0d2dd32bb3f (diff)
downloadaur-43a69e7127c65f2651a61300d3e32bf6f89a2595.tar.gz
aur-43a69e7127c65f2651a61300d3e32bf6f89a2595.tar.xz
aurjson.class.php: Limit number of RPC results
With no limit to the number of results, memory_limit set to 32M can easily be exceeded for searches that have a large number of results. This results in an HTTP error 500 for those queries. Limit results to an amount set within config.inc.php to avoid exceeding memory_limit. Introduce new JSON error code for when the result limit is hit. Fixes FS#31849 Signed-off-by: canyonknight <canyonknight@gmail.com> Signed-off-by: Lukas Fleischer <archlinux@cryptocrack.de>
Diffstat (limited to 'web/lib/aurjson.class.php')
-rw-r--r--web/lib/aurjson.class.php22
1 files changed, 17 insertions, 5 deletions
diff --git a/web/lib/aurjson.class.php b/web/lib/aurjson.class.php
index fbdc7118..949c34f3 100644
--- a/web/lib/aurjson.class.php
+++ b/web/lib/aurjson.class.php
@@ -117,6 +117,7 @@ class AurJSON {
}
private function process_query($type, $where_condition) {
+ global $MAX_RPC_RESULTS;
$fields = implode(',', self::$fields);
$query = "SELECT Users.Username as Maintainer, {$fields} " .
"FROM Packages LEFT JOIN Users " .
@@ -149,6 +150,10 @@ class AurJSON {
}
}
+ if ($resultcount === $MAX_RPC_RESULTS) {
+ return $this->json_error('Too many package results.');
+ }
+
return $this->json_results($type, $resultcount, $search_data);
}
else {
@@ -191,6 +196,7 @@ class AurJSON {
* @return mixed Returns an array of package matches.
**/
private function search($keyword_string) {
+ global $MAX_RPC_RESULTS;
if (strlen($keyword_string) < 2) {
return $this->json_error('Query arg too small');
}
@@ -198,7 +204,8 @@ class AurJSON {
$keyword_string = $this->dbh->quote("%" . addcslashes($keyword_string, '%_') . "%");
$where_condition = "(Name LIKE {$keyword_string} OR ";
- $where_condition.= "Description LIKE {$keyword_string})";
+ $where_condition.= "Description LIKE {$keyword_string}) ";
+ $where_condition.= "LIMIT {$MAX_RPC_RESULTS}";
return $this->process_query('search', $where_condition);
}
@@ -227,6 +234,7 @@ class AurJSON {
* @return mixed Returns an array of results containing the package data
**/
private function multiinfo($pqdata) {
+ global $MAX_RPC_RESULTS;
$args = $this->parse_multiinfo_args($pqdata);
$ids = $args['ids'];
$names = $args['names'];
@@ -238,17 +246,19 @@ class AurJSON {
$where_condition = "";
if ($ids) {
$ids_value = implode(',', $args['ids']);
- $where_condition .= "ID IN ({$ids_value})";
+ $where_condition .= "ID IN ({$ids_value}) ";
}
if ($ids && $names) {
- $where_condition .= " OR ";
+ $where_condition .= "OR ";
}
if ($names) {
// individual names were quoted in parse_multiinfo_args()
$names_value = implode(',', $args['names']);
- $where_condition .= "Name IN ({$names_value})";
+ $where_condition .= "Name IN ({$names_value}) ";
}
+ $where_condition .= "LIMIT {$MAX_RPC_RESULTS}";
+
return $this->process_query('multiinfo', $where_condition);
}
@@ -258,9 +268,11 @@ class AurJSON {
* @return mixed Returns an array of value data containing the package data
**/
private function msearch($maintainer) {
+ global $MAX_RPC_RESULTS;
$maintainer = $this->dbh->quote($maintainer);
- $where_condition = "Users.Username = {$maintainer}";
+ $where_condition = "Users.Username = {$maintainer} ";
+ $where_condition .= "LIMIT {$MAX_RPC_RESULTS}";
return $this->process_query('msearch', $where_condition);
}