From 4409908a92b849d8b3f1a87a307229917a4a0197 Mon Sep 17 00:00:00 2001 From: Yannick Warnier Date: Mon, 19 Jul 2010 23:37:24 -0500 Subject: [PATCH] Minor - ensure false is returned when no api_service is provided --- main/inc/lib/usermanager.lib.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/main/inc/lib/usermanager.lib.php b/main/inc/lib/usermanager.lib.php index 4e8b7976c8..d30b739d2a 100755 --- a/main/inc/lib/usermanager.lib.php +++ b/main/inc/lib/usermanager.lib.php @@ -2193,9 +2193,9 @@ class UserManager public static function get_api_key_id($user_id, $api_service) { if ($user_id != strval(intval($user_id))) return false; if ($user_id === false) return false; - $service_name = Database::escape_string($api_service); - if (is_string($service_name) === false) { return false; } + if (empty($api_service)) return false; $t_api = Database::get_main_table(TABLE_MAIN_USER_API_KEY); + $service_name = Database::escape_string($api_service); $sql = "SELECT id FROM $t_api WHERE user_id=".$user_id." AND api_service='".$api_service."'"; $res = Database::query($sql); if (Database::num_rows($res)<1) {