Revert "s4:tls_tstream: allow mode of SSL keyfile to be 0400, not only 0600"
authorStefan Metzmacher <metze@samba.org>
Fri, 28 Mar 2014 09:24:56 +0000 (10:24 +0100)
committerStefan Metzmacher <metze@samba.org>
Fri, 28 Mar 2014 11:37:17 +0000 (12:37 +0100)
commita2c34798782a1e4783c258d4e1950a2150d70e18
tree5e3937f7976cd89767f411a3a7d1299e6474d2d3
parent0dd648a54ed00bfeb7c6074d16173128acd3799e
Revert "s4:tls_tstream: allow mode of SSL keyfile to be 0400, not only 0600"

This reverts commit 05c1fe50556e2330e23b7efb38e653428b9bdadf.

This was discussed here:
https://bugzilla.samba.org/show_bug.cgi?id=10392#c11

This generated warnings like:
invalid permissions on file
'/memdisk/metze/W/b138235/samba/bin/ab/promoted_dc/private/tls/key.pem': has
0600 should be 0400'.

I think we need a better way. Maybe file_check_permissions()
should get allow_perms and deny_perms. And we would call it
with allow_perms = 0400 and deny_perms = 0177. And bits in none
of them are ignored.

For now we revert this and wait for a better fix.

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Autobuild-User(master): Stefan Metzmacher <metze@samba.org>
Autobuild-Date(master): Fri Mar 28 12:37:17 CET 2014 on sn-devel-104
source4/lib/tls/tls_tstream.c