ChangeSet ID: 30850 CVSROOT: /opt/cvs-commit Module name: appdb Changes by: wineowner@winehq.org 2007/01/03 18:43:58
Modified files: admin : adminAppQueue.php
Log message: Alexander Nicolaysen Sørnes alex@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']);