From 44a0f3609608f75b038959aef5241df708beae9f Mon Sep 17 00:00:00 2001 From: Sebastian Mendel Date: Tue, 11 Apr 2006 14:33:17 +0000 Subject: use PMA_getenv() --- server_privileges.php | 1296 ++++++++++++++++++++++++++----------------------- 1 file changed, 688 insertions(+), 608 deletions(-) (limited to 'server_privileges.php') diff --git a/server_privileges.php b/server_privileges.php index 4437698603..8f014b42a0 100644 --- a/server_privileges.php +++ b/server_privileges.php @@ -2,48 +2,107 @@ /* $Id$ */ // vim: expandtab sw=4 ts=4 sts=4: -require_once('./libraries/common.lib.php'); +if (! defined('PMA_NO_VARIABLES_IMPORT')) { + define('PMA_NO_VARIABLES_IMPORT', true); +} +require_once './libraries/common.lib.php'; /** * Does the common work */ $js_to_run = 'server_privileges.js'; -require('./libraries/server_common.inc.php'); +require './libraries/server_common.inc.php'; + + +/** + * Checks if the user is allowed to do what he tries to... + */ +if (!$is_superuser) { + require './libraries/server_links.inc.php'; + echo '

' . "\n" + . ($GLOBALS['cfg']['MainPageIconic'] ? '' : '') + . $GLOBALS['strPrivileges'] . "\n" + . '

' . "\n" + . $GLOBALS['strNoPrivileges'] . "\n"; + require_once './libraries/footer.inc.php'; +} /** * Checks if a dropdown box has been used for selecting a database / table */ -if (isset($pred_dbname) && strlen($pred_dbname)) { - $dbname = $pred_dbname; - unset($pred_dbname); +if (isset($_REQUEST['pred_dbname']) && strlen($_REQUEST['pred_dbname'])) { + $dbname = $_REQUEST['pred_dbname']; + unset($_REQUEST['pred_dbname']); +} elseif (isset($_REQUEST['dbname']) && strlen($_REQUEST['dbname'])) { + $dbname = $_REQUEST['dbname']; + unset($_REQUEST['dbname']); +} else { + $dbname = ''; } -if (isset($pred_tablename) && strlen($pred_tablename)) { - $tablename = $pred_tablename; - unset($pred_tablename); + +if (isset($_REQUEST['pred_tablename']) && strlen($_REQUEST['pred_tablename'])) { + $tablename = $_REQUEST['pred_tablename']; + unset($_REQUEST['pred_tablename']); +} else { + $tablename = ''; } -// check if given $dbanem is a wildcard or not -if ( isset( $dbname ) ) { - //if ( preg_match( '/\\\\(?:_|%)/i', $dbname ) ) { - if ( preg_match( '/(?' . "\n" - . ($GLOBALS['cfg']['MainPageIconic'] ? '' : '') - . $GLOBALS['strPrivileges'] . "\n" - . '' . "\n" - . $GLOBALS['strNoPrivileges'] . "\n"; - require_once('./libraries/footer.inc.php'); +function PMA_getTablePrivilegs() +{ + $av_grants = PMA_DBI_fetch_value( + 'SHOW COLUMNS FROM `mysql`.`tables_priv` LIKE \'Table_priv\';', + 0, 'Type'); + return explode("','", substr($av_grants, 5, -2)); +} + +/** + * void PMA_addUSerTablePrivs(&$user_privs) + * + * adds user table privs to given user privilegs array $user_privs + * + * @param array &$user_privs user privilegs + */ +function PMA_addUSerTablePrivs(&$user_privs) +{ + if (empty($user_privs) || ! isset($user_privs['Table_priv'])) { + return; + } + + $av_grants = PMA_getTablePrivilegs(); + $users_grants = explode(',', $user_privs['Table_priv']); + + foreach ($av_grants as $current_grant) { + $user_privs[$current_grant . '_priv'] = + in_array($current_grant, $users_grants) ? 'Y' : 'N'; + } + + unset($user_privs['Table_priv']); } /** @@ -52,9 +111,10 @@ if (!$is_superuser) { * @param string the user's initial * @return string the generated condition */ -function PMA_RangeOfUsers($initial = '') { -// strtolower() is used because the User field -// might be BINARY, so LIKE would be case sensitive +function PMA_RangeOfUsers($initial = '') +{ + // strtolower() is used because the User field + // might be BINARY, so LIKE would be case sensitive if (!empty($initial)) { $ret = " WHERE " . PMA_convert_using('User') . " LIKE " . PMA_convert_using($initial . '%', 'quoted') @@ -76,7 +136,7 @@ function PMA_RangeOfUsers($initial = '') { * * @return array */ -function PMA_extractPrivInfo($row = '', $enableHTML = FALSE) +function PMA_extractPrivInfo($row = '', $enableHTML = false) { $grants = array( array('Select_priv', 'SELECT', $GLOBALS['strPrivDescSelect']), @@ -114,24 +174,10 @@ function PMA_extractPrivInfo($row = '', $enableHTML = FALSE) $grants[] = array('Execute_priv', 'EXECUTE', $GLOBALS['strPrivDescExecute5']); } - if (!empty($row) && isset($row['Table_priv'])) { - $res = PMA_DBI_query( - 'SHOW COLUMNS FROM `mysql`.`tables_priv` LIKE \'Table_priv\';', - $GLOBALS['userlink'] ); - $row1 = PMA_DBI_fetch_assoc($res); - PMA_DBI_free_result($res); - $av_grants = explode ('\',\'', substr($row1['Type'], 5, strlen($row1['Type']) - 7)); - unset($row1); - $users_grants = explode(',', $row['Table_priv']); - foreach ($av_grants as $current_grant) { - $row[$current_grant . '_priv'] = in_array($current_grant, $users_grants) ? 'Y' : 'N'; - } - unset($current_grant); - unset($av_grants); - unset($users_grants); - } + PMA_addUSerTablePrivs($row); + $privs = array(); - $allPrivileges = TRUE; + $allPrivileges = true; foreach ($grants as $current_grant) { if ((!empty($row) && isset($row[$current_grant[0]])) || (empty($row) && isset($GLOBALS[$current_grant[0]]))) { if ((!empty($row) && $row[$current_grant[0]] == 'Y') || (empty($row) && ($GLOBALS[$current_grant[0]] == 'Y' || (is_array($GLOBALS[$current_grant[0]]) && count($GLOBALS[$current_grant[0]]) == $GLOBALS['column_count'] && empty($GLOBALS[$current_grant[0] . '_none']))))) { @@ -148,7 +194,7 @@ function PMA_extractPrivInfo($row = '', $enableHTML = FALSE) } $privs[] = $priv_string . ' (`' . join('`, `', $GLOBALS[$current_grant[0]]) . '`)'; } else { - $allPrivileges = FALSE; + $allPrivileges = false; } } } @@ -172,27 +218,36 @@ function PMA_extractPrivInfo($row = '', $enableHTML = FALSE) /** * Displays on which column(s) a table-specific privilege is granted */ -function PMA_display_column_privs($spaces, $columns, $row, $name_for_select, $priv_for_header, $name, $name_for_dfn, $name_for_current) { - - echo $spaces . '
' . "\n" - . $spaces . ' ' . "\n" - . $spaces . ' ' . "\n"; + + foreach ($columns as $current_column => $current_column_privileges) { + echo $spaces . ' ' . "\n"; + } - echo $spaces . ' ' . "\n" - . $spaces . ' ' . $GLOBALS['strOr'] . '' . "\n" - . $spaces . ' ' . "\n" - . $spaces . '
' . "\n"; + echo $spaces . ' ' . "\n" + . $spaces . ' ' . $GLOBALS['strOr'] . '' . "\n" + . $spaces . ' ' . "\n" + . $spaces . '' . "\n"; } // end function /** @@ -208,54 +263,50 @@ function PMA_display_column_privs($spaces, $columns, $row, $name_for_select, $pr * * @return void */ -function PMA_displayPrivTable($db = '*', $table = '*', $submit = TRUE, $indent = 0) +function PMA_displayPrivTable($db = '*', $table = '*', $submit = true, $indent = 0) { if ($db == '*') { $table = '*'; } - $spaces = str_repeat( ' ', $indent ); + $spaces = str_repeat(' ', $indent); - if (isset($GLOBALS['username'])) { - $username = $GLOBALS['username']; - $hostname = $GLOBALS['hostname']; + if (isset($_REQUEST['username'])) { if ($db == '*') { $sql_query = 'SELECT * FROM `mysql`.`user`' .' WHERE ' . PMA_convert_using('User') - .' = ' . PMA_convert_using(PMA_sqlAddslashes($username), 'quoted') + .' = ' . PMA_convert_using(PMA_sqlAddslashes($_REQUEST['username']), 'quoted') .' AND ' . PMA_convert_using('Host') - .' = ' . PMA_convert_using($hostname, 'quoted') . ';'; + .' = ' . PMA_convert_using($_REQUEST['hostname'], 'quoted') . ';'; } elseif ($table == '*') { $sql_query = 'SELECT * FROM `mysql`.`db`' .' WHERE ' . PMA_convert_using('`User`') - .' = ' . PMA_convert_using(PMA_sqlAddslashes($username), 'quoted') + .' = ' . PMA_convert_using(PMA_sqlAddslashes($_REQUEST['username']), 'quoted') .' AND ' . PMA_convert_using('`Host`') - .' = ' . PMA_convert_using($hostname, 'quoted') - .' AND ' . PMA_convert_using( PMA_unescape_mysql_wildcards( $db ), 'quoted' ) - .' LIKE ' . PMA_convert_using( '`Db`' ) . ';'; + .' = ' . PMA_convert_using($_REQUEST['hostname'], 'quoted') + .' AND ' . PMA_convert_using(PMA_unescape_mysql_wildcards($db), 'quoted') + .' LIKE ' . PMA_convert_using('`Db`') . ';'; } else { $sql_query = 'SELECT `Table_priv`' .' FROM `mysql`.`tables_priv`' .' WHERE ' . PMA_convert_using('`User`') - .' = ' . PMA_convert_using(PMA_sqlAddslashes($username), 'quoted') + .' = ' . PMA_convert_using(PMA_sqlAddslashes($_REQUEST['username']), 'quoted') .' AND ' .PMA_convert_using('`Host`') - .' = ' . PMA_convert_using( $hostname, 'quoted' ) + .' = ' . PMA_convert_using($_REQUEST['hostname'], 'quoted') .' AND ' .PMA_convert_using('`Db`') - .' = ' . PMA_convert_using( PMA_unescape_mysql_wildcards( $db ), 'quoted' ) + .' = ' . PMA_convert_using(PMA_unescape_mysql_wildcards($db), 'quoted') .' AND ' . PMA_convert_using('`Table_name`') .' = ' . PMA_convert_using($table, 'quoted') . ';'; } - $res = PMA_DBI_query($sql_query); - $row = PMA_DBI_fetch_assoc($res); - PMA_DBI_free_result($res); + $row = PMA_DBI_fetch_single_row($sql_query); } if (empty($row)) { if ($table == '*') { if ($db == '*') { $sql_query = 'SHOW COLUMNS FROM `mysql`.`user`;'; - } elseif ($table == '*') { + } else { $sql_query = 'SHOW COLUMNS FROM `mysql`.`db`;'; } $res = PMA_DBI_query($sql_query); @@ -272,35 +323,18 @@ function PMA_displayPrivTable($db = '*', $table = '*', $submit = TRUE, $indent = } } if (isset($row['Table_priv'])) { - $res = PMA_DBI_query( - 'SHOW COLUMNS FROM `mysql`.`tables_priv` LIKE \'Table_priv\';', - $GLOBALS['userlink'] ); - // note: in MySQL 5.0.3 we get "Create View', 'Show view'; - // the View for Create is spelled with uppercase V - // the view for Show is spelled with lowercase v - // and there is a space between the words - - $row1 = PMA_DBI_fetch_assoc($res); - PMA_DBI_free_result($res); - $av_grants = explode ('\',\'', substr($row1['Type'], strpos($row1['Type'], '(') + 2, strpos($row1['Type'], ')') - strpos($row1['Type'], '(') - 3)); - unset($res, $row1); - $users_grants = explode(',', $row['Table_priv']); - - foreach ($av_grants as $current_grant) { - $row[$current_grant . '_priv'] = in_array($current_grant, $users_grants) ? 'Y' : 'N'; - } - unset($row['Table_priv'], $current_grant, $av_grants, $users_grants); + PMA_addUSerTablePrivs($row); // get collumns - $res = PMA_DBI_try_query('SHOW COLUMNS FROM `' . PMA_unescape_mysql_wildcards( $db ) . '`.`' . $table . '`;'); + $res = PMA_DBI_try_query('SHOW COLUMNS FROM `' . PMA_unescape_mysql_wildcards($db) . '`.`' . $table . '`;'); $columns = array(); - if ( $res ) { + if ($res) { while ($row1 = PMA_DBI_fetch_row($res)) { $columns[$row1[0]] = array( - 'Select' => FALSE, - 'Insert' => FALSE, - 'Update' => FALSE, - 'References' => FALSE + 'Select' => false, + 'Insert' => false, + 'Update' => false, + 'References' => false ); } PMA_DBI_free_result($res); @@ -308,27 +342,27 @@ function PMA_displayPrivTable($db = '*', $table = '*', $submit = TRUE, $indent = unset($res, $row1); } // t a b l e - s p e c i f i c p r i v i l e g e s - if ( ! empty( $columns ) ) { + if (! empty($columns)) { $res = PMA_DBI_query( 'SELECT `Column_name`, `Column_priv`' .' FROM `mysql`.`columns_priv`' .' WHERE ' . PMA_convert_using('`User`') - .' = ' . PMA_convert_using(PMA_sqlAddslashes($username), 'quoted') + .' = ' . PMA_convert_using(PMA_sqlAddslashes($_REQUEST['username']), 'quoted') .' AND ' . PMA_convert_using('`Host`') - .' = ' . PMA_convert_using($hostname, 'quoted') + .' = ' . PMA_convert_using($_REQUEST['hostname'], 'quoted') .' AND ' . PMA_convert_using('`Db`') - .' = ' . PMA_convert_using( PMA_unescape_mysql_wildcards( $db ), 'quoted') + .' = ' . PMA_convert_using(PMA_unescape_mysql_wildcards($db), 'quoted') .' AND ' . PMA_convert_using('`Table_name`') .' = ' . PMA_convert_using($table, 'quoted') . ';'); while ($row1 = PMA_DBI_fetch_row($res)) { $row1[1] = explode(',', $row1[1]); foreach ($row1[1] as $current) { - $columns[$row1[0]][$current] = TRUE; + $columns[$row1[0]][$current] = true; } } PMA_DBI_free_result($res); - unset( $res, $row1, $current ); + unset($res, $row1, $current); echo $spaces . '' . "\n" . $spaces . '' . "\n" @@ -476,7 +510,7 @@ function PMA_displayPrivTable($db = '*', $table = '*', $submit = TRUE, $indent = . $spaces . '

' . $GLOBALS['strEnglishPrivileges'] . '

' . "\n" . $spaces . '
' . "\n" . $spaces . ' ' . $GLOBALS['strData'] . '' . "\n"; - foreach ( $privTable[0] as $priv ) + foreach ($privTable[0] as $priv) { echo $spaces . '
' . "\n" . $spaces . ' ' . "\n" @@ -486,7 +520,7 @@ function PMA_displayPrivTable($db = '*', $table = '*', $submit = TRUE, $indent = echo $spaces . '
' . "\n" . $spaces . '
' . "\n" . $spaces . ' ' . $GLOBALS['strStructure'] . '' . "\n"; - foreach ( $privTable[1] as $priv ) + foreach ($privTable[1] as $priv) { echo $spaces . '
' . "\n" . $spaces . ' ' . "\n" @@ -496,7 +530,7 @@ function PMA_displayPrivTable($db = '*', $table = '*', $submit = TRUE, $indent = echo $spaces . '
' . "\n" . $spaces . '
' . "\n" . $spaces . ' ' . $GLOBALS['strAdministration'] . '' . "\n"; - foreach ( $privTable[2] as $priv ) + foreach ($privTable[2] as $priv) { echo $spaces . '
' . "\n" . $spaces . ' ' . "\n" @@ -556,10 +590,11 @@ function PMA_displayPrivTable($db = '*', $table = '*', $submit = TRUE, $indent = * * @return void */ -function PMA_displayLoginInformationFields($mode = 'new', $indent = 0 ) { - $spaces = str_repeat( ' ', $indent); +function PMA_displayLoginInformationFields($mode = 'new', $indent = 0) +{ + $spaces = str_repeat(' ', $indent); - // Get user/host name lengths + // Get maximum user/host name column lengths from mysql table structure $fields_info = PMA_DBI_get_fields('mysql', 'user'); $username_length = 16; $hostname_length = 41; @@ -578,10 +613,10 @@ function PMA_displayLoginInformationFields($mode = 'new', $indent = 0 ) { } } } - unset($fields_info); + unset($fields_info, $v, $key, $val); - if ( isset( $GLOBALS['username'] ) && strlen( $GLOBALS['username'] ) === 0 ) { - $GLOBALS['pred_username'] = 'any'; + if (isset($_REQUEST['username']) && strlen($_REQUEST['username']) === 0) { + $_REQUEST['pred_username'] = 'any'; } echo $spaces . '
' . "\n" . $spaces . '' . $GLOBALS['strLoginInformation'] . '' . "\n" @@ -592,11 +627,11 @@ function PMA_displayLoginInformationFields($mode = 'new', $indent = 0 ) { . $spaces . '' . "\n" . $spaces . ' ' . "\n" . $spaces . '' . "\n" - . $spaces . '' . "\n" + . $spaces . '' . "\n" . $spaces . '
' . "\n" . $spaces . '
' . "\n" . $spaces . '' . "\n" . $spaces . '' . "\n" . $spaces . ' ' . "\n" . $spaces . '' . "\n" - . $spaces . '' . "\n" + . $spaces . '' . "\n" . $spaces . '
' . "\n" . $spaces . '
' . "\n" . $spaces . '
' . "\n"; } // end of the 'PMA_displayUserAndHostFields()' function +/** + * string PMA_getPrivCondition(string $username, string $hostname[, + * string $db = null[, string $table = null]]) + * + * returns (simplified): + * WEHRE `User` = '$username' + * AND `Host` = '$hostname' + * [ AND `Db` = '$db'] + * [ AND `Table_name` = '$table']; + * + * @param string $username user name + * @param string $hostname host name + * @return string user host WHERE condition + */ +function PMA_getPrivCondition($username, $hostname, $db = null, $table = null) +{ + $return = + ' WHERE ' . PMA_convert_using('`User`') + .' = ' . PMA_convert_using(PMA_sqlAddslashes($username), 'quoted') + .' AND ' . PMA_convert_using('`Host`') + .' = ' . PMA_convert_using($hostname, 'quoted'); + + if ( null !== $db ) { + $return .= + ' AND ' . PMA_convert_using('`Db`') + .' = ' . PMA_convert_using($db, 'quoted'); + } + if ( null !== $table ) { + $return .= + ' AND ' . PMA_convert_using('`Table_name`') + .' = ' . PMA_convert_using($table, 'quoted'); + } + + return $return . ';'; +} + +/** + * string PMA_getDbTable(string $db, string $table) + * + * escaping a wildcard character in a GRANT is only accepted at the global + * or database level, not at table level; this is why I remove + * the escaping character + * Note: in the phpMyAdmin list of Database-specific privileges, + * we will have for example + * test\_db SELECT (this one is for privileges on a db level) + * test_db USAGE (this one is for table-specific privileges) + * + * It looks curious but reflects the way MySQL works + * + * @param string $db database name + * @param string $table table name + */ +function PMA_getDbTable($db, $table) +{ + if (strlen($db)) { + if (strlen($table)) { + $db_and_table = PMA_backquote(PMA_unescape_mysql_wildcards($db)); + $db_and_table .= '.' . PMA_backquote($table); + } else { + $db_and_table = PMA_backquote($db) . '.*'; + } + } else { + $db_and_table = '*.*'; + } + + return $db_and_table; +} + +/** + * returns WITH privileg options + * + * @param array $privs privilegs to include + * @return string WITH privilegs + */ +function PMA_checkWithPrivs(&$privs) +{ + $with = ''; + + if (isset($privs['Grant_priv']) && $privs['Grant_priv'] == 'Y') { + $with .= ' GRANT OPTION'; + } + + if (PMA_MYSQL_INT_VERSION >= 40002) { + if (isset($privs['max_questions'])) { + // avoid negative values + $privs['max_questions'] = + max(0, (int) $privs['max_questions']); + $with .= ' MAX_QUERIES_PER_HOUR ' . $privs['max_questions']; + } + if (isset($privs['max_connections'])) { + $privs['max_connections'] = + max(0, (int) $privs['max_connections']); + $with .= ' MAX_CONNECTIONS_PER_HOUR ' . $privs['max_connections']; + } + if (isset($privs['max_updates'])) { + $privs['max_updates'] = + max(0, (int) $privs['max_updates']); + $with .= ' MAX_UPDATES_PER_HOUR ' . $privs['max_updates']; + } + } + + if (PMA_MYSQL_INT_VERSION >= 50003) { + if (isset($privs['max_user_connections'])) { + $privs['max_user_connections'] = + max(0, (int) $privs['max_user_connections']); + $with .= ' MAX_USER_CONNECTIONS ' . $privs['max_user_connections']; + } + } + + if (strlen($with)) { + $with = ' WITH' . $with . ' '; + } + + return $with; +} /** * Changes / copies a user, part I + * + * chek if given user exists */ -if (!empty($change_copy)) { - $user_host_condition = - ' WHERE ' . PMA_convert_using('User') - .' = ' . PMA_convert_using(PMA_sqlAddslashes($old_username), 'quoted') - .' AND ' . PMA_convert_using('Host') - .' = ' . PMA_convert_using($old_hostname, 'quoted') . ';'; - $res = PMA_DBI_query('SELECT * FROM `mysql`.`user` ' . $user_host_condition); - if (!$res) { +$mysql_user = array(); +if (!empty($_REQUEST['change_copy'])) { + $user_host_condition = PMA_getPrivCondition($_REQUEST['old_username'], + $_REQUEST['old_hostname']); + $mysql_user = PMA_DBI_fetch_single_row( + 'SELECT * FROM `mysql`.`user` ' . $user_host_condition); + if (!$mysql_user) { + $mysql_user = array(); $message = $GLOBALS['strNoUsersFound']; - unset($change_copy); + unset($_REQUEST['change_copy']); } else { - $row = PMA_DBI_fetch_assoc($res); - extract($row, EXTR_OVERWRITE); // Recent MySQL versions have the field "Password" in mysql.user, // so the previous extract creates $Password but this script // uses $password - if (!isset($password) && isset($Password)) { - $password=$Password; + if (!isset($mysql_user['Password']) && isset($mysql_user['password'])) { + $mysql_user['Password'] =& $mysql_user['password']; } - PMA_DBI_free_result($res); $queries = array(); } } @@ -717,124 +863,91 @@ if (!empty($change_copy)) { * Adds a user * (Changes / copies a user, part II) */ -if (!empty($adduser_submit) || !empty($change_copy)) { +if (!empty($_REQUEST['adduser_submit']) || !empty($_REQUEST['change_copy'])) { unset($sql_query); - if ($pred_username == 'any') { - $username = ''; + if ($_REQUEST['pred_username'] == 'any') { + $_REQUEST['username'] = ''; } - switch ($pred_hostname) { + switch ($_REQUEST['pred_hostname']) { case 'any': - $hostname = '%'; + $_REQUEST['hostname'] = '%'; break; case 'localhost': - $hostname = 'localhost'; + $_REQUEST['hostname'] = 'localhost'; break; case 'hosttable': - $hostname = ''; + $_REQUEST['hostname'] = ''; break; case 'thishost': - $res = PMA_DBI_query('SELECT USER();'); - $row = PMA_DBI_fetch_row($res); - PMA_DBI_free_result($res); - unset($res); - $hostname = substr($row[0], (strrpos($row[0], '@') + 1)); - unset($row); + $_REQUEST['hostname'] = PMA_DBI_fetch_value('SELECT USER();'); + $_REQUEST['hostname'] = substr($_REQUEST['hostname'], + (strrpos($_REQUEST['hostname'], '@') + 1)); break; } - $res = PMA_DBI_query( - 'SELECT \'foo\' FROM `mysql`.`user`' - .' WHERE ' . PMA_convert_using('User') - .' = ' . PMA_convert_using(PMA_sqlAddslashes($username), 'quoted') - .' AND ' . PMA_convert_using('Host') - .' = ' . PMA_convert_using($hostname, 'quoted') . ';', - null, PMA_DBI_QUERY_STORE); - if (PMA_DBI_num_rows($res) == 1) { - PMA_DBI_free_result($res); - $message = sprintf($GLOBALS['strUserAlreadyExists'], '[i]\'' . $username . '\'@\'' . $hostname . '\'[/i]'); - $adduser = 1; + $user_exists = PMA_DBI_fetch_value( + 'SELECT COUNT(*) FROM `mysql`.`user`' + . PMA_getPrivCondition($_REQUEST['username'], $_REQUEST['hostname'])); + if ($user_exists) { + $message = sprintf($GLOBALS['strUserAlreadyExists'], + "[i]'" . $_REQUEST['username'] . "'@'" . $_REQUEST['hostname'] . "'[/i]"); + // display add user page again + $_REQUEST['adduser'] = true; } else { - PMA_DBI_free_result($res); - if (50002 <= PMA_MYSQL_INT_VERSION) { // MySQL 5 requires CREATE USER before any GRANT on this user can done - $create_user_real = 'CREATE USER \'' . PMA_sqlAddslashes($username) . '\'@\'' . $hostname . '\''; + $create_user_real = 'CREATE USER \'' + . PMA_sqlAddslashes($_REQUEST['username']) . '\'@\'' + . $_REQUEST['hostname'] . '\''; } $real_sql_query = 'GRANT ' . join(', ', PMA_extractPrivInfo()) . ' ON *.* TO \'' - . PMA_sqlAddslashes($username) . '\'@\'' . $hostname . '\''; - if ($pred_password != 'none' && $pred_password != 'keep') { - $pma_pw_hidden = ''; - for ($i = 0; $i < strlen($pma_pw); $i++) { - $pma_pw_hidden .= '*'; - } - $sql_query = $real_sql_query . ' IDENTIFIED BY \'' . $pma_pw_hidden . '\''; - $real_sql_query .= ' IDENTIFIED BY \'' . $pma_pw . '\''; - if ( isset( $create_user_real ) ) { - $create_user_show = $create_user_real . ' IDENTIFIED BY \'' . $pma_pw_hidden . '\''; - $create_user_real .= ' IDENTIFIED BY \'' . $pma_pw . '\''; + . PMA_sqlAddslashes($_REQUEST['username']) . '\'@\'' + . $_REQUEST['hostname'] . '\''; + if ($_REQUEST['pred_password'] != 'none' + && $_REQUEST['pred_password'] != 'keep') { + $sql_query = $real_sql_query . ' IDENTIFIED BY \'*****\''; + $real_sql_query .= ' IDENTIFIED BY \'' . $_REQUEST['pma_pw'] . '\''; + if (isset($create_user_real)) { + $create_user_show = $create_user_real . ' IDENTIFIED BY \'*****\''; + $create_user_real .= ' IDENTIFIED BY \'' . $_REQUEST['pma_pw'] . '\''; } } else { - if ($pred_password == 'keep' && !empty($password)) { - $real_sql_query .= ' IDENTIFIED BY PASSWORD \'' . $password . '\''; - if ( isset( $create_user_real ) ) { - $create_user_real .= ' IDENTIFIED BY PASSWORD \'' . $password . '\''; + if ($_REQUEST['pred_password'] == 'keep' + && !empty($mysql_user['Password'])) { + $real_sql_query .= ' IDENTIFIED BY PASSWORD \'' + . $mysql_user['Password'] . '\''; + if (isset($create_user_real)) { + $create_user_real .= ' IDENTIFIED BY PASSWORD \'' + . $mysql_user['Password'] . '\''; } } $sql_query = $real_sql_query; - if ( isset( $create_user_real ) ) { + if (isset($create_user_real)) { $create_user_show = $create_user_real; } } - // FIXME: similar code appears twice in this script - if ((isset($Grant_priv) && $Grant_priv == 'Y') || (PMA_MYSQL_INT_VERSION >= 40002 && (isset($max_questions) || isset($max_connections) || isset($max_updates) || isset($max_user_connections)))) { - $real_sql_query .= 'WITH'; - $sql_query .= 'WITH'; - if (isset($Grant_priv) && $Grant_priv == 'Y') { - $real_sql_query .= ' GRANT OPTION'; - $sql_query .= ' GRANT OPTION'; - } - if (PMA_MYSQL_INT_VERSION >= 40002) { - if (isset($max_questions)) { - // avoid negative values - $max_questions = max(0, (int)$max_questions); - $real_sql_query .= ' MAX_QUERIES_PER_HOUR ' . $max_questions; - $sql_query .= ' MAX_QUERIES_PER_HOUR ' . $max_questions; - } - if (isset($max_connections)) { - $max_connections = max(0, (int)$max_connections); - $real_sql_query .= ' MAX_CONNECTIONS_PER_HOUR ' . $max_connections; - $sql_query .= ' MAX_CONNECTIONS_PER_HOUR ' . $max_connections; - } - if (isset($max_updates)) { - $max_updates = max(0, (int)$max_updates); - $real_sql_query .= ' MAX_UPDATES_PER_HOUR ' . $max_updates; - $sql_query .= ' MAX_UPDATES_PER_HOUR ' . $max_updates; - } - } - if (PMA_MYSQL_INT_VERSION >= 50003) { - if (isset($max_user_connections)) { - $max_user_connections = max(0, (int)$max_user_connections); - $real_sql_query .= ' MAX_USER_CONNECTIONS ' . $max_user_connections; - $sql_query .= ' MAX_USER_CONNECTIONS ' . $max_user_connections; - } - } - } - if ( isset( $create_user_real ) ) { + + $with = PMA_checkWithPrivs($mysql_user); + $real_sql_query .= $with . ';'; + $sql_query .= $with . ';'; + if (isset($create_user_real)) { $create_user_real .= ';'; $create_user_show .= ';'; } - $real_sql_query .= ';'; - $sql_query .= ';'; - if (empty($change_copy)) { - if ( isset( $create_user_real ) ) { - PMA_DBI_try_query($create_user_real) or PMA_mysqlDie(PMA_DBI_getError(), $create_user_show); + unset($with); + + if (empty($_REQUEST['change_copy'])) { + if (isset($create_user_real)) { + PMA_DBI_try_query($create_user_real) + or PMA_mysqlDie(PMA_DBI_getError(), $create_user_show); $sql_query = $create_user_show . $sql_query; } - PMA_DBI_try_query($real_sql_query) or PMA_mysqlDie(PMA_DBI_getError(), $sql_query); + PMA_DBI_try_query($real_sql_query) + or PMA_mysqlDie(PMA_DBI_getError(), $sql_query); $message = $GLOBALS['strAddUserMessage']; } else { - if ( isset( $create_user_real ) ) { + if (isset($create_user_real)) { $queries[] = $create_user_real; } $queries[] = $real_sql_query; @@ -842,12 +955,15 @@ if (!empty($adduser_submit) || !empty($change_copy)) { // $queries_for_display, at the same position occupied // by the real query in $queries $tmp_count = count($queries); - if ( isset( $create_user_real ) ) { + if (isset($create_user_real)) { $queries_for_display[$tmp_count - 2] = $create_user_show; } $queries_for_display[$tmp_count - 1] = $sql_query; } - unset($res, $real_sql_query); + unset($real_sql_query); + if (isset($create_user_real)) { + unset($create_user_show, $create_user_real); + } } } @@ -855,39 +971,31 @@ if (!empty($adduser_submit) || !empty($change_copy)) { /** * Changes / copies a user, part III */ -if (!empty($change_copy)) { - $user_host_condition = - ' WHERE ' . PMA_convert_using('User') - .' = ' . PMA_convert_using(PMA_sqlAddslashes($old_username), 'quoted') - .' AND ' . PMA_convert_using('Host') - .' = ' . PMA_convert_using($old_hostname, 'quoted') . ';'; - $res = PMA_DBI_query('SELECT * FROM `mysql`.`db`' . $user_host_condition ); +if (!empty($_REQUEST['change_copy'])) { + $user_host_condition = PMA_getPrivCondition($_REQUEST['old_username'], + $_REQUEST['old_hostname']); + $res = PMA_DBI_query('SELECT * FROM `mysql`.`db`' . $user_host_condition); while ($row = PMA_DBI_fetch_assoc($res)) { $queries[] = 'GRANT ' . join(', ', PMA_extractPrivInfo($row)) - .' ON `' . $row['Db'] . '`.*' - .' TO \'' . PMA_sqlAddslashes($username) . '\'@\'' . $hostname . '\'' - . ( $row['Grant_priv'] == 'Y' ? ' WITH GRANT OPTION;' : ';' ); + . ' ON `' . $row['Db'] . '`.* TO \'' + . PMA_sqlAddslashes($_REQUEST['username']) . "'@'" + . $_REQUEST['hostname'] . "'" + . ($row['Grant_priv'] == 'Y' ? ' WITH GRANT OPTION;' : ';'); } PMA_DBI_free_result($res); + $res = PMA_DBI_query( 'SELECT `Db`, `Table_name`, `Table_priv`' .' FROM `mysql`.`tables_priv`' . $user_host_condition, - $GLOBALS['userlink'], PMA_DBI_QUERY_STORE ); + $GLOBALS['userlink'], PMA_DBI_QUERY_STORE); while ($row = PMA_DBI_fetch_assoc($res)) { $res2 = PMA_DBI_QUERY( 'SELECT `Column_name`, `Column_priv`' - .' FROM `mysql`.`columns_priv`' - .' WHERE ' . PMA_convert_using('User') - .' = ' . PMA_convert_using(PMA_sqlAddslashes($old_username), 'quoted') - .' AND ' . PMA_convert_using('`Host`') - .' = ' . PMA_convert_using($old_hostname, 'quoted') - .' AND ' . PMA_convert_using('`Db`') - .' = ' . PMA_convert_using($row['Db'], 'quoted') - .' AND ' . PMA_convert_using('`Table_name`') - .' = ' . PMA_convert_using($row['Table_name'], 'quoted') - .';', + . ' FROM `mysql`.`columns_priv`' + . PMA_getPrivCondition($_REQUEST['old_username'], + $_REQUEST['old_hostname'], $row['Db'], $row['Table_name']), null, PMA_DBI_QUERY_STORE); $tmp_privs1 = PMA_extractPrivInfo($row); @@ -898,6 +1006,7 @@ if (!empty($change_copy)) { 'References' => array() ); + $tmp_array = array(); while ($row2 = PMA_DBI_fetch_assoc($res2)) { $tmp_array = explode(',', $row2['Column_priv']); if (in_array('Select', $tmp_array)) { @@ -912,173 +1021,121 @@ if (!empty($change_copy)) { if (in_array('References', $tmp_array)) { $tmp_privs2['References'][] = $row2['Column_name']; } - unset($tmp_array); } - if (count($tmp_privs2['Select']) > 0 && !in_array('SELECT', $tmp_privs1)) { + PMA_DBI_free_result($res2); + + if (count($tmp_privs2['Select']) && !in_array('SELECT', $tmp_privs1)) { $tmp_privs1[] = 'SELECT (`' . join('`, `', $tmp_privs2['Select']) . '`)'; } - if (count($tmp_privs2['Insert']) > 0 && !in_array('INSERT', $tmp_privs1)) { + if (count($tmp_privs2['Insert']) && !in_array('INSERT', $tmp_privs1)) { $tmp_privs1[] = 'INSERT (`' . join('`, `', $tmp_privs2['Insert']) . '`)'; } - if (count($tmp_privs2['Update']) > 0 && !in_array('UPDATE', $tmp_privs1)) { + if (count($tmp_privs2['Update']) && !in_array('UPDATE', $tmp_privs1)) { $tmp_privs1[] = 'UPDATE (`' . join('`, `', $tmp_privs2['Update']) . '`)'; } - if (count($tmp_privs2['References']) > 0 && !in_array('REFERENCES', $tmp_privs1)) { + if (count($tmp_privs2['References']) && !in_array('REFERENCES', $tmp_privs1)) { $tmp_privs1[] = 'REFERENCES (`' . join('`, `', $tmp_privs2['References']) . '`)'; } - unset($tmp_privs2); $queries[] = 'GRANT ' . join(', ', $tmp_privs1) - . ' ON `' . $row['Db'] . '`.`' . $row['Table_name'] - . '` TO \'' . PMA_sqlAddslashes($username) . '\'@\'' . $hostname . '\'' + . ' ON `' . $row['Db'] . '`.`' . $row['Table_name'] . '` TO \'' + . PMA_sqlAddslashes($_REQUEST['username']) . '\'@\'' + . $_REQUEST['hostname'] . '\'' . (in_array('Grant', explode(',', $row['Table_priv'])) ? ' WITH GRANT OPTION;' : ';'); } + PMA_DBI_free_result($res); + unset($res, $res2, $row2, $tmp_array, $tmp_privs2, $tmp_privs1, $row, + $user_host_condition); } /** * Updates privileges */ -if (!empty($update_privs)) { - // escaping a wildcard character in a GRANT is only accepted at the global - // or database level, not at table level; this is why I remove - // the escaping character - // Note: in the phpMyAdmin list of Database-specific privileges, - // we will have for example - // test\_db SELECT (this one is for privileges on a db level) - // test_db USAGE (this one is for table-specific privileges) - // - // It looks curious but reflects the way MySQL works - - if (! isset($dbname) || ! strlen($dbname)) { - $db_and_table = '*.*'; - } else { - if ( isset( $tablename ) && strlen($tablename) ) { - $db_and_table = PMA_backquote( PMA_unescape_mysql_wildcards( $dbname ) ) . '.'; - $db_and_table .= PMA_backquote( $tablename ); - } else { - $db_and_table = PMA_backquote( $dbname ) . '.'; - $db_and_table .= '*'; - } - } +if (!empty($_REQUEST['update_privs'])) { + $db_and_table = PMA_getDbTable($dbname, $tablename); $sql_query0 = 'REVOKE ALL PRIVILEGES ON ' . $db_and_table - . ' FROM \'' . PMA_sqlAddslashes($username) . '\'@\'' . $hostname . '\';'; - if (!isset($Grant_priv) || $Grant_priv != 'Y') { + . ' FROM \'' . PMA_sqlAddslashes($_REQUEST['username']) . '\'@\'' + . $_REQUEST['hostname'] . '\';'; + if (!isset($mysql_user['Grant_priv']) || $mysql_user['Grant_priv'] != 'Y') { $sql_query1 = 'REVOKE GRANT OPTION ON ' . $db_and_table - . ' FROM \'' . PMA_sqlAddslashes($username) . '\'@\'' . $hostname . '\';'; + . ' FROM \'' . PMA_sqlAddslashes($_REQUEST['username']) . '\'@\'' + . $_REQUEST['hostname'] . '\';'; } $sql_query2 = 'GRANT ' . join(', ', PMA_extractPrivInfo()) . ' ON ' . $db_and_table - . ' TO \'' . PMA_sqlAddslashes($username) . '\'@\'' . $hostname . '\''; - - // FIXME: similar code appears twice in this script - if ( ( isset($Grant_priv) && $Grant_priv == 'Y') - || ( ( ! isset($dbname) || ! strlen($dbname) ) && PMA_MYSQL_INT_VERSION >= 40002 - && ( isset($max_questions) || isset($max_connections) - || isset($max_updates) || isset($max_user_connections)))) - { - $sql_query2 .= 'WITH'; - if (isset($Grant_priv) && $Grant_priv == 'Y') { - $sql_query2 .= ' GRANT OPTION'; - } - if (PMA_MYSQL_INT_VERSION >= 40002) { - if (isset($max_questions)) { - $max_questions = max(0, (int)$max_questions); - $sql_query2 .= ' MAX_QUERIES_PER_HOUR ' . $max_questions; - } - if (isset($max_connections)) { - $max_connections = max(0, (int)$max_connections); - $sql_query2 .= ' MAX_CONNECTIONS_PER_HOUR ' . $max_connections; - } - if (isset($max_updates)) { - $max_updates = max(0, (int)$max_updates); - $sql_query2 .= ' MAX_UPDATES_PER_HOUR ' . $max_updates; - } - } - if (PMA_MYSQL_INT_VERSION >= 50003) { - if (isset($max_user_connections)) { - $max_user_connections = max(0, (int)$max_user_connections); - $sql_query2 .= ' MAX_USER_CONNECTIONS ' . $max_user_connections; - } - } - } - $sql_query2 .= ';'; - if (!PMA_DBI_try_query($sql_query0)) { // this query may fail, but this does not matter :o) + . ' TO \'' . PMA_sqlAddslashes($_REQUEST['username']) . '\'@\'' + . $_REQUEST['hostname'] . '\''; + + $sql_query2 .= PMA_checkWithPrivs($mysql_user) . ';'; + + if (!PMA_DBI_try_query($sql_query0)) { + // this query may fail, but this does not matter :o) unset($sql_query0); } - if (isset($sql_query1) && !PMA_DBI_try_query($sql_query1)) { // this one may fail, too... + if (isset($sql_query1) && !PMA_DBI_try_query($sql_query1)) { + // this one may fail, too... unset($sql_query1); } PMA_DBI_query($sql_query2); $sql_query = (isset($sql_query0) ? $sql_query0 . ' ' : '') . (isset($sql_query1) ? $sql_query1 . ' ' : '') . $sql_query2; - $message = sprintf($GLOBALS['strUpdatePrivMessage'], '\'' . $username . '\'@\'' . $hostname . '\''); + $message = sprintf($GLOBALS['strUpdatePrivMessage'], '\'' . $_REQUEST['username'] . '\'@\'' . $_REQUEST['hostname'] . '\''); + unset($sql_query2, $db_and_table); } /** * Revokes Privileges */ -if (!empty($revokeall)) { +if (!empty($_REQUEST['revokeall'])) { - if ( ! isset($dbname) || ! strlen($dbname) ) { - $db_and_table = '*.*'; - } else { - if ( ! isset( $tablename ) || ! strlen($tablename) ) { - $db_and_table = PMA_backquote( $dbname ) . '.'; - $db_and_table .= '*'; - } else { - $db_and_table = PMA_backquote( PMA_unescape_mysql_wildcards( $dbname ) ) . '.'; - $db_and_table .= PMA_backquote( $tablename ); - } - } + $db_and_table = PMA_getDbTable($dbname, $tablename); $sql_query0 = 'REVOKE ALL PRIVILEGES ON ' . $db_and_table - . ' FROM \'' . $username . '\'@\'' . $hostname . '\';'; + . ' FROM \'' . $_REQUEST['username'] . '\'@\'' . $_REQUEST['hostname'] . '\';'; $sql_query1 = 'REVOKE GRANT OPTION ON ' . $db_and_table - . ' FROM \'' . $username . '\'@\'' . $hostname . '\';'; + . ' FROM \'' . $_REQUEST['username'] . '\'@\'' . $_REQUEST['hostname'] . '\';'; PMA_DBI_query($sql_query0); if (!PMA_DBI_try_query($sql_query1)) { // this one may fail, too... unset($sql_query1); } $sql_query = $sql_query0 . (isset($sql_query1) ? ' ' . $sql_query1 : ''); - $message = sprintf($GLOBALS['strRevokeMessage'], '\'' . $username . '\'@\'' . $hostname . '\''); - if ( ! isset($tablename) || ! strlen($tablename) ) { - unset($dbname); - } else { + $message = sprintf($GLOBALS['strRevokeMessage'], '\'' . $_REQUEST['username'] . '\'@\'' . $_REQUEST['hostname'] . '\''); + if (strlen($tablename)) { unset($tablename); + } else { + unset($dbname); } + + unset($sql_query0, $db_and_table); } /** * Updates the password */ -if (!empty($change_pw)) { - if ($nopass == 1) { - $sql_query = 'SET PASSWORD FOR \'' . $username . '\'@\'' . $hostname . '\' = \'\';'; +if (!empty($_REQUEST['change_pw'])) { + if ($_REQUEST['$nopass'] == 1) { + $sql_query = 'SET PASSWORD FOR \'' . $_REQUEST['username'] . '\'@\'' . $_REQUEST['hostname'] . '\' = \'\';'; PMA_DBI_query($sql_query); - $message = sprintf($GLOBALS['strPasswordChanged'], '\'' . $username . '\'@\'' . $hostname . '\''); - } elseif (empty($pma_pw) || empty($pma_pw2)) { + $message = sprintf($GLOBALS['strPasswordChanged'], '\'' . $_REQUEST['username'] . '\'@\'' . $_REQUEST['hostname'] . '\''); + } elseif (empty($_REQUEST['pma_pw']) || empty($_REQUEST['pma_pw2'])) { $message = $GLOBALS['strPasswordEmpty']; - } elseif ($pma_pw != $pma_pw2) { + } elseif ($_REQUEST['pma_pw'] != $_REQUEST['pma_pw2']) { $message = $GLOBALS['strPasswordNotSame']; } else { - $hidden_pw = ''; - for ($i = 0; $i < strlen($pma_pw); $i++) { - $hidden_pw .= '*'; - } - $local_query = 'SET PASSWORD FOR \'' . PMA_sqlAddslashes($username) . '\'@\'' . $hostname . '\' = PASSWORD(\'' . PMA_sqlAddslashes($pma_pw) . '\')'; - $sql_query = 'SET PASSWORD FOR \'' . PMA_sqlAddslashes($username) . '\'@\'' . $hostname . '\' = PASSWORD(\'' . $hidden_pw . '\')'; + $local_query = 'SET PASSWORD FOR \'' . PMA_sqlAddslashes($_REQUEST['username']) . '\'@\'' . $_REQUEST['hostname'] . '\' = PASSWORD(\'' . PMA_sqlAddslashes($_REQUEST['pma_pw']) . '\')'; + $sql_query = 'SET PASSWORD FOR \'' . PMA_sqlAddslashes($_REQUEST['username']) . '\'@\'' . $_REQUEST['hostname'] . '\' = PASSWORD(\'*****\')'; PMA_DBI_try_query($local_query) or PMA_mysqlDie(PMA_DBI_getError(), $sql_query); - $message = sprintf($GLOBALS['strPasswordChanged'], '\'' . $username . '\'@\'' . $hostname . '\''); + $message = sprintf($GLOBALS['strPasswordChanged'], '\'' . $_REQUEST['username'] . '\'@\'' . $_REQUEST['hostname'] . '\''); } } @@ -1089,31 +1146,36 @@ if (!empty($change_pw)) { */ $user_host_separator = chr(27); -if (!empty($delete) || (!empty($change_copy) && $mode < 4)) { - if (!empty($change_copy)) { - $selected_usr = array($old_username . $user_host_separator . $old_hostname); +if (!empty($_REQUEST['delete']) || (!empty($_REQUEST['change_copy']) && $_REQUEST['mode'] < 4)) { + if (!empty($_REQUEST['change_copy'])) { + $_REQUEST['selected_usr'] = + array($_REQUEST['old_username'] . $user_host_separator . $_REQUEST['old_hostname']); } else { $queries = array(); } - for ($i = 0; isset($selected_usr[$i]); $i++) { - list($this_user, $this_host) = explode($user_host_separator, $selected_usr[$i]); - $queries[] = '# ' . sprintf($GLOBALS['strDeleting'], '\'' . $this_user . '\'@\'' . $this_host . '\'') . ' ...'; - if ( PMA_MYSQL_INT_VERSION >= 50002 ) { - $queries[] = 'DROP USER \'' . PMA_sqlAddslashes($this_user) . '\'@\'' . $this_host . '\';'; + foreach ($_REQUEST['selected_usr'] as $each_selected_user) { + list($this_user, $this_host) = explode($user_host_separator, $each_selected_user); + $this_user_host = "'" . PMA_sqlAddslashes($this_user) . "'@'" . $this_host . "'"; + $queries[] = '# ' . sprintf($GLOBALS['strDeleting'], $this_user_host) . ' ...'; + if (PMA_MYSQL_INT_VERSION >= 50002) { + $queries[] = 'DROP USER ' . $this_user_host . ';'; } else { - if ($mode == 2) { + if ($_REQUEST['mode'] == 2) { // The SHOW GRANTS query may fail if the user has not been loaded // into memory - $res = PMA_DBI_try_query('SHOW GRANTS FOR \'' . PMA_sqlAddslashes($this_user) . '\'@\'' . $this_host . '\';'); + $res = PMA_DBI_try_query('SHOW GRANTS FOR ' . $this_user_host . ';'); if ($res) { - $queries[] = 'REVOKE ALL PRIVILEGES ON *.* FROM \'' . PMA_sqlAddslashes($this_user) . '\'@\'' . $this_host . '\';'; + $queries[] = 'REVOKE ALL PRIVILEGES ON *.* FROM ' . $this_user_host . ';'; while ($row = PMA_DBI_fetch_row($res)) { - $this_table = substr($row[0], (strpos($row[0], 'ON') + 3), (strpos($row[0], ' TO ') - strpos($row[0], 'ON') - 3)); + $this_table = substr($row[0], (strpos($row[0], 'ON') + 3), + (strpos($row[0], ' TO ') - strpos($row[0], 'ON') - 3)); if ($this_table != '*.*') { - $queries[] = 'REVOKE ALL PRIVILEGES ON ' . $this_table . ' FROM \'' . PMA_sqlAddslashes($this_user) . '\'@\'' . $this_host . '\';'; + $queries[] = 'REVOKE ALL PRIVILEGES ON ' . $this_table + . ' FROM ' . $this_user_host . ';'; if (strpos($row[0], 'WITH GRANT OPTION')) { - $queries[] = 'REVOKE GRANT OPTION ON ' . $this_table . ' FROM \'' . PMA_sqlAddslashes($this_user) . '\'@\'' . $this_host . '\';'; + $queries[] = 'REVOKE GRANT OPTION ON ' . $this_table + . ' FROM ' . $this_user_host . ';'; } } unset($this_table); @@ -1122,19 +1184,16 @@ if (!empty($delete) || (!empty($change_copy) && $mode < 4)) { } unset($res); } - if ( PMA_MYSQL_INT_VERSION >= 40101 ) { - $queries[] = 'DROP USER \'' . PMA_sqlAddslashes($this_user) . '\'@\'' . $this_host . '\';'; + if (PMA_MYSQL_INT_VERSION >= 40101) { + $queries[] = 'DROP USER ' . $this_user_host . ';'; } else { - $queries[] = 'DELETE FROM `mysql`.`user` WHERE ' . PMA_convert_using('User') . ' = ' . PMA_convert_using(PMA_sqlAddslashes($this_user), 'quoted') . ' AND ' . PMA_convert_using('Host') . ' = ' . PMA_convert_using($this_host, 'quoted') . ';'; + $queries[] = 'DELETE FROM `mysql`.`user`' + . PMA_getPrivCondition($this_user, $this_host); } - if ($mode != 2) { + if ($_REQUEST['mode'] != 2) { // If we REVOKE the table grants, we should not need to modify the // `mysql`.`db`, `mysql`.`tables_priv` and `mysql`.`columns_priv` tables manually... - $user_host_condition = - ' WHERE ' . PMA_convert_using('User') - . ' = ' . PMA_convert_using(PMA_sqlAddslashes($this_user), 'quoted') - . ' AND ' . PMA_convert_using('Host') - . ' = ' . PMA_convert_using($this_host, 'quoted') . ';'; + $user_host_condition = PMA_getPrivCondition($this_user, $this_host); $queries[] = 'DELETE FROM `mysql`.`db`' . $user_host_condition; $queries[] = 'DELETE FROM `mysql`.`tables_priv`' . $user_host_condition; $queries[] = 'DELETE FROM `mysql`.`columns_priv`' . $user_host_condition; @@ -1142,16 +1201,16 @@ if (!empty($delete) || (!empty($change_copy) && $mode < 4)) { } if (!empty($drop_users_db)) { $queries[] = 'DROP DATABASE IF EXISTS ' . PMA_backquote($this_user) . ';'; - $GLOBALS['reload'] = TRUE; + $GLOBALS['reload'] = true; PMA_reloadNavigation(); } } - if (empty($change_copy)) { + if (empty($_REQUEST['change_copy'])) { if (empty($queries)) { - $show_error_header = TRUE; + $show_error_header = true; $message = $GLOBALS['strDeleteNoUsersSelected']; } else { - if ($mode == 3) { + if ($_REQUEST['mode'] == 3) { $queries[] = '# ' . $GLOBALS['strReloadingThePrivileges'] . ' ...'; $queries[] = 'FLUSH PRIVILEGES;'; } @@ -1162,8 +1221,8 @@ if (!empty($delete) || (!empty($change_copy) && $mode < 4)) { } $sql_query = join("\n", $queries); $message = $GLOBALS['strUsersDeleted']; + unset($queries); } - unset($queries); } } @@ -1171,7 +1230,7 @@ if (!empty($delete) || (!empty($change_copy) && $mode < 4)) { /** * Changes / copies a user, part V */ -if (!empty($change_copy)) { +if (!empty($_REQUEST['change_copy'])) { $tmp_count = 0; foreach ($queries as $sql_query) { if ($sql_query{0} != '#') { @@ -1192,7 +1251,7 @@ if (!empty($change_copy)) { /** * Reloads the privilege tables into memory */ -if (!empty($flush_privileges)) { +if (!empty($_REQUEST['flush_privileges'])) { $sql_query = 'FLUSH PRIVILEGES;'; PMA_DBI_query($sql_query); $message = $GLOBALS['strPrivilegesReloaded']; @@ -1201,17 +1260,20 @@ if (!empty($flush_privileges)) { /** * Displays the links + * + * $_REQUEST['checkprivs'] come from 'Rights'-tab in database pages + * $_REQUEST['viewing_mode'] come from 'Rights'-tab in database pages */ -if (isset($viewing_mode) && $viewing_mode == 'db') { - $db = $checkprivs; +if (isset($_REQUEST['viewing_mode']) && $_REQUEST['viewing_mode'] == 'db') { + $db = $_REQUEST['checkprivs']; $url_query .= '&goto=db_operations.php'; + $url_params['goto'] = 'db_operations.php'; // Gets the database structure $sub_part = '_structure'; - require('./libraries/db_details_db_info.inc.php'); - echo "\n"; + require './libraries/db_details_db_info.inc.php'; } else { - require('./libraries/server_links.inc.php'); + require './libraries/server_links.inc.php'; } @@ -1223,8 +1285,11 @@ $link_edit = ''; -if ( $GLOBALS['cfg']['PropertiesIconic'] ) { - $link_edit .= '' . $GLOBALS['strEditPrivileges'] . ''; +if ($GLOBALS['cfg']['PropertiesIconic']) { + $link_edit .= '' . $GLOBALS['strEditPrivileges'] . ''; } else { $link_edit .= $GLOBALS['strEditPrivileges']; } @@ -1236,8 +1301,11 @@ $link_revoke = ''; -if ( $GLOBALS['cfg']['PropertiesIconic'] ) { - $link_revoke .= '' . $GLOBALS['strRevoke'] . ''; +if ($GLOBALS['cfg']['PropertiesIconic']) { + $link_revoke .= '' . $GLOBALS['strRevoke'] . ''; } else { $link_revoke .= $GLOBALS['strRevoke']; } @@ -1246,14 +1314,19 @@ $link_revoke .= ''; /** * Displays the page */ -if ( empty( $adduser ) && ( ! isset( $checkprivs ) || ! strlen($checkprivs) ) ) { - if ( ! isset( $username ) ) { +if (empty($_REQUEST['adduser']) && (! isset($_REQUEST['checkprivs']) || ! strlen($_REQUEST['checkprivs']))) { + if (! isset($_REQUEST['username'])) { // No username is given --> display the overview echo '

' . "\n" - . ($GLOBALS['cfg']['MainPageIconic'] ? '' : '') + . ($GLOBALS['cfg']['MainPageIconic'] + ? '' + : '') . $GLOBALS['strUserOverview'] . "\n" . '

' . "\n"; + /** + * @todo why not SELECT * ??? + */ $sql_query = 'SELECT `User`,' . ' `Host`,' . @@ -1287,7 +1360,7 @@ if ( empty( $adduser ) && ( ! isset( $checkprivs ) || ! strlen($checkprivs) ) ) $sql_query .= ' FROM `mysql`.`user`'; - $sql_query .= (isset($initial) ? PMA_RangeOfUsers($initial) : ''); + $sql_query .= (isset($_REQUEST['initial']) ? PMA_RangeOfUsers($_REQUEST['initial']) : ''); $sql_query .= ' ORDER BY `User` ASC, `Host` ASC;'; $res = PMA_DBI_try_query($sql_query, null, PMA_DBI_QUERY_STORE); @@ -1323,9 +1396,11 @@ if ( empty( $adduser ) && ( ! isset( $checkprivs ) || ! strlen($checkprivs) ) ) ); $db_rights_sqls = array(); - foreach ( $tables_to_search_for_users as $table_search_in ) { - if ( in_array( $table_search_in, $tables ) ) { - $db_rights_sqls[] = 'SELECT DISTINCT `User`, `Host` FROM `mysql`.`' . $table_search_in . '` ' . (isset($initial) ? PMA_RangeOfUsers($initial) : ''); + foreach ($tables_to_search_for_users as $table_search_in) { + if (in_array($table_search_in, $tables)) { + $db_rights_sqls[] = + 'SELECT DISTINCT `User`, `Host`' + . ' FROM `mysql`.`' . $table_search_in . '` ' . (isset($_REQUEST['initial']) ? PMA_RangeOfUsers($_REQUEST['initial']) : ''); } } @@ -1334,73 +1409,73 @@ if ( empty( $adduser ) && ( ! isset( $checkprivs ) || ! strlen($checkprivs) ) ) 'Host' => '%', 'Password' => '?', 'Grant_priv' => 'N', - 'privs' => array( 'USAGE' ), + 'privs' => array('USAGE'), ); - // for all initials, even non A-Z - $array_initials = array(); // for the rights $db_rights = array(); - // do not use UNION DISTINCT, as it's not allowed before + // do not use UNION DISTINCT, as it's not allowed before // MySQL 4.0.17, and because "it does nothing" (cf manual) - if ( PMA_MYSQL_INT_VERSION >= 40000 ) { - $db_rights_sql = '(' . implode( ') UNION (', $db_rights_sqls ) . ')' + if (PMA_MYSQL_INT_VERSION >= 40000) { + $db_rights_sql = '(' . implode(') UNION (', $db_rights_sqls) . ')' .' ORDER BY `User` ASC, `Host` ASC'; - $db_rights_result = PMA_DBI_query( $db_rights_sql ); + $db_rights_result = PMA_DBI_query($db_rights_sql); - while ( $db_rights_row = PMA_DBI_fetch_assoc( $db_rights_result ) ) { - $db_rights_row = array_merge( $user_defaults, $db_rights_row ); + while ($db_rights_row = PMA_DBI_fetch_assoc($db_rights_result)) { + $db_rights_row = array_merge($user_defaults, $db_rights_row); $db_rights[$db_rights_row['User']][$db_rights_row['Host']] = $db_rights_row; } } else { - foreach ( $db_rights_sqls as $db_rights_sql ) { - $db_rights_result = PMA_DBI_query( $db_rights_sql ); + foreach ($db_rights_sqls as $db_rights_sql) { + $db_rights_result = PMA_DBI_query($db_rights_sql); - while ( $db_rights_row = PMA_DBI_fetch_assoc( $db_rights_result ) ) { - $db_rights_row = array_merge( $user_defaults, $db_rights_row ); + while ($db_rights_row = PMA_DBI_fetch_assoc($db_rights_result)) { + $db_rights_row = array_merge($user_defaults, $db_rights_row); $db_rights[$db_rights_row['User']][$db_rights_row['Host']] = $db_rights_row; } } } - PMA_DBI_free_result( $db_rights_result ); - unset( $db_rights_sql, $db_rights_sqls, $db_rights_result, $db_rights_row ); - ksort( $db_rights ); + PMA_DBI_free_result($db_rights_result); + unset($db_rights_sql, $db_rights_sqls, $db_rights_result, $db_rights_row); + ksort($db_rights); /** * Displays the initials */ + // for all initials, even non A-Z + $array_initials = PMA_DBI_fetch_result( + 'SELECT DISTINCT UPPER(LEFT(' . PMA_convert_using('User') . ',1))' + . ' FROM `user` ORDER BY `User` ASC', + 0, 0); - // initialize to FALSE the letters A-Z + // initialize to false the letters A-Z for ($letter_counter = 1; $letter_counter < 27; $letter_counter++) { - if ( ! isset( $array_initials[chr($letter_counter + 64)] ) ) { - $array_initials[chr($letter_counter + 64)] = FALSE; + if (isset($array_initials[chr($letter_counter + 64)])) { + $array_initials[chr($letter_counter + 64)] = true; + } else { + $array_initials[chr($letter_counter + 64)] = false; } } - $initials = PMA_DBI_try_query('SELECT DISTINCT UPPER(LEFT(' . PMA_convert_using('User') . ',1)) FROM `user` ORDER BY `User` ASC', null, PMA_DBI_QUERY_STORE); - while (list($tmp_initial) = PMA_DBI_fetch_row($initials)) { - $array_initials[$tmp_initial] = TRUE; - } - // Display the initials, which can be any characters, not // just letters. For letters A-Z, we add the non-used letters // as greyed out. - uksort( $array_initials, "strnatcasecmp" ); + uksort($array_initials, "strnatcasecmp"); echo ''; foreach ($array_initials as $tmp_initial => $initial_was_found) { if ($initial_was_found) { - echo '' . "\n"; + echo '' . "\n"; } else { echo ''; } } - echo '' . "\n"; + echo '' . "\n"; echo '
' . $tmp_initial . '' . $tmp_initial . '' . $tmp_initial . '[' . $GLOBALS['strShowAll'] . '][' . $GLOBALS['strShowAll'] . ']
'; /** @@ -1408,17 +1483,18 @@ if ( empty( $adduser ) && ( ! isset( $checkprivs ) || ! strlen($checkprivs) ) ) * (if less than 50 users, display them immediately) */ - if (isset($initial) || isset($showall) || PMA_DBI_num_rows($res) < 50) { + if (isset($_REQUEST['initial']) || isset($_REQUEST['showall']) + || PMA_DBI_num_rows($res) < 50) { - while ( $row = PMA_DBI_fetch_assoc( $res ) ) { - $row['privs'] = PMA_extractPrivInfo( $row, true ); + while ($row = PMA_DBI_fetch_assoc($res)) { + $row['privs'] = PMA_extractPrivInfo($row, true); $db_rights[$row['User']][$row['Host']] = $row; } - @PMA_DBI_free_result( $res ); - unset( $res ); + @PMA_DBI_free_result($res); + unset($res); echo '
' . "\n" - . PMA_generate_common_hidden_inputs('', '', 1) + . PMA_generate_common_hidden_inputs() . ' ' . "\n" . ' ' . "\n" . ' ' . "\n" @@ -1426,22 +1502,32 @@ if ( empty( $adduser ) && ( ! isset( $checkprivs ) || ! strlen($checkprivs) ) ) . ' ' . "\n" . ' ' . "\n" . ' ' . "\n" + . PMA_showHint($GLOBALS['strEnglishPrivileges']) . '' . "\n" . ' ' . "\n" - . ' ' . ($GLOBALS['cfg']['PropertiesIconic'] ? '' : '') . "\n"; + . ' ' . ($GLOBALS['cfg']['PropertiesIconic'] + ? '' + : '') + . "\n"; echo ' ' . "\n"; echo ' ' . "\n"; echo ' ' . "\n"; $odd_row = true; $index_checkbox = -1; - foreach ( $db_rights as $user ) { + foreach ($db_rights as $user) { $index_checkbox++; - ksort( $user ); - foreach ( $user as $host ) { + ksort($user); + foreach ($user as $host) { $index_checkbox++; - echo ' ' . "\n" - . ' ' . "\n" - . ' ' . "\n" + echo ' ' . "\n" + . ' ' . "\n" + . ' ' . "\n" . ' ' . "\n"; echo ' ' . "\n" . ' ' . "\n" . ' ' . "\n" . ' ' . "\n" . ' ' . "\n"; $odd_row = ! $odd_row; } } - unset( $user, $host, $odd_row ); + unset($user, $host, $odd_row); echo '
' . $GLOBALS['strHost'] . '' . $GLOBALS['strPassword'] . '' . $GLOBALS['strGlobalPrivileges'] . ' ' - . PMA_showHint( $GLOBALS['strEnglishPrivileges'] ) . '' . $GLOBALS['strGrantOption'] . '' . $GLOBALS['strAction'] . '' . $GLOBALS['strAction'] . '
' . htmlspecialchars($host['Host']) . ''; switch ($host['Password']) { @@ -1458,46 +1544,46 @@ if ( empty( $adduser ) && ( ! isset( $checkprivs ) || ! strlen($checkprivs) ) ) } // end switch echo '' . "\n" - . ' ' . implode( ',' . "\n" . ' ', $host['privs'] ) . "\n" + . ' ' . implode(',' . "\n" . ' ', $host['privs']) . "\n" . ' ' . ($host['Grant_priv'] == 'Y' ? $GLOBALS['strYes'] : $GLOBALS['strNo']) . ''; - printf( $link_edit, urlencode( $host['User'] ), - urlencode( $host['Host'] ), '', '' ); + printf($link_edit, urlencode($host['User']), + urlencode($host['Host']), '', ''); echo '
' . "\n" .'' . $GLOBALS['strWithChecked'] . '' . "\n" .'' + .' onclick="if (markAllRows(\'usersForm\')) return false;">' . $GLOBALS['strCheckAll'] . '' . "\n" .'/' . "\n" .'' + .' onclick="if (unMarkAllRows(\'usersForm\')) return false;">' . $GLOBALS['strUncheckAll'] . '' . "\n"; // add/delete user fieldset echo '
' . "\n" . ' ' . "\n" - . ($GLOBALS['cfg']['PropertiesIconic'] ? ' ' . "\n" : '' ) + . ($GLOBALS['cfg']['PropertiesIconic'] ? ' ' . "\n" : '') . ' ' . $GLOBALS['strAddUser'] . '' . "\n" . '
' . "\n" . '
' . ' ' . "\n" - . ($GLOBALS['cfg']['PropertiesIconic'] ? ' ' . "\n" : '' ) + . ($GLOBALS['cfg']['PropertiesIconic'] ? ' ' . "\n" : '') . ' ' . $GLOBALS['strRemoveSelectedUsers'] . '' . "\n" . ' ' . "\n"; - + // before MySQL 4.1.1, we offer some choices for the delete - // mode, but for 4.1.1+, it will be done with REVOKEing the + // mode, but for 4.1.1+, it will be done with REVOKEing the // privileges then a DROP USER (even no REVOKE at all // for MySQL 5), so no need to offer so many options if (PMA_MYSQL_INT_VERSION < 40101) { @@ -1516,7 +1602,7 @@ if ( empty( $adduser ) && ( ! isset( $checkprivs ) || ! strlen($checkprivs) ) ) } else { echo ' ' . "\n" . '( ' . $GLOBALS['strRevokeAndDelete'] . ' )
' . "\n"; - } + } echo ' ' . "\n" . '