[SCM] Samba Shared Repository - branch master updated

2012-03-05 Thread Jelmer Vernooij
The branch, master has been updated
   via  f033c22 policy/python: Move python binding tests to standard 
location.
   via  aaffd99 selftest/tests.py: Fix name of selftest.py for benefit of 
--load-list.
   via  b9ac1f4 selftest.py: Only run teardown_all when actually relevant.
  from  074ee6f s3-rpc_server: Remove remaining code for embedded endpoint 
mapper

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


- Log -
commit f033c221f8f459759aee259be8f52a79fd7e8f3f
Author: Jelmer Vernooij 
Date:   Tue Mar 6 00:46:55 2012 +0100

policy/python: Move python binding tests to standard location.

Autobuild-User: Jelmer Vernooij 
Autobuild-Date: Tue Mar  6 02:26:29 CET 2012 on sn-devel-104

commit aaffd996abd9404bb942570f49ca01598ebe0fc3
Author: Jelmer Vernooij 
Date:   Tue Mar 6 00:39:11 2012 +0100

selftest/tests.py: Fix name of selftest.py for benefit of --load-list.

commit b9ac1f45a14b9783602fe4508b1f450dd2676bb6
Author: Jelmer Vernooij 
Date:   Tue Mar 6 00:38:54 2012 +0100

selftest.py: Only run teardown_all when actually relevant.

---

Summary of changes:
 selftest/selftest.py   |4 +---
 .../python/samba/tests/policy.py}  |0
 source4/selftest/tests.py  |4 ++--
 3 files changed, 3 insertions(+), 5 deletions(-)
 rename source4/{lib/policy/tests/python/bindings.py => 
scripting/python/samba/tests/policy.py} (100%)


Changeset truncated at 500 lines:

diff --git a/selftest/selftest.py b/selftest/selftest.py
index 26c409a..1b20441 100755
--- a/selftest/selftest.py
+++ b/selftest/selftest.py
@@ -538,12 +538,10 @@ else:
 
 if opts.resetup_environment:
 env_manager.teardown_env(envname)
+env_manager.teardown_all()
 
 sys.stdout.write("\n")
 
-if not opts.list:
-env_manager.teardown_all()
-
 # if there were any valgrind failures, show them
 for fn in os.listdir(prefix):
 if fn.startswith("valgrind.log"):
diff --git a/source4/lib/policy/tests/python/bindings.py 
b/source4/scripting/python/samba/tests/policy.py
similarity index 100%
rename from source4/lib/policy/tests/python/bindings.py
rename to source4/scripting/python/samba/tests/policy.py
diff --git a/source4/selftest/tests.py b/source4/selftest/tests.py
index bc8bf12..d49f253 100755
--- a/source4/selftest/tests.py
+++ b/source4/selftest/tests.py
@@ -402,7 +402,7 @@ def planoldpythontestsuite(env, module, name=None, 
extra_path=[], environ={}, ex
 name = module
 plantestsuite(name, env, args)
 
-planpythontestsuite("none", "selftest.tests.test_suite", 
name="selftest.py.tests", extra_path=[srcdir()])
+planpythontestsuite("none", "selftest.tests.test_suite", extra_path=[srcdir()])
 planpythontestsuite("none", "api", name="ldb.python", 
extra_path=['lib/ldb/tests/python'])
 planpythontestsuite("none", "samba.tests.credentials")
 planoldpythontestsuite("dc:local", "samba.tests.gensec", 
extra_args=['-U"$USERNAME%$PASSWORD"'])
@@ -466,7 +466,7 @@ planpythontestsuite("none", "samba.tests.ntacls")
 plantestsuite("samba4.deletetest.python(dc)", "dc", 
['PYTHONPATH="$PYTHONPATH:%s/lib/subunit/python:%s/lib/testtools"' % (srcdir(), 
srcdir()),
  python, 
os.path.join(samba4srcdir, "dsdb/tests/python/deletetest.py"),
  '$SERVER', 
'-U"$USERNAME%$PASSWORD"', '--workgroup=$DOMAIN'])
-planpythontestsuite("none", "bindings", 
extra_path=["%s/lib/policy/tests/python" % samba4srcdir], 
name="samba4.policy.python")
+planpythontestsuite("none", "samba.tests.policy")
 plantestsuite("samba4.blackbox.samba3dump", "none", [python, 
os.path.join(samba4srcdir, "scripting/bin/samba3dump"), 
os.path.join(samba4srcdir, "../testdata/samba3")], allow_empty_output=True)
 plantestsuite("samba4.blackbox.upgrade", "none", ["PYTHON=%s" % python, 
os.path.join(samba4srcdir, "setup/tests/blackbox_s3upgrade.sh"), 
'$PREFIX/provision'])
 plantestsuite("samba4.blackbox.provision.py", "none", ["PYTHON=%s" % python, 
os.path.join(samba4srcdir, "setup/tests/blackbox_provision.sh"), 
'$PREFIX/provision'])


-- 
Samba Shared Repository


[SCM] Samba Shared Repository - branch master updated

2012-03-05 Thread Andreas Schneider
The branch, master has been updated
   via  074ee6f s3-rpc_server: Remove remaining code for embedded endpoint 
mapper
   via  be7bcf0 s3-rpc_server: Only init and register embedded RPC services 
in dcesrv_ep_setup()
  from  cae455f s3: Fix a "Invalid (state->nread >= 0)" warning

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


- Log -
commit 074ee6f34c3c105d7c5d940b420ccd785c05867e
Author: Andrew Bartlett 
Date:   Sat Mar 3 12:49:10 2012 +1100

s3-rpc_server: Remove remaining code for embedded endpoint mapper

Signed-off-by: Andreas Schneider 

Autobuild-User: Andreas Schneider 
Autobuild-Date: Mon Mar  5 23:14:33 CET 2012 on sn-devel-104

commit be7bcf0e550229eb5ee15c65d3f50b8ddd5e9dff
Author: Andrew Bartlett 
Date:   Mon Mar 5 10:59:01 2012 +1100

s3-rpc_server: Only init and register embedded RPC services in 
dcesrv_ep_setup()

This consults the two definitions for embedded, that is if the deamon is 
forking
or if the rpc_server: line is set to embedded.

Andrew Bartlett

Signed-off-by: Andreas Schneider 

---

Summary of changes:
 source3/rpc_server/rpc_service_setup.c |  126 +---
 1 files changed, 66 insertions(+), 60 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/rpc_server/rpc_service_setup.c 
b/source3/rpc_server/rpc_service_setup.c
index fc770c5..203eeb1 100644
--- a/source3/rpc_server/rpc_service_setup.c
+++ b/source3/rpc_server/rpc_service_setup.c
@@ -53,24 +53,6 @@
 #include "rpc_server/rpc_config.h"
 #include "rpc_server/epmapper/srv_epmapper.h"
 
-static bool rpc_setup_epmapper(struct tevent_context *ev_ctx,
-  struct messaging_context *msg_ctx)
-{
-   enum rpc_service_mode_e epm_mode = rpc_epmapper_mode();
-   enum rpc_daemon_type_e epm_type = rpc_epmapper_daemon();
-   NTSTATUS status;
-
-   if (epm_mode != RPC_SERVICE_MODE_DISABLED &&
-   epm_type != RPC_DAEMON_DISABLED) {
-   status = rpc_epmapper_init(NULL);
-   if (!NT_STATUS_IS_OK(status)) {
-   return false;
-   }
-   }
-
-   return true;
-}
-
 /* Common routine for embedded RPC servers */
 static bool rpc_setup_embedded(struct tevent_context *ev_ctx,
   struct messaging_context *msg_ctx,
@@ -140,6 +122,10 @@ static bool rpc_setup_winreg(struct tevent_context *ev_ctx,
const struct ndr_interface_table *t = &ndr_table_winreg;
const char *pipe_name = "winreg";
NTSTATUS status;
+   enum rpc_service_mode_e service_mode = rpc_service_mode(t->name);
+   if (service_mode != RPC_SERVICE_MODE_EMBEDDED) {
+   return true;
+   }
 
status = rpc_winreg_init(NULL);
if (!NT_STATUS_IS_OK(status)) {
@@ -156,6 +142,10 @@ static bool rpc_setup_srvsvc(struct tevent_context *ev_ctx,
const struct ndr_interface_table *t = &ndr_table_srvsvc;
const char *pipe_name = "srvsvc";
NTSTATUS status;
+   enum rpc_service_mode_e service_mode = rpc_service_mode(t->name);
+   if (service_mode != RPC_SERVICE_MODE_EMBEDDED) {
+   return true;
+   }
 
status = rpc_srvsvc_init(NULL);
if (!NT_STATUS_IS_OK(status)) {
@@ -171,20 +161,19 @@ static bool rpc_setup_lsarpc(struct tevent_context 
*ev_ctx,
 {
const struct ndr_interface_table *t = &ndr_table_lsarpc;
const char *pipe_name = "lsarpc";
-   enum rpc_service_mode_e lsarpc_mode = rpc_lsarpc_mode();
enum rpc_daemon_type_e lsasd_type = rpc_lsasd_daemon();
NTSTATUS status;
+   enum rpc_service_mode_e service_mode = rpc_service_mode(t->name);
+   if (service_mode != RPC_SERVICE_MODE_EMBEDDED || lsasd_type != 
RPC_DAEMON_EMBEDDED) {
+   return true;
+   }
 
status = rpc_lsarpc_init(NULL);
if (!NT_STATUS_IS_OK(status)) {
return false;
}
 
-   if (lsarpc_mode == RPC_SERVICE_MODE_EMBEDDED &&
-   lsasd_type != RPC_DAEMON_DISABLED) {
-   return rpc_setup_embedded(ev_ctx, msg_ctx, v, t, pipe_name);
-   }
-   return true;
+   return rpc_setup_embedded(ev_ctx, msg_ctx, v, t, pipe_name);
 }
 
 static bool rpc_setup_samr(struct tevent_context *ev_ctx,
@@ -193,21 +182,19 @@ static bool rpc_setup_samr(struct tevent_context *ev_ctx,
 {
const struct ndr_interface_table *t = &ndr_table_samr;
const char *pipe_name = "samr";
-   enum rpc_service_mode_e samr_mode = rpc_samr_mode();
enum rpc_daemon_type_e lsasd_type = rpc_lsasd_daemon();
NTSTATUS status;
+   enum rpc_service_mode_e service_mode = rpc_service_mode(t->name);
+   if (service_mode != RPC_SERVICE_MODE_EMBEDDED || lsasd_type != 
RPC_DAEMON_EMBEDDED) {
+   return true;
+

[SCM] Samba Shared Repository - branch v3-5-test updated

2012-03-05 Thread Karolin Seeger
The branch, v3-5-test has been updated
   via  abb2dcd WHATSNEW: Start to add changes since 3.5.12.
  from  8e141d6 s3-winbindd: Close netlogon connection if the status 
returned by the NetrSamLogonEx call is timeout in the pam_auth_crap path

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=v3-5-test


- Log -
commit abb2dcde786b1656c4df1e3bbe09757d640c6549
Author: Karolin Seeger 
Date:   Mon Mar 5 21:18:13 2012 +0100

WHATSNEW: Start to add changes since 3.5.12.

To be continued...

Karolin

---

Summary of changes:
 WHATSNEW.txt |   40 +++-
 1 files changed, 39 insertions(+), 1 deletions(-)


Changeset truncated at 500 lines:

diff --git a/WHATSNEW.txt b/WHATSNEW.txt
index 224f13d..3cef783 100644
--- a/WHATSNEW.txt
+++ b/WHATSNEW.txt
@@ -1,6 +1,6 @@
==
Release Notes for Samba 3.5.13
- , 2011
+ March 12, 2012
==
 
 
@@ -14,6 +14,44 @@ Changes since 3.5.12:
 
 
 
+o   Jeremy Allison 
+* BUG 8561: Fully observe password change settings.
+* BUG 8631: Fix POSIX ACE x permission mapping to and from a DACL.
+
+
+o   Günther Deschner 
+* BUG 8176: Fix perl path.
+
+
+o   Björn Jacke 
+* BUG 8652: Document the ignore system acls option of vfs_acl_xattr and
+  vfs_acl_tdb.
+
+
+o   Jeff Layton 
+* BUG 8648: Document more undocumented mount.cifs options.
+
+
+o   Volker Lendecke 
+* BUG 8639: Fix the vfs_commit module.
+
+
+o   Stefan Metzmacher 
+* BUG 8608: Don't fail on users without a uid (Winbind).
+* BUG 8645: Add missing prefixpath options for mount.cifs manpage.
+
+
+o   Karolin Seeger 
+* BUG 7705: Fix rpm build issues on RHEL4.
+
+
+o   Richard Sharpe 
+* BUG 8607: Simplify building modules outside the Samba source tree.
 
 
 ##


-- 
Samba Shared Repository


[SCM] Samba Shared Repository - branch master updated

2012-03-05 Thread Volker Lendecke
The branch, master has been updated
   via  cae455f s3: Fix a "Invalid (state->nread >= 0)" warning
   via  216769f s3: Move the drain_socket on error to reply_write_and_X
   via  82b948a s3: Use "goto out;" in reply_write_and_X
  from  07386bb s3: Remove "size" param from switch_message

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


- Log -
commit cae455f68802be2fa31328878be4119df1c1d864
Author: Volker Lendecke 
Date:   Mon Mar 5 15:56:55 2012 +0100

s3: Fix a "Invalid (state->nread >= 0)" warning

Both read_from_internal_pipe and tstream_readv_pdu_queue_recv return
ssize_t.

Autobuild-User: Volker Lendecke 
Autobuild-Date: Mon Mar  5 17:38:16 CET 2012 on sn-devel-104

commit 216769f2ce7d18a7ca83bf440b6efd7ad4675675
Author: Volker Lendecke 
Date:   Mon Mar 5 15:40:49 2012 +0100

s3: Move the drain_socket on error to reply_write_and_X

That's the only case where this can happen, so we should not clutter the 
main
code path.

commit 82b948a8160e3b5bb80238413967ed7f5798a35c
Author: Volker Lendecke 
Date:   Mon Mar 5 15:47:52 2012 +0100

s3: Use "goto out;" in reply_write_and_X

---

Summary of changes:
 source3/rpc_server/srv_pipe_hnd.c |2 +-
 source3/smbd/process.c|9 -
 source3/smbd/reply.c  |   36 ++--
 3 files changed, 19 insertions(+), 28 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/rpc_server/srv_pipe_hnd.c 
b/source3/rpc_server/srv_pipe_hnd.c
index 3616922..e4172e7 100644
--- a/source3/rpc_server/srv_pipe_hnd.c
+++ b/source3/rpc_server/srv_pipe_hnd.c
@@ -687,7 +687,7 @@ struct np_read_state {
struct np_proxy_state *p;
struct np_ipc_readv_next_vector_state next_vector;
 
-   size_t nread;
+   ssize_t nread;
bool is_data_outstanding;
 };
 
diff --git a/source3/smbd/process.c b/source3/smbd/process.c
index 15e1efe..f61ad5d 100644
--- a/source3/smbd/process.c
+++ b/source3/smbd/process.c
@@ -1521,15 +1521,6 @@ static void construct_reply(struct 
smbd_server_connection *sconn,
 
conn = switch_message(req->cmd, req);
 
-   if (req->unread_bytes) {
-   /* writeX failed. drain socket. */
-   if (drain_socket(req->sconn->sock, req->unread_bytes) !=
-   req->unread_bytes) {
-   smb_panic("failed to drain pending bytes");
-   }
-   req->unread_bytes = 0;
-   }
-
if (req->done) {
TALLOC_FREE(req);
return;
diff --git a/source3/smbd/reply.c b/source3/smbd/reply.c
index 26a928f..e3a3766 100644
--- a/source3/smbd/reply.c
+++ b/source3/smbd/reply.c
@@ -4494,8 +4494,7 @@ void reply_write_and_X(struct smb_request *req)
 
if ((req->wct != 12) && (req->wct != 14)) {
reply_nterror(req, NT_STATUS_INVALID_PARAMETER);
-   END_PROFILE(SMBwriteX);
-   return;
+   goto out;
}
 
numtowrite = SVAL(req->vwv+10, 0);
@@ -4512,20 +4511,17 @@ void reply_write_and_X(struct smb_request *req)
/* Can't do a recvfile write on IPC$ */
if (IS_IPC(conn)) {
reply_nterror(req, NT_STATUS_INVALID_PARAMETER);
-   END_PROFILE(SMBwriteX);
-   return;
+   goto out;
}
if (numtowrite != req->unread_bytes) {
reply_nterror(req, NT_STATUS_INVALID_PARAMETER);
-   END_PROFILE(SMBwriteX);
-   return;
+   goto out;
}
} else {
if (smb_doff > smblen || smb_doff + numtowrite < numtowrite ||
smb_doff + numtowrite > smblen) {
reply_nterror(req, NT_STATUS_INVALID_PARAMETER);
-   END_PROFILE(SMBwriteX);
-   return;
+   goto out;
}
}
 
@@ -4533,12 +4529,10 @@ void reply_write_and_X(struct smb_request *req)
if (IS_IPC(conn)) {
if (req->unread_bytes) {
reply_nterror(req, NT_STATUS_INVALID_PARAMETER);
-   END_PROFILE(SMBwriteX);
-   return;
+   goto out;
}
reply_pipe_write_and_X(req);
-   END_PROFILE(SMBwriteX);
-   return;
+   goto out;
}
 
fsp = file_fsp(req, SVAL(req->vwv+2, 0));
@@ -4546,14 +4540,12 @@ void reply_write_and_X(struct smb_request *req)
write_through = BITSETW(req->vwv+7,0);
 
if (!check_fsp(conn, req, fsp)) {
-   END_PROFILE(SMBwriteX);
-   return;
+   go

[SCM] Samba Shared Repository - branch master updated

2012-03-05 Thread Volker Lendecke
The branch, master has been updated
   via  07386bb s3: Remove "size" param from switch_message
   via  c99d245 s3: Remove "size" param from smb_dump
  from  14d3137 s3-lsasd: Fix debug messages on registration failure

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


- Log -
commit 07386bb5331a22c7dcfc4fef1ae527f20ce279e6
Author: Volker Lendecke 
Date:   Mon Mar 5 13:28:07 2012 +0100

s3: Remove "size" param from switch_message

Autobuild-User: Volker Lendecke 
Autobuild-Date: Mon Mar  5 15:13:49 CET 2012 on sn-devel-104

commit c99d245548e5fcf0a89b79749e52967dc29c012c
Author: Volker Lendecke 
Date:   Mon Mar 5 13:28:07 2012 +0100

s3: Remove "size" param from smb_dump

---

Summary of changes:
 source3/smbd/process.c |   15 ---
 1 files changed, 8 insertions(+), 7 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/smbd/process.c b/source3/smbd/process.c
index 6c92755..15e1efe 100644
--- a/source3/smbd/process.c
+++ b/source3/smbd/process.c
@@ -1304,15 +1304,16 @@ void reply_outbuf(struct smb_request *req, uint8 
num_words, uint32 num_bytes)
  Dump a packet to a file.
 /
 
-static void smb_dump(const char *name, int type, const char *data, ssize_t len)
+static void smb_dump(const char *name, int type, const char *data)
 {
+   size_t len;
int fd, i;
char *fname = NULL;
if (DEBUGLEVEL < 50) {
return;
}
 
-   if (len < 4) len = smb_len(data)+4;
+   len = smb_len_tcp(data)+4;
for (i=1;i<100;i++) {
if (asprintf(&fname, "/tmp/%s.%d.%s", name, i,
 type ? "req" : "resp") == -1) {
@@ -1344,7 +1345,7 @@ static void smb_dump(const char *name, int type, const 
char *data, ssize_t len)
  find.
 /
 
-static connection_struct *switch_message(uint8 type, struct smb_request *req, 
int size)
+static connection_struct *switch_message(uint8 type, struct smb_request *req)
 {
int flags;
uint16 session_tag;
@@ -1356,7 +1357,7 @@ static connection_struct *switch_message(uint8 type, 
struct smb_request *req, in
 
if (smb_messages[type].fn == NULL) {
DEBUG(0,("Unknown message type %d!\n",type));
-   smb_dump("Unknown", 1, (const char *)req->inbuf, size);
+   smb_dump("Unknown", 1, (const char *)req->inbuf);
reply_unknown_new(req, type);
return NULL;
}
@@ -1370,7 +1371,7 @@ static connection_struct *switch_message(uint8 type, 
struct smb_request *req, in
DEBUG(3,("switch message %s (pid %d) conn 0x%lx\n", smb_fn_name(type),
 (int)sys_getpid(), (unsigned long)conn));
 
-   smb_dump(smb_fn_name(type), 1, (const char *)req->inbuf, size);
+   smb_dump(smb_fn_name(type), 1, (const char *)req->inbuf);
 
/* Ensure this value is replaced in the incoming packet. */
SSVAL(discard_const_p(uint8_t, req->inbuf),smb_uid,session_tag);
@@ -1518,7 +1519,7 @@ static void construct_reply(struct smbd_server_connection 
*sconn,
SMB_PERFCOUNT_SET_MSGLEN_IN(&req->pcd, size);
}
 
-   conn = switch_message(req->cmd, req, size);
+   conn = switch_message(req->cmd, req);
 
if (req->unread_bytes) {
/* writeX failed. drain socket. */
@@ -2063,7 +2064,7 @@ void chain_reply(struct smb_request *req)
req->buflen = buflen;
req->buf = buf;
 
-   switch_message(chain_cmd, req, smblen);
+   switch_message(chain_cmd, req);
 
if (req->outbuf == NULL) {
/*


-- 
Samba Shared Repository


[SCM] Samba Shared Repository - branch master updated

2012-03-05 Thread Andrew Bartlett
The branch, master has been updated
   via  14d3137 s3-lsasd: Fix debug messages on registration failure
   via  6dbf632 selftest: Reduce declarations of smbclient_auth tests by 
moving into a loop
   via  fc57623 selftest: run smbtorture_s3 tests against the ntvfs file 
server
   via  4c926a7 selftest: Reduce time in cleanup loop, try waitpid on all 3 
children
   via  ebe8aa9 selftest: start to run more dcerpc tests against 
plugin_s4_dc
   via  0582d03 selftest: forward rpcecho to Samba4 in plugin_s4_dc
  from  53a147d selftest.run: Factor out read_testlist_file and 
open_file_or_pipe.

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


- Log -
commit 14d31376aab703dbb14d1cd786baeaf84361cd96
Author: Andrew Bartlett 
Date:   Sat Mar 3 12:18:21 2012 +1100

s3-lsasd: Fix debug messages on registration failure

Autobuild-User: Andrew Bartlett 
Autobuild-Date: Mon Mar  5 09:50:17 CET 2012 on sn-devel-104

commit 6dbf632bc3e035ffd68fe67e06657593870dc2ac
Author: Andrew Bartlett 
Date:   Mon Mar 5 14:52:36 2012 +1100

selftest: Reduce declarations of smbclient_auth tests by moving into a loop

commit fc5762388a725cd18d10a356960382ea050c8064
Author: Andrew Bartlett 
Date:   Mon Mar 5 14:28:15 2012 +1100

selftest: run smbtorture_s3 tests against the ntvfs file server

This checks not only the behaviour of the NTVFS file server, but also the
client library and authentication stack.

Andrew Bartlett

commit 4c926a708d5d9a4b5e986b8aa18cdb95e8dc0142
Author: Andrew Bartlett 
Date:   Mon Mar 5 11:12:00 2012 +1100

selftest: Reduce time in cleanup loop, try waitpid on all 3 children

commit ebe8aa9cc18afa5027b1afd7b6e086613268c083
Author: Andrew Bartlett 
Date:   Mon Mar 5 10:46:23 2012 +1100

selftest: start to run more dcerpc tests against plugin_s4_dc

commit 0582d03efe8ef8992b5e1e6cad68ba885844682b
Author: Andrew Bartlett 
Date:   Mon Mar 5 10:34:30 2012 +1100

selftest: forward rpcecho to Samba4 in plugin_s4_dc

---

Summary of changes:
 selftest/knownfail |   13 +
 selftest/target/Samba3.pm  |   18 --
 selftest/target/Samba4.pm  |5 +++--
 source3/rpc_server/lsasd.c |6 +++---
 source3/selftest/tests.py  |5 ++---
 source4/selftest/tests.py  |3 ++-
 6 files changed, 35 insertions(+), 15 deletions(-)


Changeset truncated at 500 lines:

diff --git a/selftest/knownfail b/selftest/knownfail
index cc5d16b..1a6c6bd 100644
--- a/selftest/knownfail
+++ b/selftest/knownfail
@@ -7,6 +7,19 @@
 ^samba3.blackbox.failure.failure # this is designed to fail, for testing our 
test infrastructure
 .*driver.add_driver_timestamps # we only can store dates, not timestamps
 ^samba3.smbtorture_s3.LOCAL-TALLOC-DICT #fails
+^samba3.smbtorture_s3.plain\(dc\).LOCK9 # Fails against the s4 ntvfs server
+^samba3.smbtorture_s3.plain\(dc\).OPLOCK2 # Fails against the s4 ntvfs server
+^samba3.smbtorture_s3.plain\(dc\).STREAMERROR # Fails against the s4 ntvfs 
server
+^samba3.smbtorture_s3.plain\(dc\).DIR1 # Fails against the s4 ntvfs server
+^samba3.smbtorture_s3.plain\(dc\).DIR-CREATETIME # Fails against the s4 ntvfs 
server
+^samba3.smbtorture_s3.plain\(dc\).DELETE-LN # Fails against the s4 ntvfs server
+^samba3.smbtorture_s3.plain\(dc\).POSIX # Fails against the s4 ntvfs server
+^samba3.smbtorture_s3.plain\(dc\).UID-REGRESSION-TEST # Fails against the s4 
ntvfs server
+^samba3.smbtorture_s3.plain\(dc\).SHORTNAME-TEST # Fails against the s4 ntvfs 
server
+^samba3.smbtorture_s3.plain\(dc\).POSIX-APPEND # Fails against the s4 ntvfs 
server
+^samba3.smbtorture_s3.plain\(dc\).NTTRANS-FSCTL # Fails against the s4 ntvfs 
server
+^samba3.smbtorture_s3.plain\(dc\).SMB2-NEGPROT # Fails against the s4 ntvfs 
server
+^samba3.smbtorture_s3.plain\(dc\).BAD-NBT-SESSION # Fails against the s4 ntvfs 
server
 ^samba3.nbt.dgram.*netlogon2
 ^samba3.*rap.sam.*.useradd # Not provided by Samba 3
 ^samba3.*rap.sam.*.userdelete # Not provided by Samba 3
diff --git a/selftest/target/Samba3.pm b/selftest/target/Samba3.pm
index 87763aa..797ae71 100755
--- a/selftest/target/Samba3.pm
+++ b/selftest/target/Samba3.pm
@@ -54,9 +54,12 @@ sub teardown_env($$)
 
# This should give it time to write out the gcov data
until ($count > 20) {
-   if (Samba::cleanup_child($smbdpid, "smbd") == -1
-   && Samba::cleanup_child($nmbdpid, "nmbd") == -1
-   && Samba::cleanup_child($winbinddpid, "winbindd") == -1) {
+   my $smbdchild = Samba::cleanup_child($smbdpid, "smbd");
+   my $nmbdchild = Samba::cleanup_child($nmbdpid, "nmbd");
+   my $winbinddchild = Samba::cleanup_child($winbinddpid, "winbindd");
+   if ($smbdchild == -1
+   && $nmbdchild == -1
+   && $winbinddchild == -1) {
last;
}