Bug 5594

Summary:	su and sulogin should use PAM
Product:	Busybox	Reporter:	Xj <xj>
Component:	Standard Compliance	Assignee:	unassigned
Status:	RESOLVED WONTFIX
Severity:	enhancement	CC:	busybox-cvs
Priority:	P5
Version:	unspecified
Target Milestone:	---
Hardware:	PC
OS:	Linux
Host:		Target:
Build:
Attachments:	patch su to use pam if ENABLE_PAM is defined

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)