Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 293699 - net-nds/openldap: some openldap contrib overlays broken
Summary: net-nds/openldap: some openldap contrib overlays broken
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo LDAP project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-11-19 06:27 UTC by Dmitry A. Bakshaev
Modified: 2009-11-24 05:35 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---

Attachments
fix to build contrib overlays (openldap-2.4.19-contrib-overlays.patch,540 bytes, patch)
2009-11-19 06:30 UTC, Dmitry A. Bakshaev 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Dmitry A. Bakshaev 2009-11-19 06:27:20 UTC 
loading some overlay modules (allowed,nops-overlay,denyop-overlay,lastmod,trace) do not work.


Reproducible: Always

Steps to Reproduce:
1. USE="overlays" emerge =net-nds/openldap-2.4.19
2. adding "moduleload lastmod.la" to slapd.conf
3. start slapd

Actual Results:  
slapd failed to start with
"<moduleload> handler exited with 1"


Expected Results:  
working slapd

"strace slapd ..." get 
init_module(0, 0, 0x5) = -1 ENOEXEC (Exec format error)
Comment 1 Dmitry A. Bakshaev 2009-11-19 06:30:25 UTC 
Created attachment 210601 [details, diff]
fix to build contrib overlays

broken overlays has conditional compilation rules like
#if SLAPD_OVER_ALLOWED == SLAPD_MOD_DYNAMIC
but ebuild do not set these
Comment 2 Dustin Polke 2009-11-19 11:16:39 UTC 
Please don't resolve your bugs as FIXED if your patch hasn't been added to the tree yet. Your bug report (and patch) is likely to be missed.

Please reopen your bug as I don't have the privileges to do this myself.

Thanks
Comment 3 Dmitry A. Bakshaev 2009-11-19 13:13:48 UTC 
follow Comment #2
Comment 4 Robin Johnson archtester Gentoo Infrastructure gentoo-dev Security 2009-11-24 05:21:58 UTC 
That tr statement is NOT safe in all locates. Please explictly call it with LC_ALL=C.
Comment 5 Robin Johnson archtester Gentoo Infrastructure gentoo-dev Security 2009-11-24 05:35:48 UTC 
Fixed in 2.4.19-r1. I used a bit of bash magic to avoid that extra execution entirely.