From 3609cf140eaca30878d0a69fba0d837b4fec41a0 Mon Sep 17 00:00:00 2001
From: Dan McGee <dan@archlinux.org>
Date: Mon, 25 Apr 2011 23:23:01 -0500
Subject: SQL: treat all UID/ID values as numbers, not strings

Ensure we are not quoting these values in any of our SQL queries.

Thanks-to: elij <elij.mx@gmail.com>
Signed-off-by: Dan McGee <dan@archlinux.org>
Signed-off-by: Lukas Fleischer <archlinux@cryptocrack.de>
---
 web/lib/acctfuncs.inc | 18 +++++++++---------
 web/lib/aur.inc       |  6 +++---
 2 files changed, 12 insertions(+), 12 deletions(-)

(limited to 'web/lib')

diff --git a/web/lib/acctfuncs.inc b/web/lib/acctfuncs.inc
index 8e2ecb3..8ffa2f7 100644
--- a/web/lib/acctfuncs.inc
+++ b/web/lib/acctfuncs.inc
@@ -619,18 +619,18 @@ function try_login() {
 					# last ($MAX_SESSIONS_PER_USER - 1).
 					$q = "DELETE s.* FROM Sessions s ";
 					$q.= "LEFT JOIN (SELECT SessionID FROM Sessions ";
-					$q.= "WHERE UsersId = '" . $userID . "' ";
+					$q.= "WHERE UsersId = " . $userID . " ";
 					$q.= "ORDER BY LastUpdateTS DESC ";
 					$q.= "LIMIT " . ($MAX_SESSIONS_PER_USER - 1) . ") q ";
 					$q.= "ON s.SessionID = q.SessionID ";
-					$q.= "WHERE s.UsersId = '" . $userID . "' ";
+					$q.= "WHERE s.UsersId = " . $userID . " ";
 					$q.= "AND q.SessionID IS NULL;";
 					db_query($q, $dbh);
 				}
 
 				$new_sid = new_sid();
 				$q = "INSERT INTO Sessions (UsersID, SessionID, LastUpdateTS)"
-				  ." VALUES ( $userID, '" . $new_sid . "', UNIX_TIMESTAMP())";
+				  ." VALUES (" . $userID . ", '" . $new_sid . "', UNIX_TIMESTAMP())";
 				$result = db_query($q, $dbh);
 
 				# Query will fail if $new_sid is not unique
@@ -749,7 +749,7 @@ function valid_passwd( $userID, $passwd )
 		if ($salt) {
 			# use salt
 			$passwd_q = "SELECT ID FROM Users" .
-				" WHERE ID = '$userID' AND Passwd = '" .
+				" WHERE ID = " . $userID  . " AND Passwd = '" .
 				salted_hash($passwd, $salt) . "'";
 			$passwd_result = mysql_fetch_row(db_query($passwd_q, $dbh));
 			if ($passwd_result[0]) {
@@ -758,14 +758,14 @@ function valid_passwd( $userID, $passwd )
 		} else {
 			# check without salt
 			$nosalt_q = "SELECT ID FROM Users".
-				" WHERE ID = '$userID'" .
+				" WHERE ID = " . $userID .
 				" AND Passwd = '" . md5($passwd) . "'";
 			$nosalt_result = mysql_fetch_row(db_query($nosalt_q, $dbh));
 			if ($nosalt_result[0]) {
 				# password correct, but salt it first
 				if (!save_salt($userID, $passwd)) {
 					trigger_error("Unable to salt user's password;" .
-						" ID $userID", E_USER_WARNING);
+						" ID " . $userID, E_USER_WARNING);
 					return false;
 				}
 
@@ -782,7 +782,7 @@ function valid_passwd( $userID, $passwd )
 function user_suspended( $id )
 {
 	$dbh = db_connect();
-	$q = "SELECT Suspended FROM Users WHERE ID = '$id'";
+	$q = "SELECT Suspended FROM Users WHERE ID = " . $id;
 	$result = mysql_fetch_row(db_query($q, $dbh));
 	if ($result[0] == 1 ) {
 		return true;
@@ -796,7 +796,7 @@ function user_suspended( $id )
 function user_delete( $id )
 {
 	$dbh = db_connect();
-	$q = "DELETE FROM Users WHERE ID = '$id'";
+	$q = "DELETE FROM Users WHERE ID = " . $id;
 	$result = mysql_fetch_row(db_query($q, $dbh));
 	return;
 }
@@ -808,7 +808,7 @@ function user_delete( $id )
 function user_is_privileged( $id )
 {
 	$dbh = db_connect();
-	$q = "SELECT AccountTypeID FROM Users WHERE ID = '$id'";
+	$q = "SELECT AccountTypeID FROM Users WHERE ID = " . $id;
 	$result = mysql_fetch_row(db_query($q, $dbh));
 	if( $result[0] > 1) {
 		return $result[0];
diff --git a/web/lib/aur.inc b/web/lib/aur.inc
index 744b31e..66ae1c2 100644
--- a/web/lib/aur.inc
+++ b/web/lib/aur.inc
@@ -478,7 +478,7 @@ function mkurl($append) {
 function get_salt($user_id)
 {
 	$dbh = db_connect();
-	$salt_q = "SELECT Salt FROM Users WHERE ID = '$user_id'";
+	$salt_q = "SELECT Salt FROM Users WHERE ID = " . $user_id;
 	$salt_result = mysql_fetch_row(db_query($salt_q, $dbh));
 	return $salt_result[0];
 }
@@ -488,8 +488,8 @@ function save_salt($user_id, $passwd)
 	$dbh = db_connect();
 	$salt = generate_salt();
 	$hash = salted_hash($passwd, $salt);
-	$salting_q = "UPDATE Users SET Salt = '$salt'" .
-		", Passwd = '$hash' WHERE ID = '$user_id'";
+	$salting_q = "UPDATE Users SET Salt = '" . $salt . "', " .
+		"Passwd = '" . $hash . "' WHERE ID = " . $user_id;
 	return db_query($salting_q, $dbh);
 }
 
-- 
cgit v1.2.3-70-g09d2