Project:Persistent Login
Version:6.x-1.4-beta6
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

Before security warning I got this php notice with error reporting enabled:

Notice: Trying to get property of non-object in /home/example.com/public_html/includes/common.inc on line 892

Using an old firefox profile (restored from a backup) can cause security alert.

Comments

#1

Status:active» needs work

It seems we need to include a bit more stuff in that case.

#2

Status:needs work» needs review

Apologies for the delay in this issue. Here's a patch that I think it fixes this issue. We need a full Drupal bootstrap to properly initialize the global language variable that we need to render the warning to the user, but we also need it to properly process the watchdog() request, otherwise we do not store anything because dblog module is not loaded yet. We're here processing hook_boot().

AttachmentSize
persistent_login-621916-2.patch 901 bytes

#3

Status:needs review» fixed

A full Drupal bootstrap here should cause no harm and seems to fix the issue here, patch committed to CVS.

http://drupal.org/cvs?commit=311662

It should be available in 1.4 release, soon to be out.

#4

Status:fixed» closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

nobody click here