author | Jiri Sasek <Jiri.Sasek@Oracle.COM> |
Thu, 28 Jan 2016 19:04:31 -0800 | |
branch | s11u3-sru |
changeset 5462 | 3717a6234e82 |
permissions | -rw-r--r-- |
5462
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
1 |
Patch source: https://www.samba.org/samba/ftp/patches/security/samba-3.6.25-security-2015-12-16.patch |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
2 |
Community info: https://www.samba.org/samba/history/security.html |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
3 |
................................................................................ |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
4 |
From 2e94b6ec10f1d15e24867bab3063bb85f173406a Mon Sep 17 00:00:00 2001 |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
5 |
From: Jeremy Allison <[email protected]> |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
6 |
Date: Thu, 9 Jul 2015 10:58:11 -0700 |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
7 |
Subject: [PATCH] CVE-2015-5252: s3: smbd: Fix symlink verification (file |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
8 |
access outside the share). |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
9 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
10 |
Ensure matching component ends in '/' or '\0'. |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
11 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
12 |
BUG: https://bugzilla.samba.org/show_bug.cgi?id=11395 |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
13 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
14 |
Signed-off-by: Jeremy Allison <[email protected]> |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
15 |
Reviewed-by: Volker Lendecke <[email protected]> |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
16 |
--- |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
17 |
source3/smbd/vfs.c | 7 +++++-- |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
18 |
1 file changed, 5 insertions(+), 2 deletions(-) |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
19 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
20 |
diff --git a/source3/smbd/vfs.c b/source3/smbd/vfs.c |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
21 |
index 6c56964..bd93b7f 100644 |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
22 |
--- a/source3/smbd/vfs.c |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
23 |
+++ b/source3/smbd/vfs.c |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
24 |
@@ -982,6 +982,7 @@ NTSTATUS check_reduced_name(connection_struct *conn, const char *fname) |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
25 |
if (!allow_widelinks || !allow_symlinks) { |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
26 |
const char *conn_rootdir; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
27 |
size_t rootdir_len; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
28 |
+ bool matched; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
29 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
30 |
conn_rootdir = SMB_VFS_CONNECTPATH(conn, fname); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
31 |
if (conn_rootdir == NULL) { |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
32 |
@@ -992,8 +993,10 @@ NTSTATUS check_reduced_name(connection_struct *conn, const char *fname) |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
33 |
} |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
34 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
35 |
rootdir_len = strlen(conn_rootdir); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
36 |
- if (strncmp(conn_rootdir, resolved_name, |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
37 |
- rootdir_len) != 0) { |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
38 |
+ matched = (strncmp(conn_rootdir, resolved_name, |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
39 |
+ rootdir_len) == 0); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
40 |
+ if (!matched || (resolved_name[rootdir_len] != '/' && |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
41 |
+ resolved_name[rootdir_len] != '\0')) { |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
42 |
DEBUG(2, ("check_reduced_name: Bad access " |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
43 |
"attempt: %s is a symlink outside the " |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
44 |
"share path\n", fname)); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
45 |
-- |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
46 |
2.5.0 |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
47 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
48 |
From 25139116756cc285a3a5534834cc276ef1b7baaa Mon Sep 17 00:00:00 2001 |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
49 |
From: Stefan Metzmacher <[email protected]> |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
50 |
Date: Wed, 30 Sep 2015 21:17:02 +0200 |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
51 |
Subject: [PATCH 1/2] CVE-2015-5296: s3:libsmb: force signing when requiring |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
52 |
encryption in do_connect() |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
53 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
54 |
BUG: https://bugzilla.samba.org/show_bug.cgi?id=11536 |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
55 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
56 |
Signed-off-by: Stefan Metzmacher <[email protected]> |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
57 |
Reviewed-by: Jeremy Allison <[email protected]> |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
58 |
--- |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
59 |
source3/libsmb/clidfs.c | 7 ++++++- |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
60 |
1 file changed, 6 insertions(+), 1 deletion(-) |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
61 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
62 |
diff --git a/source3/libsmb/clidfs.c b/source3/libsmb/clidfs.c |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
63 |
index 23e1471..f153b6b 100644 |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
64 |
--- a/source3/libsmb/clidfs.c |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
65 |
+++ b/source3/libsmb/clidfs.c |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
66 |
@@ -98,6 +98,11 @@ static struct cli_state *do_connect(TALLOC_CTX *ctx, |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
67 |
const char *username; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
68 |
const char *password; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
69 |
NTSTATUS status; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
70 |
+ int signing_state = get_cmdline_auth_info_signing_state(auth_info); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
71 |
+ |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
72 |
+ if (force_encrypt) { |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
73 |
+ signing_state = Required; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
74 |
+ } |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
75 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
76 |
/* make a copy so we don't modify the global string 'service' */ |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
77 |
servicename = talloc_strdup(ctx,share); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
78 |
@@ -132,7 +137,7 @@ static struct cli_state *do_connect(TALLOC_CTX *ctx, |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
79 |
zero_sockaddr(&ss); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
80 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
81 |
/* have to open a new connection */ |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
82 |
- c = cli_initialise_ex(get_cmdline_auth_info_signing_state(auth_info)); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
83 |
+ c = cli_initialise_ex(signing_state); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
84 |
if (c == NULL) { |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
85 |
d_printf("Connection to %s failed\n", server_n); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
86 |
return NULL; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
87 |
-- |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
88 |
2.5.0 |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
89 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
90 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
91 |
From 060adb0abdeda51b8b622c6020b5dea0c8dde1cf Mon Sep 17 00:00:00 2001 |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
92 |
From: Stefan Metzmacher <[email protected]> |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
93 |
Date: Wed, 30 Sep 2015 21:17:02 +0200 |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
94 |
Subject: [PATCH 2/2] CVE-2015-5296: s3:libsmb: force signing when requiring |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
95 |
encryption in SMBC_server_internal() |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
96 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
97 |
BUG: https://bugzilla.samba.org/show_bug.cgi?id=11536 |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
98 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
99 |
Signed-off-by: Stefan Metzmacher <[email protected]> |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
100 |
Reviewed-by: Jeremy Allison <[email protected]> |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
101 |
--- |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
102 |
source3/libsmb/libsmb_server.c | 13 +++++++++++-- |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
103 |
1 file changed, 11 insertions(+), 2 deletions(-) |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
104 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
105 |
diff --git a/source3/libsmb/libsmb_server.c b/source3/libsmb/libsmb_server.c |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
106 |
index 45be660..167f2c9 100644 |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
107 |
--- a/source3/libsmb/libsmb_server.c |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
108 |
+++ b/source3/libsmb/libsmb_server.c |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
109 |
@@ -258,6 +258,7 @@ SMBC_server_internal(TALLOC_CTX *ctx, |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
110 |
const char *username_used; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
111 |
NTSTATUS status; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
112 |
char *newserver, *newshare; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
113 |
+ int signing_state = Undefined; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
114 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
115 |
zero_sockaddr(&ss); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
116 |
ZERO_STRUCT(c); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
117 |
@@ -404,8 +405,12 @@ again: |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
118 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
119 |
zero_sockaddr(&ss); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
120 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
121 |
+ if (context->internal->smb_encryption_level != SMBC_ENCRYPTLEVEL_NONE) { |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
122 |
+ signing_state = Required; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
123 |
+ } |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
124 |
+ |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
125 |
/* have to open a new connection */ |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
126 |
- if ((c = cli_initialise()) == NULL) { |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
127 |
+ if ((c = cli_initialise_ex(signing_state)) == NULL) { |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
128 |
errno = ENOMEM; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
129 |
return NULL; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
130 |
} |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
131 |
@@ -750,6 +755,7 @@ SMBC_attr_server(TALLOC_CTX *ctx, |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
132 |
ipc_srv = SMBC_find_server(ctx, context, server, "*IPC$", |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
133 |
pp_workgroup, pp_username, pp_password); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
134 |
if (!ipc_srv) { |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
135 |
+ int signing_state = Undefined; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
136 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
137 |
/* We didn't find a cached connection. Get the password */ |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
138 |
if (!*pp_password || (*pp_password)[0] == '\0') { |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
139 |
@@ -771,6 +777,9 @@ SMBC_attr_server(TALLOC_CTX *ctx, |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
140 |
if (smbc_getOptionUseCCache(context)) { |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
141 |
flags |= CLI_FULL_CONNECTION_USE_CCACHE; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
142 |
} |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
143 |
+ if (context->internal->smb_encryption_level != SMBC_ENCRYPTLEVEL_NONE) { |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
144 |
+ signing_state = Required; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
145 |
+ } |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
146 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
147 |
zero_sockaddr(&ss); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
148 |
nt_status = cli_full_connection(&ipc_cli, |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
149 |
@@ -780,7 +789,7 @@ SMBC_attr_server(TALLOC_CTX *ctx, |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
150 |
*pp_workgroup, |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
151 |
*pp_password, |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
152 |
flags, |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
153 |
- Undefined); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
154 |
+ signing_state); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
155 |
if (! NT_STATUS_IS_OK(nt_status)) { |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
156 |
DEBUG(1,("cli_full_connection failed! (%s)\n", |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
157 |
nt_errstr(nt_status))); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
158 |
-- |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
159 |
2.5.0 |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
160 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
161 |
From 8e49de7754f7171a58a1f94dee0f1138dbee3c60 Mon Sep 17 00:00:00 2001 |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
162 |
From: Jeremy Allison <[email protected]> |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
163 |
Date: Fri, 23 Oct 2015 14:54:31 -0700 |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
164 |
Subject: [PATCH] CVE-2015-5299: s3-shadow-copy2: fix missing access check on |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
165 |
snapdir |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
166 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
167 |
Fix originally from <[email protected]> |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
168 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
169 |
https://bugzilla.samba.org/show_bug.cgi?id=11529 |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
170 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
171 |
Signed-off-by: Jeremy Allison <[email protected]> |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
172 |
Reviewed-by: David Disseldorp <[email protected]> |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
173 |
--- |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
174 |
source3/modules/vfs_shadow_copy2.c | 47 ++++++++++++++++++++++++++++++++++++++ |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
175 |
1 file changed, 47 insertions(+) |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
176 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
177 |
diff --git a/source3/modules/vfs_shadow_copy2.c b/source3/modules/vfs_shadow_copy2.c |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
178 |
index fedfb53..16c1ed7 100644 |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
179 |
--- a/source3/modules/vfs_shadow_copy2.c |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
180 |
+++ b/source3/modules/vfs_shadow_copy2.c |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
181 |
@@ -21,6 +21,8 @@ |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
182 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
183 |
#include "includes.h" |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
184 |
#include "smbd/smbd.h" |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
185 |
+#include "smbd/globals.h" |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
186 |
+#include "../libcli/security/security.h" |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
187 |
#include "system/filesys.h" |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
188 |
#include "ntioctl.h" |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
189 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
190 |
@@ -764,6 +766,43 @@ static int shadow_copy2_mkdir(vfs_handle_struct *handle, const char *fname, mod |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
191 |
SHADOW2_NEXT(MKDIR, (handle, name, mode), int, -1); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
192 |
} |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
193 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
194 |
+static bool check_access_snapdir(struct vfs_handle_struct *handle, |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
195 |
+ const char *path) |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
196 |
+{ |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
197 |
+ struct smb_filename smb_fname; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
198 |
+ int ret; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
199 |
+ NTSTATUS status; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
200 |
+ uint32_t access_granted = 0; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
201 |
+ |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
202 |
+ ZERO_STRUCT(smb_fname); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
203 |
+ smb_fname.base_name = talloc_asprintf(talloc_tos(), |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
204 |
+ "%s", |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
205 |
+ path); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
206 |
+ if (smb_fname.base_name == NULL) { |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
207 |
+ return false; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
208 |
+ } |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
209 |
+ |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
210 |
+ ret = SMB_VFS_NEXT_STAT(handle, &smb_fname); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
211 |
+ if (ret != 0 || !S_ISDIR(smb_fname.st.st_ex_mode)) { |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
212 |
+ TALLOC_FREE(smb_fname.base_name); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
213 |
+ return false; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
214 |
+ } |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
215 |
+ |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
216 |
+ status = smbd_check_open_rights(handle->conn, |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
217 |
+ &smb_fname, |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
218 |
+ SEC_DIR_LIST, |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
219 |
+ &access_granted); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
220 |
+ if (!NT_STATUS_IS_OK(status)) { |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
221 |
+ DEBUG(0,("user does not have list permission " |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
222 |
+ "on snapdir %s\n", |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
223 |
+ smb_fname.base_name)); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
224 |
+ TALLOC_FREE(smb_fname.base_name); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
225 |
+ return false; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
226 |
+ } |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
227 |
+ TALLOC_FREE(smb_fname.base_name); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
228 |
+ return true; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
229 |
+} |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
230 |
+ |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
231 |
static int shadow_copy2_rmdir(vfs_handle_struct *handle, const char *fname) |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
232 |
{ |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
233 |
SHADOW2_NEXT(RMDIR, (handle, name), int, -1); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
234 |
@@ -877,6 +916,7 @@ static int shadow_copy2_get_shadow_copy2_data(vfs_handle_struct *handle, |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
235 |
SMB_STRUCT_DIRENT *d; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
236 |
TALLOC_CTX *tmp_ctx = talloc_new(handle->data); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
237 |
char *snapshot; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
238 |
+ bool ret; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
239 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
240 |
snapdir = shadow_copy2_find_snapdir(tmp_ctx, handle); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
241 |
if (snapdir == NULL) { |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
242 |
@@ -886,6 +926,13 @@ static int shadow_copy2_get_shadow_copy2_data(vfs_handle_struct *handle, |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
243 |
talloc_free(tmp_ctx); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
244 |
return -1; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
245 |
} |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
246 |
+ ret = check_access_snapdir(handle, snapdir); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
247 |
+ if (!ret) { |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
248 |
+ DEBUG(0,("access denied on listing snapdir %s\n", snapdir)); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
249 |
+ errno = EACCES; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
250 |
+ talloc_free(tmp_ctx); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
251 |
+ return -1; |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
252 |
+ } |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
253 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
254 |
p = SMB_VFS_NEXT_OPENDIR(handle, snapdir, NULL, 0); |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
255 |
|
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
256 |
-- |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
257 |
2.5.0 |
3717a6234e82
22526544 problem in UTILITY/SAMBA
Jiri Sasek <Jiri.Sasek@Oracle.COM>
parents:
diff
changeset
|
258 |