Tickets :: [#8600] SQL syntax error in session handling with MSSQL

6.0.0-beta1

7/15/25

Summary	SQL syntax error in session handling with MSSQL
Queue	Horde Base
Queue Version	3.3.5
Type	Bug
State	Resolved
Priority	2. Medium
Owners	selsky (at) columbia (dot) edu
Requester	moreda (at) allenta (dot) com
Created	09/23/2009 (5774 days ago)
Due
Updated	09/25/2009 (5772 days ago)
Assigned
Resolved	09/25/2009 (5772 days ago)
Github Issue Link
Github Pull Request
Milestone	3.3.6
Patch	Yes

09/25/2009 02:22:41 PM	CVS Commit	Comment #4	Reply to this comment
Changes have been made in CVS for this ticket: ~~Bug: 8600~~ BFN http://cvs.horde.org/diff.php/horde/docs/CHANGES?rt=horde&r1=1.1263&r2=1.1264&ty=u

09/25/2009 09:13:05 AM	Jan Schneider	Assigned to Matt Selsky State ⇒ Resolved Milestone ⇒ 3.3.6

09/25/2009 08:06:36 AM	CVS Commit	Comment #3	Reply to this comment
Changes have been made in Git for this ticket: MFB: 1.22.10.20 Fix SQL syntax. ~~Bug: 8600~~ Submitted by: moreda@allenta.com http://git.horde.org/diff.php/framework/SessionHandler/lib/Horde/SessionHandler/Sql.php?rt=horde-git&r1=def290e68df5243289d2217150d7d1bd4c76581c&r2=2d8c8de6751fdff001a14d83637cb6198245f764

09/25/2009 08:03:57 AM	CVS Commit	Comment #2	Reply to this comment
Changes have been made in CVS for this ticket: Fix SQL syntax. ~~Bug: 8600~~ Submitted by: moreda@allenta.com http://cvs.horde.org/diff.php/framework/SessionHandler/SessionHandler/Attic/sql.php?rt=horde&r1=1.22.10.19&r2=1.22.10.20&ty=u

09/23/2009 05:12:58 PM	moreda (at) allenta (dot) com	Comment #1 Priority ⇒ 2. Medium New Attachment: fix-greater-than-or-equal-to.diff Patch ⇒ Yes Milestone ⇒ Summary ⇒ SQL syntax error in session handling with MSSQL Type ⇒ Bug State ⇒ Unconfirmed Queue ⇒ Horde Base	Reply to this comment
This seems to me a syntax error in a SQL sentence that arised when using MSSQL as a backend for session information through PEAR's DB abstraction layer. In lib/Horde/SessionHandler/sql.php, line 312 you can find this query: /* Build the SQL query. */ $query = 'SELECT session_id FROM ' . $this->_params['table'] . ' WHERE session_lastmodified => ?'; which seems incorrect because of the "=>" that should be ">=". See trivial patch attached.