Skip to content

Commit

Permalink
Fix "Wrong permissions ownership of installed binaries" (references #10)
Browse files Browse the repository at this point in the history
  • Loading branch information
@davidcoutadeur
davidcoutadeur committed Mar 31, 2018
1 parent fcf9a7b commit af7cbc9
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions debian/paquet-openldap-debian/openldap-ltb-2.4.46/debian/openldap-ltb.postinst
View file
Original file line number Diff line number Diff line change
Expand Up @@ -26,13 +26,13 @@ fi


# Create some dirs and change owner
/bin/chown -R ${LDAPUSER}:${LDAPGROUP} ${LDAPSERVERDIR}
mkdir -p ${LDAPDATADIR}
/bin/chown -R ${LDAPUSER}:${LDAPGROUP} ${LDAPDATADIR}
mkdir -p ${LDAPLOGSDIR}
/bin/chown -R ${LDAPUSER}:${LDAPGROUP} ${LDAPLOGSDIR}
mkdir -p ${LDAPBACKUPDIR}
/bin/chown -R ${LDAPUSER}:${LDAPGROUP} ${LDAPBACKUPDIR}

This comment has been minimized.

Sign in to view

Copy link
@coudot

coudot Apr 2, 2018

Member

Hello @davidcoutadeur, are you sure we can remove rights on backup dir? We still use 'su' in slapd-cli to do backup and restore
/bin/chown -R root:root ${LDAPSERVERDIR}
/bin/chown -R ${LDAPUSER}:${LDAPGROUP} ${LDAPDATADIR}
/bin/chown -R ${LDAPUSER}:${LDAPGROUP} ${LDAPLOGSDIR}
/bin/chown -R ${LDAPUSER}:${LDAPGROUP} ${LDAPSERVERDIR}/var/run


# Automatize init script
Expand Down

0 comments on commit af7cbc9

Please sign in to comment.