[aur-dev] [PATCH] Verify that the target of merge operations exists
Lukas Fleischer
archlinux at cryptocrack.de
Wed Jul 23 09:21:54 EDT 2014
Make sure that the target of a merge operation is either empty or an
existing package base name.
Signed-off-by: Lukas Fleischer <archlinux at cryptocrack.de>
---
web/lib/pkgreqfuncs.inc.php | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/web/lib/pkgreqfuncs.inc.php b/web/lib/pkgreqfuncs.inc.php
index 5b86eaa..41d1515 100644
--- a/web/lib/pkgreqfuncs.inc.php
+++ b/web/lib/pkgreqfuncs.inc.php
@@ -95,6 +95,10 @@ function pkgreq_file($ids, $type, $merge_into, $comments) {
return array(false, __("Invalid name: only lowercase letters are allowed."));
}
+ if (!empty($merge_into) && !pkgbase_from_name($merge_into)) {
+ return array(false, __("Cannot find package to merge votes and comments into."));
+ }
+
if (empty($comments)) {
return array(false, __("The comment field must not be empty."));
}
--
2.0.2
More information about the aur-dev
mailing list