Tickets :: Comment on [#2094] Choosing Delete action on query builder repeatedly brings up unserialize error

* Your Email Address
* Spam protection	Enter the letters below: . ..___\ / ..___. \|\ \|[__ >< \| \| \| \\|[___/ \\__\| \|
Comment	> Choosing the delete action on the 'current query' repeatedly first > brings up a 'permission denied' error, then going back to the query > builder, the following messages are printed > > > > Notice: unserialize(): Argument is not a string in > /usr/local/horde/whups/query.php on line 28 > > > > Fatal error: Call to a member function on a non-object in > /usr/local/horde/whups/query.php on line 29 > > > > Looking in the source: > > > > // Find our current query. > > if (isset($_SESSION['whups_query'])) { > > $whups_query = unserialize($_SESSION['whups_query']); > > if (!$whups_query->hasPermission(Auth::getAuth(), PERMS_READ)) { > > > > It looks as if along with an isset, the session variable should be > checked to see if it is a unserializable string. > > > > Logging out (destroying the session) causes this error to go away, > and the query browser is usable again.
Attachment
Watch this ticket