Ticket 1015 - pam: libslurm.so loaded multiple times
Summary: pam: libslurm.so loaded multiple times
Status: RESOLVED FIXED
Alias: None
Product: Slurm
Classification: Unclassified
Component: Other (show other tickets)
Version: 2.5.x
Hardware: Linux Linux
: 6 - No support contract
Assignee: David Bigagli
QA Contact:
URL:
Depends on:
Blocks:
 
Reported: 2014-08-04 02:20 MDT by Mehdi Dogguy
Modified: 2014-08-05 05:51 MDT (History)
1 user (show)

See Also:
Site: EDF - Electricite de France
Slinky Site: ---
Alineos Sites: ---
Atos/Eviden Sites: ---
Confidential Site: ---
Coreweave sites: ---
Cray Sites: ---
DS9 clusters: ---
Google sites: ---
HPCnow Sites: ---
HPE Sites: ---
IBM Sites: ---
NOAA SIte: ---
NoveTech Sites: ---
Nvidia HWinf-CS Sites: ---
OCF Sites: ---
Recursion Pharma Sites: ---
SFW Sites: ---
SNIC sites: ---
Tzag Elita Sites: ---
Linux Distro: ---
Machine Name:
CLE Version:
Version Fixed: 14.03.7
Target Release: ---
DevPrio: ---
Emory-Cloud Sites: ---

Attachments
Patch to contribs/pam/pam_slurm.c (1.18 KB, patch)
2014-08-04 20:07 MDT, Mehdi Dogguy 		Details | Diff
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this ticket.
Description Mehdi Dogguy 2014-08-04 02:20:06 MDT 
Hi,

We are using the pam plugin and it seems that the code is trying to load libslurm.so multiple times, even if the first one succeeded. It is not problematic for Slurm but it adds useless warnings in the log file. An example is:

Jan  6 14:40:40 atcn758 pam_slurm[47822]: Unable to dlopen libslurm.so: libslurm.so: cannot open shared object file: No such file or directory

In that case, the plugin managed to load libslurm.so.Something but not libslurm.so.

Attached a patch to fix that behavior.

Please note that the aforementioned behavior is still present in 14.03.x.

Regards,
Comment 1 Moe Jette 2014-08-04 05:37:05 MDT 
(In reply to Mehdi Dogguy from comment #0)
> Attached a patch to fix that behavior.

I do not see an attached patch. Could you try again. (There is an "Attachments" box with an "Add an attachment" link).
Comment 2 Mehdi Dogguy 2014-08-04 20:07:05 MDT 
Created attachment 1106 [details]
Patch to contribs/pam/pam_slurm.c
Comment 3 Mehdi Dogguy 2014-08-04 20:15:44 MDT 
I've added the attachment now.
Comment 4 David Bigagli 2014-08-05 05:51:31 MDT 
Fixed in commit: 40dabac254e6.

Thanks, David