Switching all the mysql_* functions to mysqli_* functions results in warning errors

Question

I'm currently switching all the mysql_* functions to mysqli_* functions and i am getting the following errors .

 PHP Warning: mysqli_connect(): (HY000/1040): Too many connections in

 PHP Warning: mysqli_query() expects parameter 1 to be mysqli, boolean given in 

 PHP Warning: mysqli_error() expects parameter 1 to be mysqli, boolean

In config.php :

            $dbuser = "xxx";
            $dbpass = "xxx";
            $dbhost = "xxxx";
            $dbname = "xxxxxx";
            $connection = mysqli_connect($dbhost, $dbuser, $dbpass) or die("could not connect to mysql");
            if($connection){
                echo "\n Database connected ....\n\n";
            }
            mysqli_select_db($connection,$dbname) or die(mysqli_error($connection));

In other.php

require_once 'Lib/config.php';

 class Mutex {

    protected $connection;
    public function __construct()
    {
    global $dbuser,$dbpass,$dbname,$dbhost; // globally declaring the config variables
    $this->connection = mysqli_connect($dbhost, $dbuser, $dbpass,$dbname);   
    if (mysqli_connect_errno())
    {
      echo "Failed to connect to MySQL: " . mysqli_connect_error();
      //you need to exit the script, if there is an error
        exit();
    }
    }



    public function Prog($pack = null) {

        if(isset($pack) && $pack != "") {
            $sql_qry    = "SELECT id from xxxx WHERE package like '" . addslashes($pack) . "'";
            showSqlQuery($sql_qry);
            $result     = mysqli_query($this->connection,$sql_qry)or die(mysqli_error($this->connection));
            if($result && mysqli_num_rows($result)>0) {
                $row    = mysqli_fetch_assoc($result);
                if(!empty($row)) {
                    return "Exists";
                }
            }
            return "NotExists";
        }
        return "InvalidProcess";
    }
}

Tried many solutions, but none of them worked for me

Getting errors as shown above. Please help me to solve this..

Thanks in advance.

Possible duplicate of [Can I blindly replace all mysql\_ functions with mysqli\_?](https://stackoverflow.com/questions/26476162/can-i-blindly-replace-all-mysql-functions-with-mysqli) — Carl Binalla, Jan 17 '18 at 07:53
Why have a db connection in `config.php` using 1 method and then another connection in `Mutex` using a different method? — Professor Abronsius, Jan 17 '18 at 07:56
How are you calling the method? And `addslashes()` should not be used to avoid sql injection, use a prepared statement instead. — jeroen, Jan 17 '18 at 08:04
@jeroen , $ObjMutex = new Mutex (); // Mutex object $ObjMutex->CheckProcessExists ( $PkgsName ); // Functions called — Cliff_Cleet, Jan 17 '18 at 09:46
@Swellar , I have read through the mentioned above question, but its not the similar case and didn't solve my error. — Cliff_Cleet, Jan 17 '18 at 09:50

Professor Abronsius · Answer 1 · 2018-01-17T10:06:30.387

Rather than duplicating ( nearly ) the connection you could try the approach where you pass a reference to the db connection as a parameter to the Mutex constructor

<?php
    /* lib/config.php */
    $dbhost =   'localhost';
    $dbuser =   'root'; 
    $dbpass =   'xxx'; 
    $dbname =   'xxx';
    $db =   new mysqli( $dbhost, $dbuser, $dbpass, $dbname );

?>

<?php

    include 'lib/config.php';

    class Mutex{
        private $connection;

        public function __construct( $dbo=object ){
            $this->connection=$dbo;
        }

        public function Prog($pack = null) {
            $result='InvalidProcess';
            if( !empty( $pack ) ) {

                $sql='select `id` from `xxxx` where `package` like ?';
                $stmt=$this->connection->prepare( $sql );
                if( $stmt ){

                    $var = '%'.$pack.'%';
                    $stmt->bind_param('s', $var );

                    $result=$stmt->execute();
                    if( $result && $stmt->num_rows > 0  ){

                        $stmt->store_result();
                        $stmt->bind_result( $id );

                        while( $stmt->fetch() ){/* do stuff perhaps */
                            echo $id;
                        }

                        $stmt->free_result();
                        $stmt->close();
                        $result='Exists';
                    }
                }
            }
            return $result;
        }



    $pack='banana';

    $mutex=new Mutex( $db );
    $mutex->Prog( $pack );

?>

Although better, I don't see how this is any different from the code the OP is running; in the end the database connection is generated exactly the same way. — jeroen, Jan 17 '18 at 08:07
true - more or less the same but now only 1 connection to debug — Professor Abronsius, Jan 17 '18 at 08:10

score 0 · Answer 2 · answered Jan 17 '18 at 08:05

I think your DB configuration has some issue.

Replace the following code of config.php file to bellow.

$dbuser = "xxx";
$dbpass = "xxx";
$dbhost = "xxxx";
$dbname = "xxxxxx";
$connection = mysqli_connect($dbhost, $dbuser, $dbpass, $dbname);
// Check connection
if (mysqli_connect_errno())
{
    echo "Failed to connect to MySQL: " . mysqli_connect_error();
}

Switching all the mysql_* functions to mysqli_* functions results in warning errors

2 Answers2