Closed (fixed)
Project:
Lightweight Directory Access Protocol
Version:
7.x-2.x-dev
Component:
SimpleTests
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
11 Mar 2016 at 11:25 UTC
Updated:
22 Aug 2016 at 23:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
queenvictoria commented@larowlan it would be super handy if you reviewed this because then we'd be that much closer to our tests passing.
Comment #4
larowlanLooks good to me
tests++
Comment #5
queenvictoria commentedCommitted thanks!
Comment #6
enriqe commentedMaybe it's related:
Notice: Constant LDAP_DEREF_NEVER already defined in include_once() (line 96 of /var/www/sites/all/modules/ldap/ldap_servers/ldap_servers.module).
Start apearing after yesterday update
Added this condition in the meantime:
if (!defined('LDAP_DEREF_NEVER')) define('LDAP_DEREF_NEVER', 0x25);
CORE 7.43
LDAP 7.x-2.0-beta8+17-dev (2016-Mar-13)
MySQL 5.6.24
Comment #7
larowlanComment #8
larowlan@queenvictoria had fixed already
Comment #9
jay.dansand commentedJust updated from 7.x-1.0-beta12+17-dev (2013-09-30) to 7.x-1.0-beta12+18-dev (2016-03-13), and this is the only commit in the update.
Immediately after updating, I started receiving
Notice: Constant LDAP_DEREF_NEVER already defined in include_once() (line 92 of .../ldap_servers.module).on every page (including in the Seven-themed admin area).The error persists after forced registry rebuild and clearing all caches. LDAP modules enabled:
I grepped our entire Drupal codebase and
LDAP_DEREF_NEVERis only defined in one place (ldap_servers.module). Many other constants are defined in the same location but don't trigger PHP notices; I have no idea why this particular constant causes the notice and the others don't.Comment #10
kaosagnt commentedhttp://php.net/manual/en/ldap.constants.php
The constants below are defined by this extension, and will only be available when the extension has either been compiled into PHP or dynamically loaded at runtime.
LDAP_DEREF_NEVER (integer)Therefore 7.x-1.0-beta12+18-dev (2016-03-13) complains as pointed out by others. This needs to be fixed.
Comment #11
freggy commentedBug present in newly released security update 7.x-1.0-beta13. Please fix.
Comment #12
jay.dansand commentedHere's a patch rolled against 7.x-1.x-dev.
Comment #13
Drupalized by Jason commentedI can confirm patch from #12 is working.
Comment #15
larowlanthanks - fixed, will roll a new 1.x branch release once #1920094: ldap authentication: allow login with null byte - null byte attack is resolved