Re: configure openldap crash warning
Andres Freund <andres@anarazel.de>
From: Andres Freund <andres@anarazel.de>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Peter Eisentraut <peter.eisentraut@enterprisedb.com>, pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2022-08-30T03:48:30Z
Lists: pgsql-hackers
Hi, On 2022-08-29 23:18:23 -0400, Tom Lane wrote: > Andres Freund <andres@anarazel.de> writes: > > Why do we continue to link the backend to ldap when we find ldap_r, given that > > we know that it can cause problems for extension libraries using libpq? > > Uh ... if we know that, it's news to me. Isn't that what the configure warning Peter mentioned upthread is about? # PGAC_LDAP_SAFE # -------------- # PostgreSQL sometimes loads libldap_r and plain libldap into the same # process. Check for OpenLDAP versions known not to tolerate doing so; assume # non-OpenLDAP implementations are safe. The dblink test suite exercises the # hazardous interaction directly. The patch applied as a result of this thread dealt with a different version of the problem, with -lldap_r picking up a different library version than -lldap. Leaving that aside it also doesn't seem like a great idea to have two different copies of the nearly same library loaded for efficiency reasons, not that it'll make a large difference... Greetings, Andres Freund
Commits
-
configure: don't probe for libldap_r if libldap is 2.5 or newer.
- c61f36d99606 10.22 landed
- aff45c879e01 15.0 landed
- af9b96767153 13.8 landed
- 8ed13fb9346d 12.12 landed
- 36d76da8156a 11.17 landed
- 12736e7dc326 14.4 landed