[drupal-devel] [bug] Session system - fix for NULL uid

Cvbge drupal-devel at drupal.org
Fri Aug 19 13:25:39 UTC 2005

Issue status update for 
Post a follow up: 

 Project:      Drupal
 Version:      4.5.5
 Component:    other
 Category:     bug reports
 Priority:     critical
 Assigned to:  Anonymous
 Reported by:  Cvbge
 Updated by:   Cvbge
 Status:       patch (code needs review)
 Attachment:   http://drupal.org/files/issues/session-uid.inc.diff (955 bytes)

I can't change schema definition because that'd mean that db update is
needed when upgrading from 4.5.5 to 4.5.6.
As I said (although not directly) 4.6 and later does not have this bug.

Heh, at first I didn't understand what's with the .inc extension...


Previous comments:

Thu, 18 Aug 2005 19:39:17 +0000 : Cvbge

Attachment: http://drupal.org/files/issues/session-uid.inc (955 bytes)

On fresh install session table is empty and session.inc tries to insert
a row. But it ommits uid field which is NOT NULL and inserting fails.
Attached patch makes it to have the same behaviour as 4.6


Thu, 18 Aug 2005 22:13:34 +0000 : Dries

Patch doesn't apply because it ends with '.inc'.  Maybe better to use
uid int(11) NOT NULL default '0' in the table definition?

More information about the drupal-devel mailing list