667325 - Open correct libgssapi library version on OpenBSD

Status: RESOLVED FIXED

(Core :: Security, defect)

Description

[Landry Breuil (:gaston)]

Attachment: Try to open libgssapi.so

Aftre leeengthy bikeshedding in https://bugzilla.mozilla.org/show_bug.cgi?id=650772, here's a patch to open the correct library on OpenBSD, where libgssapi.so.5.0 is present, and the loader will open it if asked for libgssapi.so. Not sure the component is the good one for extensions/auth subdir..

Comment 1

[Landry Breuil (:gaston)]

Erm... kai, ping ? we've been shipping this patch for ages.

Comment 2

[Landry Breuil (:gaston)]

Attachment: Open libgssapi.so for OpenBSD

Comment 3

[Landry Breuil (:gaston)]

Forgot to say it's a new patch against m-c, previous didn't apply.

Comment 4

[Landry Breuil (:gaston)]

Comment on attachment 592426 [details] [diff] [review]
Open libgssapi.so for OpenBSD

Reassing on ludo's advice...

Comment 5

[Daniel Holbert [:dholbert]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/e676b8f86ab1

Comment 6

[Daniel Holbert [:dholbert]]

Comment on attachment 592426 [details] [diff] [review]
Open libgssapi.so for OpenBSD

Also:
>         const char *const verLibNames[] = {
>             "libgssapi_krb5.so.2", /* MIT - FC, Suse10, Debian */
>             "libgssapi.so.4",      /* Heimdal - Suse10, MDK */
>-            "libgssapi.so.1"       /* Heimdal - Suse9, CITI - FC, MDK, Suse10*/
>+            "libgssapi.so.1",       /* Heimdal - Suse9, CITI - FC, MDK, Suse10*/
...before landing, I removed this----^ extra space character to keep these comments aligned after the patch's comma-insertion.

Comment 7

[Landry Breuil (:gaston)]

ah, thanks... i was pretty sure having tried to keep the aligment, but seems i failed :)

Comment 8

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/e676b8f86ab1