From 82b1dafad17904bc224b0632486006fe88301b57 Mon Sep 17 00:00:00 2001 From: Jeremy Allison Date: Wed, 18 Dec 2013 13:56:18 -0800 Subject: [PATCH] CVE-2013-6442: s3:smbcacls - ensure we don't lose an existing ACL when setting owner or group owner. Bug: https://bugzilla.samba.org/show_bug.cgi?id=10327 Bug 10327 - CVE-2013-6442: smbcacls --chown | --chgrp dacl regression Signed-off-by: Jeremy Allison Reviewed-by: Stefan Metzmacher --- source3/utils/smbcacls.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/source3/utils/smbcacls.c b/source3/utils/smbcacls.c index 11b7388..e3b7099 100644 --- a/source3/utils/smbcacls.c +++ b/source3/utils/smbcacls.c @@ -990,7 +990,7 @@ static int owner_set(struct cli_state *cli, enum chown_mode change_mode, return EXIT_FAILED; } - sd = make_sec_desc(talloc_tos(),old->revision, old->type, + sd = make_sec_desc(talloc_tos(),old->revision, SEC_DESC_SELF_RELATIVE, (change_mode == REQUEST_CHOWN) ? &sid : NULL, (change_mode == REQUEST_CHGRP) ? &sid : NULL, NULL, NULL, &sd_size); -- 1.8.5.1