CODEWITHSUNDEEP
phpsession
RatDon
RatDon

Reputation: 3543

Warning: session_destroy(): Trying to destroy uninitialized session

my class.inc file:

<?php
class logout{
    public function logout(){
        $_SESSION = array();
        if (ini_get("session.use_cookies")) {
            $params = session_get_cookie_params();
            setcookie(session_name(), '', time() - 42000, $params['path'], $params['domain'], $params['secure'], $params["httponly"]);
        }
        session_destroy();
    }   
}

?>

used code for my logout:

session_start();
require($path."include/class.inc");
if(!empty($_GET['logout'])){
    $object=new logout();
    $object->logout();
    $content='5;url='.$path.'index.php';
}

When the logout function is called, it destroys the session, but shows the warning:

Warning: session_destroy(): Trying to destroy uninitialized session in class.inc on line 9

I am unable to troubleshoot, as the session is not being destroyed by any other means before the session_destroy() of class.inc.

Upvotes: 15

Views: 86920

Answers (7)

scr4bble
scr4bble

Reputation: 309

https://www.php.net/manual/en/function.session-status.php

if (session_status() === PHP_SESSION_ACTIVE)
        session_destroy();

Upvotes: 7

AboElnouR
AboElnouR

Reputation: 316

You can add this code to start a session if it didn't start before

if(!session_id()) {
    session_start();
}

Upvotes: 1

Jijo John
Jijo John

Reputation: 1375

You have to call the function mentioned below at the top your logout function in the logout class.

session_start();

Add the above function and try it out. If you don’t start the session at the top of your file, it will throw exceptions like “headers already sent”, “can’t start the session”, etc.

Upvotes: 44

Roger Dueck
Roger Dueck

Reputation: 625

I encountered the session_destroy() error message when I started using session_write_close(). To determine if session_destroy() should be called or not, I had to do the following:

class Session {
    public static function start() {
        self::$haveSession = true;
        session_start();
    }
    public static function finish() {
        session_write_close();
        self::$haveSession = false;
    }
    public static function clear() {
        if (self::$haveSession) {
            session_unset();
            session_destroy();
        }
    }
}

In PHP >= 5.4 it should work to replace if (self::$haveSession) with if(session_status() === PHP_SESSION_ACTIVE).

Upvotes: 2

Alok Jha
Alok Jha

Reputation: 572

You'll have to call session_start() before you call session_destroy();

Are you storing session in data in files or in a database. If you are storing it in a database, I normally just delete the record from the session table that corresponds to the session id, that way you don't have to unregister each session var and it actually deletes the whole session.

Upvotes: 0

Padmanathan J
Padmanathan J

Reputation: 4620

Start your session session_start(); and you can destroy your session

    <?php
    session_start();
    class logout{
        public function logout(){
            $_SESSION = array();
            if (ini_get("session.use_cookies")) {
                $params = session_get_cookie_params();
                setcookie(session_name(), '', time() - 42000, $params['path'], $params['domain'], $params['secure'], $params["httponly"]);
            }
            session_destroy();
        }   
    }

?>

Upvotes: 0

James
James

Reputation: 4793

This error is common when you haven't started the session beforehand

if (!isset($_SESSION))
  {
    session_start();
  }

Upvotes: 15

Related Questions