Skip to contentSkip to conversation

Fix file permission for pam configuration files

PATCH New Chen Qi 1 participant 3 messages Oct 19, 2015

lore
Share & tools
lore

Chen Qi

author
#0 lore raw

cover Fix file permission for pam configuration files

The following changes since commit 10e5df3503632a6e1c54612055b19f7258c3ae2f:

lib/oe/image.py: Fix dependency handling for compressed types (2015-10-14 18:08:22 +0300)

are available in the git repository at:

git://git.openembedded.org/openembedded-core-contrib ChenQi/etc-pam
http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=ChenQi/etc-pam

Chen Qi (2):
openssh: fix file permission for /etc/pam.d/sshd
sudo: fix file permission for /etc/pam.d/sudo

 meta/recipes-connectivity/openssh/openssh_7.1p1.bb | 2 +-
 meta/recipes-extended/sudo/sudo_1.8.14p3.bb        | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
Signature · 1 line
1.9.1
author
#1 lore raw

PATCH 1/2 openssh: fix file permission for /etc/pam.d/sshd

The file permission should be 0644 instead of 0755.

---
 meta/recipes-connectivity/openssh/openssh_7.1p1.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
meta/recipes-connectivity/openssh/openssh_7.1p1.bb +1 -1
index eeeb4b4..40938cc 100644
@@ -87,7 +87,7 @@ do_compile_ptest() {
 
 do_install_append () {
 	if [ "${@bb.utils.contains('DISTRO_FEATURES', 'pam', 'pam', '', d)}" = "pam" ]; then
-		install -D -m 0755 ${WORKDIR}/sshd ${D}${sysconfdir}/pam.d/sshd
+		install -D -m 0644 ${WORKDIR}/sshd ${D}${sysconfdir}/pam.d/sshd
 		sed -i -e 's:#UsePAM no:UsePAM yes:' ${WORKDIR}/sshd_config ${D}${sysconfdir}/ssh/sshd_config
 	fi
 
Signature · 1 line
1.9.1
author
#2 lore raw

PATCH 2/2 sudo: fix file permission for /etc/pam.d/sudo

In reply to Chen Qi

The file permission should be 0644 instead of 0664.

---
 meta/recipes-extended/sudo/sudo_1.8.14p3.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
meta/recipes-extended/sudo/sudo_1.8.14p3.bb +1 -1
index 6b3cd6d..b93112f 100644
@@ -22,7 +22,7 @@ EXTRA_OECONF += " \
 
 do_install_append () {
 	if [ "${@bb.utils.contains('DISTRO_FEATURES', 'pam', 'pam', '', d)}" = "pam" ]; then
-		install -D -m 664 ${WORKDIR}/sudo.pam ${D}/${sysconfdir}/pam.d/sudo
+		install -D -m 644 ${WORKDIR}/sudo.pam ${D}/${sysconfdir}/pam.d/sudo
 	fi
 
 	chmod 4111 ${D}${bindir}/sudo
Signature · 1 line
1.9.1