s4/test-libnet: Re-align code a bit - more readable this way IMO
authorKamen Mazdrashki <kamenim@samba.org>
Fri, 11 Jun 2010 13:02:10 +0000 (16:02 +0300)
committerKamen Mazdrashki <kamenim@samba.org>
Fri, 11 Jun 2010 13:02:10 +0000 (16:02 +0300)
source4/torture/libnet/libnet_group.c
source4/torture/libnet/libnet_user.c
source4/torture/libnet/utils.c

index 6f4aa0eab6ec5a7901b5c71b116450b773f4679f..6acb3d2f60e4327d6efd3fabc3241f5aac7d5ff8 100644 (file)
@@ -83,7 +83,8 @@ bool torture_groupinfo_api(struct torture_context *torture)
                goto done;
        }
 
-       if (!test_group_cleanup(torture, ctx->samr.pipe->binding_handle, mem_ctx, &ctx->samr.handle, TEST_GROUPNAME)) {
+       if (!test_group_cleanup(torture, ctx->samr.pipe->binding_handle,
+                               mem_ctx, &ctx->samr.handle, TEST_GROUPNAME)) {
                torture_comment(torture, "cleanup failed\n");
                ret = false;
                goto done;
@@ -133,7 +134,7 @@ bool torture_grouplist(struct torture_context *torture)
 
                for (i = 0; i < req.out.count; i++) {
                        torture_comment(torture, "\tgroup: %s, sid=%s\n",
-                              req.out.groups[i].groupname, req.out.groups[i].sid);
+                                       req.out.groups[i].groupname, req.out.groups[i].sid);
                }
 
        } while (NT_STATUS_EQUAL(status, STATUS_MORE_ENTRIES));
@@ -189,7 +190,8 @@ bool torture_creategroup(struct torture_context *torture)
                goto done;
        }
 
-       if (!test_group_cleanup(torture, ctx->samr.pipe->binding_handle, mem_ctx, &ctx->samr.handle, TEST_GROUPNAME)) {
+       if (!test_group_cleanup(torture, ctx->samr.pipe->binding_handle,
+                               mem_ctx, &ctx->samr.handle, TEST_GROUPNAME)) {
                torture_comment(torture, "cleanup failed\n");
                ret = false;
                goto done;
index 6850b0e7f350c2b4958393f7766c8b5d492000a1..9dfb1b0ac4bfb9d708b02254d3cfc544268566b9 100644 (file)
@@ -369,7 +369,7 @@ bool torture_modifyuser(struct torture_context *torture)
 
 cleanup:
        if (!test_user_cleanup(torture, ctx->samr.pipe->binding_handle,
-                         torture, &ctx->samr.handle, TEST_USERNAME)) {
+                              torture, &ctx->samr.handle, TEST_USERNAME)) {
                torture_comment(torture, "cleanup failed\n");
                ret = false;
                goto done;
@@ -493,7 +493,7 @@ bool torture_userlist(struct torture_context *torture)
 
                for (i = 0; i < req.out.count; i++) {
                        torture_comment(torture, "\tuser: %s, sid=%s\n",
-                              req.out.users[i].username, req.out.users[i].sid);
+                                       req.out.users[i].username, req.out.users[i].sid);
                }
 
        } while (NT_STATUS_EQUAL(status, STATUS_MORE_ENTRIES));
index fdb4be85910478de9f5ca576390baec1eb5f814a..1364fc86aa1782afbd9be8590a6b865fee9c1d0f 100644 (file)
  * @param _dom_sid [out] If NULL, Domain SID won't be returned
  */
 bool test_domain_open(struct torture_context *tctx,
-                    struct dcerpc_binding_handle *b,
-                    struct lsa_String *domname,
-                    TALLOC_CTX *mem_ctx,
-                    struct policy_handle *_domain_handle,
-                    struct dom_sid2 *_dom_sid)
+                     struct dcerpc_binding_handle *b,
+                     struct lsa_String *domname,
+                     TALLOC_CTX *mem_ctx,
+                     struct policy_handle *_domain_handle,
+                     struct dom_sid2 *_dom_sid)
 {
        struct policy_handle connect_handle;
        struct policy_handle domain_handle;
@@ -58,10 +58,10 @@ bool test_domain_open(struct torture_context *tctx,
        r1.out.connect_handle = &connect_handle;
 
        torture_assert_ntstatus_ok(tctx,
-               dcerpc_samr_Connect_r(b, mem_ctx, &r1),
-               "Connect failed");
+                                  dcerpc_samr_Connect_r(b, mem_ctx, &r1),
+                                  "Connect failed");
        torture_assert_ntstatus_ok(tctx, r1.out.result,
-               "Connect failed");
+                                  "Connect failed");
 
        r2.in.connect_handle = &connect_handle;
        r2.in.domain_name = domname;
@@ -165,7 +165,8 @@ done:
  */
 bool test_user_cleanup(struct torture_context *tctx,
                       struct dcerpc_binding_handle *b,
-                      TALLOC_CTX *mem_ctx, struct policy_handle *domain_handle,
+                      TALLOC_CTX *mem_ctx,
+                      struct policy_handle *domain_handle,
                       const char *user_rdn)
 {
        struct samr_LookupNames r1;
@@ -300,7 +301,8 @@ bool test_user_create(struct torture_context *tctx,
  * Deletes a Group using SAMR interface
  */
 bool test_group_cleanup(struct torture_context *tctx,
-                       struct dcerpc_binding_handle *b, TALLOC_CTX *mem_ctx,
+                       struct dcerpc_binding_handle *b,
+                       TALLOC_CTX *mem_ctx,
                        struct policy_handle *domain_handle,
                        const char *name)
 {
@@ -323,10 +325,10 @@ bool test_group_cleanup(struct torture_context *tctx,
        torture_comment(tctx, "group account lookup '%s'\n", name);
 
        torture_assert_ntstatus_ok(tctx,
-               dcerpc_samr_LookupNames_r(b, mem_ctx, &r1),
-               "LookupNames failed");
+                                  dcerpc_samr_LookupNames_r(b, mem_ctx, &r1),
+                                  "LookupNames failed");
        torture_assert_ntstatus_ok(tctx, r1.out.result,
-               "LookupNames failed");
+                                  "LookupNames failed");
 
        rid = r1.out.rids->ids[0];
 
@@ -338,10 +340,10 @@ bool test_group_cleanup(struct torture_context *tctx,
        torture_comment(tctx, "opening group account\n");
 
        torture_assert_ntstatus_ok(tctx,
-               dcerpc_samr_OpenGroup_r(b, mem_ctx, &r2),
-               "OpenGroup failed");
+                                  dcerpc_samr_OpenGroup_r(b, mem_ctx, &r2),
+                                  "OpenGroup failed");
        torture_assert_ntstatus_ok(tctx, r2.out.result,
-               "OpenGroup failed");
+                                  "OpenGroup failed");
 
        r3.in.group_handle  = &group_handle;
        r3.out.group_handle = &group_handle;
@@ -349,10 +351,10 @@ bool test_group_cleanup(struct torture_context *tctx,
        torture_comment(tctx, "deleting group account\n");
 
        torture_assert_ntstatus_ok(tctx,
-               dcerpc_samr_DeleteDomainGroup_r(b, mem_ctx, &r3),
-               "DeleteGroup failed");
+                                  dcerpc_samr_DeleteDomainGroup_r(b, mem_ctx, &r3),
+                                  "DeleteGroup failed");
        torture_assert_ntstatus_ok(tctx, r3.out.result,
-               "DeleteGroup failed");
+                                  "DeleteGroup failed");
 
        return true;
 }