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

Cvbge drupal-devel at drupal.org
Sat Sep 3 15:53:01 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)
+Status:       patch (ready to be committed)

A person proposed almost identical patch in http://drupal.org/node/30268


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?


Fri, 19 Aug 2005 13:25:37 +0000 : Cvbge

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...


Fri, 19 Aug 2005 14:07:20 +0000 : chx

You can change schema IMO. It has happened already, we extended locale
field size in a minor release. Just nothing major. If you need a rule
of thumb: column attribute changes are OK but nothing else.


Fri, 19 Aug 2005 19:39:03 +0000 : Cvbge

I believe the SQL query change is better in this case, because it fixes
the problem, does not need schema change and makes 4.5 version the same
as 4.6.

More information about the drupal-devel mailing list