The branch, master has been updated
       via  8c5df22... Revert "Revert "s4-smbtorture: skip extended SetValue 
test against Samba (both dont survive).""
       via  7e859be... s4:torture/rpc/dsgetinfo.c: fix name resolving for 
ldap_url
      from  287304e... Update copyright

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


- Log -----------------------------------------------------------------
commit 8c5df22ffa4bb57078f4678c60cbec6ac7ba5710
Author: Stefan Metzmacher <me...@samba.org>
Date:   Tue Mar 16 12:48:47 2010 +0100

    Revert "Revert "s4-smbtorture: skip extended SetValue test against Samba 
(both dont survive).""
    
    This reverts commit 508c218eb2f4bf8c45dc9d481576c87ed4fecbd3.
    
    This lets make test fail.
    
    Matthias: please make sure make test still passes when you change things 
like
    this...(maybe add something to knownfail or so)
    
    metze

commit 7e859be26ac1ba1154eb5b39e0467ca48a66cbf4
Author: Stefan Metzmacher <me...@samba.org>
Date:   Tue Mar 16 14:25:11 2010 +0100

    s4:torture/rpc/dsgetinfo.c: fix name resolving for ldap_url
    
    metze

-----------------------------------------------------------------------

Summary of changes:
 source4/torture/rpc/dsgetinfo.c |    7 ++++++-
 source4/torture/rpc/winreg.c    |   14 +++++---------
 2 files changed, 11 insertions(+), 10 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source4/torture/rpc/dsgetinfo.c b/source4/torture/rpc/dsgetinfo.c
index 2745804..3c3e293 100644
--- a/source4/torture/rpc/dsgetinfo.c
+++ b/source4/torture/rpc/dsgetinfo.c
@@ -67,7 +67,7 @@ struct DsGetinfoTest {
  */
 static const char *torture_get_ldap_base_dn(struct torture_context *tctx, 
struct dcerpc_pipe *p)
 {
-       const char *hostname = p->binding->target_hostname;
+       const char *hostname = p->binding->host;
        struct ldb_context *ldb;
        const char *ldap_url = talloc_asprintf(p, "ldap://%s";, hostname);
        const char *attrs[] = { "defaultNamingContext", NULL };
@@ -82,6 +82,11 @@ static const char *torture_get_ldap_base_dn(struct 
torture_context *tctx, struct
                return NULL;
        }
 
+       if (ldb_set_opaque(ldb, "loadparm", tctx->lp_ctx)) {
+               talloc_free(ldb);
+               return NULL;
+       }
+
        ret = ldb_connect(ldb, ldap_url, 0, NULL);
        if (ret != LDB_SUCCESS) {
                torture_comment(tctx, "Failed to make LDB connection to 
target");
diff --git a/source4/torture/rpc/winreg.c b/source4/torture/rpc/winreg.c
index 6057abb..651721c 100644
--- a/source4/torture/rpc/winreg.c
+++ b/source4/torture/rpc/winreg.c
@@ -2035,8 +2035,9 @@ static bool test_SetValue_extended(struct dcerpc_pipe *p,
        };
        int t, l;
 
-       if (torture_setting_bool(tctx, "samba3", false)) {
-               torture_skip(tctx, "skipping extended SetValue test against 
Samba 3");
+       if (torture_setting_bool(tctx, "samba3", false) ||
+           torture_setting_bool(tctx, "samba4", false)) {
+               torture_skip(tctx, "skipping extended SetValue test against 
Samba");
        }
 
        torture_comment(tctx, "Testing SetValue (extended formats)\n");
@@ -2143,13 +2144,8 @@ static bool test_Open(struct torture_context *tctx, 
struct dcerpc_pipe *p,
        if (created) {
                torture_assert(tctx, test_SetValue_simple(p, tctx, &newhandle),
                        "simple SetValue test failed");
-               if (!test_SetValue_extended(p, tctx, &newhandle)) {
-                       if (torture_setting_bool(tctx, "samba3", false)) {
-                               torture_warning(tctx, "extended SetValue test 
failed");
-                       } else {
-                               torture_fail(tctx, "extended SetValue test 
failed");
-                       }
-               }
+               torture_assert(tctx, test_SetValue_extended(p, tctx, 
&newhandle),
+                       "extended SetValue test failed");
        }
 
        if (created && !test_CloseKey(p, tctx, &newhandle))


-- 
Samba Shared Repository

Reply via email to