summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--web/html/pkgreq.php10
-rw-r--r--web/lib/aur.inc.php13
-rw-r--r--web/lib/pkgfuncs.inc.php16
3 files changed, 19 insertions, 20 deletions
diff --git a/web/html/pkgreq.php b/web/html/pkgreq.php
index 156645a3..401b60c7 100644
--- a/web/html/pkgreq.php
+++ b/web/html/pkgreq.php
@@ -21,19 +21,13 @@ if (!isset($base_id)) {
/* Sanitize paging variables. */
if (isset($_GET['O'])) {
- $_GET['O'] = intval($_GET['O']);
- if ($_GET['O'] < 0)
- $_GET['O'] = 0;
+ $_GET['O'] = max(intval($_GET['O']), 0);
} else {
$_GET['O'] = 0;
}
if (isset($_GET["PP"])) {
- $_GET["PP"] = intval($_GET["PP"]);
- if ($_GET["PP"] < 50)
- $_GET["PP"] = 50;
- else if ($_GET["PP"] > 250)
- $_GET["PP"] = 250;
+ $_GET["PP"] = bound(intval($_GET["PP"]), 50, 250);
} else {
$_GET["PP"] = 50;
}
diff --git a/web/lib/aur.inc.php b/web/lib/aur.inc.php
index 33686968..7fa792bb 100644
--- a/web/lib/aur.inc.php
+++ b/web/lib/aur.inc.php
@@ -584,3 +584,16 @@ function array_pkgbuild_merge($pkgbase_info, $section_info) {
}
return $pi;
}
+
+/**
+ * Bound an integer value between two values
+ *
+ * @param int $n Integer value to bound
+ * @param int $min Lower bound
+ * @param int $max Upper bound
+ *
+ * @return int Bounded integer value
+ */
+function bound($n, $min, $max) {
+ return min(max($n, $min), $max);
+}
diff --git a/web/lib/pkgfuncs.inc.php b/web/lib/pkgfuncs.inc.php
index f5158646..7e65d091 100644
--- a/web/lib/pkgfuncs.inc.php
+++ b/web/lib/pkgfuncs.inc.php
@@ -519,22 +519,14 @@ function pkg_search_page($SID="") {
/* Sanitize paging variables. */
if (isset($_GET['O'])) {
- $_GET['O'] = intval($_GET['O']);
- if ($_GET['O'] < 0)
- $_GET['O'] = 0;
- }
- else {
+ $_GET['O'] = max(intval($_GET['O']), 0);
+ } else {
$_GET['O'] = 0;
}
if (isset($_GET["PP"])) {
- $_GET["PP"] = intval($_GET["PP"]);
- if ($_GET["PP"] < 50)
- $_GET["PP"] = 50;
- else if ($_GET["PP"] > 250)
- $_GET["PP"] = 250;
- }
- else {
+ $_GET["PP"] = bound(intval($_GET["PP"]), 50, 250);
+ } else {
$_GET["PP"] = 50;
}