Bug 5594 - su and sulogin should use PAM
Summary: su and sulogin should use PAM
Status: RESOLVED WONTFIX
Alias: None
Product: Busybox
Classification: Unclassified
Component: Standard Compliance (show other bugs)
Version: unspecified
Hardware: PC Linux
: P5 enhancement
Target Milestone: ---
Assignee: unassigned
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-10-10 16:57 UTC by Xj
Modified: 2018-11-09 14:32 UTC (History)
1 user (show)

See Also:
Host:
Target:
Build:

Attachments
patch su to use pam if ENABLE_PAM is defined (3.20 KB, patch)
2018-11-09 14:32 UTC, Lionel Gimbert 		Details
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 Xj 2012-10-10 16:57:48 UTC 
Hi,

While busybox is compile with PAM support only "login" use it. su and sulogin should also use it.
Comment 1 Denys Vlasenko 2012-10-12 13:22:21 UTC 
Patches are welcome.
Comment 2 Lionel Gimbert 2018-11-09 14:32:13 UTC 
Created attachment 7871 [details]
patch su to use pam if ENABLE_PAM is defined

This patch partially fixes the issue (only add pam support to "su", not "sulogin").

It is mostly source code taken from "networking/httpd.c” and” libbb/correct_password.c" and lightly adapted for "su".

This patch was made as part of a development for Eaton (opensource.eaton.com)