manejo de sessiones en login en php

[kriss8725]

Buenas tardes quiciera saber si algun experto me puede decir si es correcto manejar las sessiones en un login de la siguiente manera:

Código PHP:

  class login extends _GLOBAL_
 {
     public function validarUsuario($usu,$pwd,$unidad){//Inicio sección validarUsuario
        $usu = mysql_escape_string($usu);
        $pwd = mysql_escape_string($pwd);
        $unidad = mysql_escape_string($unidad);
            
        $this->conexion($unidad."_generalcms");
        //consulta para validar el usuario
        $consulta = $this->consulta(sprintf("SELECT * FROM usuarios WHERE usuario = '".$usu."' LIMIT 1"));
          //resultado de la consulta en un array
          $resultado = $this->fetch_array($consulta);
          if(count($resultado) > 0){
              foreach($resultado as $row):
                if($row['pass'] == $row['pass']){
                  if($row['activo'] == "1"){
                    $_SESSION['status'] = "autorizado";
                    $_SESSION['unidad'] = $unidad;
                    //$this->registroActividad("Inicio sesión");
                    header("Location: cms.php");
                  }else{
                    return "Acceso denegado";  
                  }
                }else{
                  return "Acceso denegado";        
                }
              endforeach;
          }else{
              return "Acceso denegado";
          }
     }//Fin sección validarUsuario
     
     public function cerrarSesion(){//Inicio sección cerrarSesion
        //$this->registroActividad("Cerro sesión");
        unset($_SESSION['status']);
        unset($_SESSION['unidad']);
        if(isset($_COOKIE[session_name()])){
            setcookie(session_name,"",time() - 10000);
            session_destroy();
        }
        header("Location: index.php");
     }//Fin sección cerrarSesion
 } 
   

De antemano gracias.

[zarkiel]

Sería mejor que manejes todo en la consulta SQL, podrías hacerlo asi:

Código PHP:

$consulta = $this->consulta(sprintf("SELECT * FROM usuarios WHERE usuario = '".$usu."' AND pass='".$pwd."' AND activo=1 LIMIT 1"));
 

Si el resultado de la consulta es mayor que 0, entonces se le concede el acceso.

Código PHP:

$resultado = $this->fetch_array($consulta);
          if(count($resultado) > 0){ 
               $_SESSION['status'] = "autorizado";
                    $_SESSION['unidad'] = $unidad;
                    //$this->registroActividad("Inicio sesión");
                    header("Location: cms.php"); 
         }else{
           return "Acceso denegado";  
        }
 

Creo que estás utilizando mysqli, si es así el código podría optimizarse mucho más, bueno espero que te sirva