appdb/admin adminAppQueue.php

WineHQ wineowner at wine.codeweavers.com
Wed Jan 3 18:43:58 CST 2007


ChangeSet ID:	30850
CVSROOT:	/opt/cvs-commit
Module name:	appdb
Changes by:	wineowner at winehq.org	2007/01/03 18:43:58

Modified files:
	admin          : adminAppQueue.php 

Log message:
	Alexander Nicolaysen Sørnes <alex at thehandofagony.com>
	If the admin tries to view an application that does not exist, print an error
	and exit instead of fetching useless data as though there actually was an
	application.

Patch: http://cvs.winehq.org/patch.py?id=30850

Old revision  New revision  Changes     Path
 1.68          1.69          +4 -0       appdb/admin/adminAppQueue.php

Index: appdb/admin/adminAppQueue.php
diff -u -p appdb/admin/adminAppQueue.php:1.68 appdb/admin/adminAppQueue.php:1.69
--- appdb/admin/adminAppQueue.php:1.68	4 Jan 2007  0:43:58 -0000
+++ appdb/admin/adminAppQueue.php	4 Jan 2007  0:43:58 -0000
@@ -145,6 +145,10 @@ if ($aClean['sSub'])
 
         $oApp = new Application($aClean['iAppId']);
 
+        /* Check that the application actually exists */
+        if(!$oApp->iAppId)
+            util_show_error_page_and_exit("There is no application with that ID.");
+
         // if we are processing a queued application there MUST be an implicitly queued 
         // version to go along with it.  
         $hResult = query_parameters("SELECT versionId from appVersion where appId='?';", $aClean['iAppId']);



More information about the wine-cvs mailing list