[aur-dev] [PATCH 3/4] Check query return value in db_cache_value()

Lukas Fleischer lfleischer at archlinux.org
Tue Apr 18 06:52:53 UTC 2017


Instead of unconditionally calling fetch on the return value of query(),
error out early if the value evaluates to false.

Signed-off-by: Lukas Fleischer <lfleischer at archlinux.org>
---
 web/lib/cachefuncs.inc.php | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/web/lib/cachefuncs.inc.php b/web/lib/cachefuncs.inc.php
index d558be4..faeae5a 100644
--- a/web/lib/cachefuncs.inc.php
+++ b/web/lib/cachefuncs.inc.php
@@ -73,6 +73,9 @@ function db_cache_value($dbq, $key, $ttl=600) {
 	$value = get_cache_value($key, $status);
 	if (!$status) {
 		$result = $dbh->query($dbq);
+		if (!$result) {
+			return false;
+		}
 		$row = $result->fetch(PDO::FETCH_NUM);
 		$value = $row[0];
 		set_cache_value($key, $value, $ttl);
-- 
2.12.2


More information about the aur-dev mailing list