[SCM] Samba Website Repository - branch master updated

2020-03-09 Thread Andrew Bartlett
The branch, master has been updated
   via  7045c7c Remove OpenIQ support listing
   via  0a34402 Update listing for Commercial IT Services Ltd
  from  515f7a4 Update Pacific ESI e-mail

https://git.samba.org/?p=samba-web.git;a=shortlog;h=master


- Log -
commit 7045c7c98e4b78b0bee1d5cf4ffec70a9b1d33b7
Author: Andrew Bartlett 
Date:   Tue Mar 10 13:27:49 2020 +1300

Remove OpenIQ support listing

e-mail to them bounces, the domain redirects to another organisation where
the e-mail also bounces.  Web enquiry also not responded to.

Signed-off-by: Andrew Bartlett 

commit 0a34402d63f64544216bfc006c699c62ee67c522
Author: Andrew Bartlett 
Date:   Tue Mar 10 13:26:22 2020 +1300

Update listing for Commercial IT Services Ltd

Roland requested by e-mail that the old phone number be removed.

Signed-off-by: Andrew Bartlett 

---

Summary of changes:
 support/australia.html  | 19 ---
 support/newzealand.html |  1 -
 2 files changed, 20 deletions(-)


Changeset truncated at 500 lines:

diff --git a/support/australia.html b/support/australia.html
index 8bf38cf..85fe923 100644
--- a/support/australia.html
+++ b/support/australia.html
@@ -88,25 +88,6 @@ We encourage you to contact Loftus IT to explore the 
benefits of an Open Source
 
 
 
-
-
-OpenIQ Pty Ltd
-
-50 York St,
-Sydney, NSW 2000
-
-http://www.OpenIQ.com.au;>http://www.OpenIQ.com.au
-+61-2-8001-6199
-mailto:he...@openiq.com.au;>he...@openiq.com.au
-Lee Curtis
-
-
-A crack team of enterprise architects and certified engineers with a bias 
towards data centre-grade Linux. 
-
-High availability installation of NAS setups backed by enterprise single 
sign-on. 
-
-
-
 
 
 Pacific ESI
diff --git a/support/newzealand.html b/support/newzealand.html
index 2b6b0b6..38ea67b 100644
--- a/support/newzealand.html
+++ b/support/newzealand.html
@@ -40,7 +40,6 @@ beyond.
 PO Box 301094, Albany
 
 http://www.cits.co.nz
-+64-9-448 2711
 i...@cits.co.nz
 Roland Venter
 


-- 
Samba Website Repository



[SCM] Samba Shared Repository - branch master updated

2020-03-09 Thread Andrew Bartlett
The branch, master has been updated
   via  54f26cfcf25 autobuild: Run the none env in the samba-o3 build
   via  609c9903473 Require Python 3.6 for Samba 4.13
   via  e9ce0f13e69 .gitlab-ci.yml: Do not build Samba for Ubuntu 16.04 or 
Debian 9 any longer
   via  d048d7e17d7 bootstrap: Remove long-unsupported OS versions
  from  0ae4f368c6c smbd: reuse close_free_pending_aio() in 
close_directory()

https://git.samba.org/?p=samba.git;a=shortlog;h=master


- Log -
commit 54f26cfcf2587a2b1d97f466a886fa89a116eea1
Author: Andreas Schneider 
Date:   Fri Nov 22 15:23:35 2019 +0100

autobuild: Run the none env in the samba-o3 build

This includes tests which should make sure that certain code is not
optimized away, like memset_s().

Signed-off-by: Andreas Schneider 
Reviewed-by: Andrew Bartlett 

Autobuild-User(master): Andrew Bartlett 
Autobuild-Date(master): Mon Mar  9 23:42:26 UTC 2020 on sn-devel-184

commit 609c9903473b139fdf1b2ed02b9c93b648eeea8c
Author: Andrew Bartlett 
Date:   Wed Mar 4 13:51:23 2020 +1300

Require Python 3.6 for Samba 4.13

This allows Samba to use formatted string literals, which
are quite handy.

REF: https://docs.python.org/3/whatsnew/3.6.html#whatsnew36-pep498

Signed-off-by: Andrew Bartlett 
Reviewed-by: Andreas Schneider 

commit e9ce0f13e695f1d7e719923628255ea786a90c20
Author: Andrew Bartlett 
Date:   Wed Mar 4 13:55:27 2020 +1300

.gitlab-ci.yml: Do not build Samba for Ubuntu 16.04 or Debian 9 any longer

These only have Python 3.5 and we want to increase the minimum to Python 
3.6.

Signed-off-by: Andrew Bartlett 
Reviewed-by: Andreas Schneider 

commit d048d7e17d756099e208fa4d6b931a147b0b1489
Author: Andrew Bartlett 
Date:   Wed Mar 4 13:58:48 2020 +1300

bootstrap: Remove long-unsupported OS versions

Samba has not built on these versions for quite some time due to
the need for Python 3.5 and GnuTLS 3.4.7

These were always marked as broken, but given the requirements
these are never likely to come back.

Signed-off-by: Andrew Bartlett 
Reviewed-by: Andreas Schneider 

---

Summary of changes:
 .gitlab-ci.yml|  14 +--
 WHATSNEW.txt  |  11 +++
 bootstrap/.gitlab-ci.yml  |  18 
 bootstrap/config.py   |  83 
 bootstrap/generated-dists/Vagrantfile |  42 -
 bootstrap/generated-dists/centos6/Dockerfile  |  27 --
 bootstrap/generated-dists/centos6/bootstrap.sh| 109 --
 bootstrap/generated-dists/centos6/locale.sh   |  55 ---
 bootstrap/generated-dists/centos6/packages.yml|  89 --
 bootstrap/generated-dists/debian7/Dockerfile  |  27 --
 bootstrap/generated-dists/debian7/bootstrap.sh| 101 
 bootstrap/generated-dists/debian7/locale.sh   |  55 ---
 bootstrap/generated-dists/debian7/packages.yml|  86 -
 bootstrap/generated-dists/debian8/Dockerfile  |  27 --
 bootstrap/generated-dists/debian8/bootstrap.sh| 105 -
 bootstrap/generated-dists/debian8/locale.sh   |  55 ---
 bootstrap/generated-dists/debian8/packages.yml|  90 --
 bootstrap/generated-dists/debian9/Dockerfile  |  27 --
 bootstrap/generated-dists/debian9/bootstrap.sh| 105 -
 bootstrap/generated-dists/debian9/locale.sh   |  55 ---
 bootstrap/generated-dists/debian9/packages.yml|  90 --
 bootstrap/generated-dists/ubuntu1404/Dockerfile   |  27 --
 bootstrap/generated-dists/ubuntu1404/bootstrap.sh | 103 
 bootstrap/generated-dists/ubuntu1404/locale.sh|  55 ---
 bootstrap/generated-dists/ubuntu1404/packages.yml |  88 -
 bootstrap/generated-dists/ubuntu1604/Dockerfile   |  27 --
 bootstrap/generated-dists/ubuntu1604/bootstrap.sh | 104 -
 bootstrap/generated-dists/ubuntu1604/locale.sh|  55 ---
 bootstrap/generated-dists/ubuntu1604/packages.yml |  89 --
 bootstrap/sha1sum.txt |   2 +-
 buildtools/wafsamba/samba_python.py   |   2 +-
 script/autobuild.py   |  11 +--
 32 files changed, 16 insertions(+), 1818 deletions(-)
 delete mode 100644 bootstrap/generated-dists/centos6/Dockerfile
 delete mode 100755 bootstrap/generated-dists/centos6/bootstrap.sh
 delete mode 100755 bootstrap/generated-dists/centos6/locale.sh
 delete mode 100644 bootstrap/generated-dists/centos6/packages.yml
 delete mode 100644 bootstrap/generated-dists/debian7/Dockerfile
 delete mode 100755 

[SCM] Samba Shared Repository - branch master updated

2020-03-09 Thread Jeremy Allison
The branch, master has been updated
   via  0ae4f368c6c smbd: reuse close_free_pending_aio() in 
close_directory()
   via  f94cd10a211 smbd: call tevent_req_nterror() for directories when 
cleaning up pending aio
   via  acb0b017613 smbd: move pending aio cleanup to a helper function
   via  95cfcda13fe vfs_default: Protect vfs_getxattrat_done() from 
accessing a freed req pointer
   via  0e894f3e482 vfs_default: pass in state as the callback data to the 
subreq
  from  54c21a99e6c winexe: add configure option to control whether to 
build it (default: auto)

https://git.samba.org/?p=samba.git;a=shortlog;h=master


- Log -
commit 0ae4f368c6c8d2c8c7aa34069007a984055df0da
Author: Ralph Boehme 
Date:   Mon Mar 9 11:18:23 2020 +0100

smbd: reuse close_free_pending_aio() in close_directory()

A directory fsp can have outstanding aio requests as well.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14301

Signed-off-by: Ralph Boehme 
Reviewed-by: Jeremy Allison 

Autobuild-User(master): Jeremy Allison 
Autobuild-Date(master): Mon Mar  9 19:34:27 UTC 2020 on sn-devel-184

commit f94cd10a211e2eae966ba4bd26921556bbe513fc
Author: Ralph Boehme 
Date:   Mon Mar 9 11:31:00 2020 +0100

smbd: call tevent_req_nterror() for directories when cleaning up pending aio

smbd_smb2_query_directory_recv() calls tevent_req_is_nterror() which 
requires a
NTSTATUS error code.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14301

Signed-off-by: Ralph Boehme 
Reviewed-by: Jeremy Allison 

commit acb0b01761330864a23932f643f7ad4e3d374634
Author: Ralph Boehme 
Date:   Mon Mar 9 11:16:37 2020 +0100

smbd: move pending aio cleanup to a helper function

We'll be reusing this from close_directory() in the next commit.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14301

Signed-off-by: Ralph Boehme 
Reviewed-by: Jeremy Allison 

commit 95cfcda13fe9a70b9955a7c44173d619eacb34c1
Author: Ralph Boehme 
Date:   Mon Mar 9 11:54:37 2020 +0100

vfs_default: Protect vfs_getxattrat_done() from accessing a freed req 
pointer

If the fsp is forced closed by a SHUTDOWN_CLOSE whilst the request is in
flight (share forced closed by smbcontrol), then we set state->req = NULL 
in the
state destructor.

The existing state destructor prevents the state memory from being freed, so
when the thread completes and calls vfs_getxattrat_done(), just throw away 
the result
if state->req == NULL.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14301

Signed-off-by: Ralph Boehme 
Reviewed-by: Jeremy Allison 

commit 0e894f3e48285415f72cf7a68e26f1802fe8045d
Author: Ralph Boehme 
Date:   Mon Mar 9 11:54:28 2020 +0100

vfs_default: pass in state as the callback data to the subreq

Find the req we're finishing off by looking inside the state.  In a shutdown
close the caller calls talloc_free(req), so we can't access it directly as
callback data.

The next commit will NULL out the state->req pointer when a caller calls
talloc_free(req), and the request is still in flight.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14301

Signed-off-by: Ralph Boehme 
Reviewed-by: Jeremy Allison 

---

Summary of changes:
 source3/modules/vfs_default.c |  28 ++--
 source3/smbd/close.c  | 152 --
 2 files changed, 94 insertions(+), 86 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/modules/vfs_default.c b/source3/modules/vfs_default.c
index fac7fa30ab7..cf24e66e048 100644
--- a/source3/modules/vfs_default.c
+++ b/source3/modules/vfs_default.c
@@ -3289,6 +3289,15 @@ struct vfswrap_getxattrat_state {
 static int vfswrap_getxattrat_state_destructor(
struct vfswrap_getxattrat_state *state)
 {
+   /*
+* This destructor only gets called if the request is still
+* in flight, which is why we deny it by returning -1. We
+* also set the req pointer to NULL so the _done function
+* can detect the caller doesn't want the result anymore.
+*
+* Forcing the fsp closed by a SHUTDOWN_CLOSE can cause this.
+*/
+   state->req = NULL;
return -1;
 }
 
@@ -3408,7 +3417,7 @@ static struct tevent_req *vfswrap_getxattrat_send(
if (tevent_req_nomem(subreq, req)) {
return tevent_req_post(req, ev);
}
-   tevent_req_set_callback(subreq, vfswrap_getxattrat_done, req);
+   tevent_req_set_callback(subreq, vfswrap_getxattrat_done, state);
 
talloc_set_destructor(state, vfswrap_getxattrat_state_destructor);
 
@@ -3503,10 +3512,9 @@ end_profile:
 
 static void vfswrap_getxattrat_done(struct tevent_req *subreq)
 {
-   struct tevent_req *req 

[SCM] Samba Shared Repository - branch master updated

2020-03-09 Thread Andreas Schneider
The branch, master has been updated
   via  54c21a99e6c winexe: add configure option to control whether to 
build it (default: auto)
   via  ebda529b591 librpc: fix IDL for svcctl_ChangeServiceConfigW
   via  c3fa0b2df9f s4-torture: add ndr svcctl testsuite
   via  0825324bc75 s4-torture: add rpc test for ChangeServiceConfigW
  from  f92af661902 vfs_recycle: prevent flooding the log if we're called 
on non-existant paths

https://git.samba.org/?p=samba.git;a=shortlog;h=master


- Log -
commit 54c21a99e6ca54bdb963c70d322f6778b57a384f
Author: Günther Deschner 
Date:   Wed Mar 4 18:51:01 2020 +0100

winexe: add configure option to control whether to build it (default: auto)

Guenther

Signed-off-by: Guenther Deschner 
Reviewed-by: Andreas Schneider 

Autobuild-User(master): Andreas Schneider 
Autobuild-Date(master): Mon Mar  9 16:27:21 UTC 2020 on sn-devel-184

commit ebda529b59105e9b70cc74377fe4d54cc16b4f37
Author: Günther Deschner 
Date:   Wed Mar 4 15:23:43 2020 +0100

librpc: fix IDL for svcctl_ChangeServiceConfigW

Found while trying to run winexe against Windows Server 2019.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14313

Guenther

Signed-off-by: Guenther Deschner 
Reviewed-by: Andreas Schneider 

commit c3fa0b2df9fc53dddcc3160b6a3dc751bbb389a4
Author: Günther Deschner 
Date:   Thu Mar 5 20:42:21 2020 +0100

s4-torture: add ndr svcctl testsuite

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14313

Guenther

Signed-off-by: Guenther Deschner 
Reviewed-by: Andreas Schneider 

commit 0825324bc75d2ab10164a1f137be782d84c822b8
Author: Günther Deschner 
Date:   Thu Mar 5 22:45:48 2020 +0100

s4-torture: add rpc test for ChangeServiceConfigW

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14313

Guenther

Signed-off-by: Guenther Deschner 
Reviewed-by: Andreas Schneider 

---

Summary of changes:
 examples/winexe/winexe.c  |  2 +
 examples/winexe/wscript_build |  3 +-
 librpc/idl/svcctl.idl | 25 +---
 selftest/knownfail|  3 ++
 source3/wscript   | 17 +
 source4/torture/ndr/ndr.c |  1 +
 source4/torture/ndr/svcctl.c  | 88 +++
 source4/torture/rpc/svcctl.c  | 81 ++-
 source4/torture/wscript_build |  1 +
 9 files changed, 213 insertions(+), 8 deletions(-)
 create mode 100644 source4/torture/ndr/svcctl.c


Changeset truncated at 500 lines:

diff --git a/examples/winexe/winexe.c b/examples/winexe/winexe.c
index 22f748b1d45..fc6b15f8e52 100644
--- a/examples/winexe/winexe.c
+++ b/examples/winexe/winexe.c
@@ -625,8 +625,10 @@ static NTSTATUS winexe_svc_install(
NULL,   /* load_order_group */
NULL,   /* tag_id */
NULL,   /* dependencies */
+   0,  /* dwDependSize */
NULL,   /* service_start_name */
NULL,   /* password */
+   0,  /* dwPwSize */
NULL,   /* display_name */
);
 
diff --git a/examples/winexe/wscript_build b/examples/winexe/wscript_build
index 43c09717e3d..559ed3fc706 100644
--- a/examples/winexe/wscript_build
+++ b/examples/winexe/wscript_build
@@ -106,4 +106,5 @@ if winexesvc_binaries != '':
   LOADPARM_CTX
   libsmb
   msrpc3
-  ''')
+  ''',
+  enabled=bld.env.build_winexe)
diff --git a/librpc/idl/svcctl.idl b/librpc/idl/svcctl.idl
index 671a1dc47be..a9dd3dec990 100644
--- a/librpc/idl/svcctl.idl
+++ b/librpc/idl/svcctl.idl
@@ -13,6 +13,17 @@ import "misc.idl", "security.idl";
   helpstring("Service Control")
 ] interface svcctl
 {
+   const int MAX_SERVICE_NAME_LENGTH = 256;
+   const short SC_MAX_DEPEND_SIZE = 4 * 1024;
+   const short SC_MAX_NAME_LENGTH = MAX_SERVICE_NAME_LENGTH + 1;
+   const short SC_MAX_PATH_LENGTH = 32 * 1024;
+   const short SC_MAX_PWD_SIZE = 514;
+   const short SC_MAX_COMPUTER_NAME_LENGTH = 1024;
+   const short SC_MAX_ACCOUNT_NAME_LENGTH = 2 * 1024;
+   const short SC_MAX_COMMENT_LENGTH = 128;
+   const short SC_MAX_ARGUMENT_LENGTH = 1024;
+   const short SC_MAX_ARGUMENTS = 1024;
+
typedef struct {
uint32 is_locked;
[string,charset(UTF16)] uint16 *lock_owner;
@@ -188,18 +199,20 @@ import "misc.idl", "security.idl";
SVCCTL_DISABLED = 0x0004
} svcctl_StartType;
 
-   WERROR svcctl_ChangeServiceConfigW(
+   [public] WERROR svcctl_ChangeServiceConfigW(
  

[SCM] Samba Shared Repository - branch master updated

2020-03-09 Thread Isaac Boukris
The branch, master has been updated
   via  f92af661902 vfs_recycle: prevent flooding the log if we're called 
on non-existant paths
  from  be90ab01bb4 ctdb-docs: Improve recovery lock documentation

https://git.samba.org/?p=samba.git;a=shortlog;h=master


- Log -
commit f92af66190281a04c6687eb91fea3b41c616c19e
Author: Ralph Boehme 
Date:   Fri Mar 6 12:22:25 2020 +0100

vfs_recycle: prevent flooding the log if we're called on non-existant paths

vfs_recycle is assuming that any path passed to unlink must exist, 
otherwise it
logs this error. Turn this into a DEBUG level message.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14316
See also: https://bugzilla.redhat.com/show_bug.cgi?id=1780802

Signed-off-by: Ralph Boehme 
Reviewed-by: Isaac Boukris 

Autobuild-User(master): Isaac Boukris 
Autobuild-Date(master): Mon Mar  9 14:15:06 UTC 2020 on sn-devel-184

---

Summary of changes:
 source3/modules/vfs_recycle.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/modules/vfs_recycle.c b/source3/modules/vfs_recycle.c
index a1d32bf10cb..ab1e6aa4dcf 100644
--- a/source3/modules/vfs_recycle.c
+++ b/source3/modules/vfs_recycle.c
@@ -236,8 +236,8 @@ static off_t recycle_get_file_size(vfs_handle_struct 
*handle,
}
 
if (SMB_VFS_STAT(handle->conn, smb_fname_tmp) != 0) {
-   DEBUG(0,("recycle: stat for %s returned %s\n",
-smb_fname_str_dbg(smb_fname_tmp), strerror(errno)));
+   DBG_DEBUG("stat for %s returned %s\n",
+smb_fname_str_dbg(smb_fname_tmp), strerror(errno));
size = (off_t)0;
goto out;
}


-- 
Samba Shared Repository