mysqli_real_escape_string() expects parameter 1 to be mysqli, boolean given #12735

Closed
Ramzzes opened this Issue Nov 25, 2016 · 1 comment

Projects

None yet

3 participants

@Ramzzes
Ramzzes commented Nov 25, 2016

Steps to reproduce

  1. Open phpMyAdmin main page

Expected behaviour

The database list is empty. phpMyAdmin displays an error.

Actual behaviour

The database list is not empty. phpMyAdmin does not display errors.

Server configuration

Operating system:
Debian 8.6
Web server:
nginx 1.10.2 + php-fpm
Database:
MySQL 5.5.53
PHP version:
PHP 5.6.27
phpMyAdmin version:
phpMyAdmin 4.6.5

Client configuration

Browser:
Google Chrome 54.0.2840.99
Operating system:
Windows 7 SP1 x64

Warning in ./libraries/dbi/DBIMysqli.php#629
 mysqli_real_escape_string() expects parameter 1 to be mysqli, boolean given

Backtrace

./libraries/dbi/DBIMysqli.php#629: mysqli_real_escape_string(
boolean true,
string '(information_schema|mysql)',
)
./libraries/DatabaseInterface.php#2670: PMA\libraries\dbi\DBIMysqli->escapeString(
boolean true,
string '(information_schema|mysql)',
)
./libraries/navigation/nodes/Node.php#725: PMA\libraries\DatabaseInterface->escapeString(
string '(information_schema|mysql)',
boolean true,
)
./libraries/navigation/nodes/Node.php#440: PMA\libraries\navigation\nodes\Node->_getWhereClause(
string 'SCHEMA_NAME',
string '',
)
./libraries/navigation/NavigationTree.php#282: PMA\libraries\navigation\nodes\Node->getData(
string 'databases',
integer 0,
string '',
)
./libraries/navigation/NavigationTree.php#840: PMA\libraries\navigation\NavigationTree->_buildPath()
./libraries/navigation/Navigation.php#43: PMA\libraries\navigation\NavigationTree->renderState()
./libraries/Header.php#427: PMA\libraries\navigation\Navigation->getDisplay()
./libraries/Response.php#266: PMA\libraries\Header->getDisplay()
./libraries/Response.php#279: PMA\libraries\Response->_getDisplay()
./libraries/Response.php#443: PMA\libraries\Response->_htmlResponse()
PMA\libraries\Response::response()
@devenbansod devenbansod added the bug label Nov 25, 2016
@devenbansod devenbansod self-assigned this Nov 25, 2016
@devenbansod devenbansod added a commit that referenced this issue Nov 25, 2016
@devenbansod devenbansod Fix some more instances of wrong arguments
Issue #12735

Signed-off-by: Deven Bansod <devenbansod.bits@gmail.com>
a600ff3
@devenbansod devenbansod added a commit that referenced this issue Nov 25, 2016
@devenbansod devenbansod ChangeLog for #12735
Signed-off-by: Deven Bansod <devenbansod.bits@gmail.com>
b9ee595
@devenbansod devenbansod added a commit that closed this issue Nov 25, 2016
@devenbansod devenbansod Avoid use of improper parameters in escapeString
Fix #12735

Signed-off-by: Deven Bansod <devenbansod.bits@gmail.com>
deeb3e6
@ibennetch ibennetch added this to the 4.6.5.1 milestone Nov 25, 2016
@ibennetch
Contributor

I've just released version 4.6.5.1 as a fix for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment