smbd/posix_acls.c: Use create_conn_struct(). Don't hand-create connection structs.
authorJeremy Allison <jra@samba.org>
Tue, 8 Jan 2013 22:17:51 +0000 (14:17 -0800)
committerAndrew Bartlett <abartlet@samba.org>
Wed, 9 Jan 2013 03:54:26 +0000 (14:54 +1100)
Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
source3/smbd/posix_acls.c

index 0f3951f83a095864bbdf8f7aa49d5e2975dd99fc..fbeb66264c68fcb878bbe590e4e57aa414be2ad7 100644 (file)
@@ -4610,27 +4610,19 @@ NTSTATUS get_nt_acl_no_snum(TALLOC_CTX *ctx, const char *fname,
                return NT_STATUS_NO_MEMORY;
        }
 
-       conn = talloc_zero(frame, connection_struct);
-       if (conn == NULL) {
-               TALLOC_FREE(frame);
-               DEBUG(0, ("talloc failed\n"));
-               return NT_STATUS_NO_MEMORY;
-       }
+       status = create_conn_struct(ctx,
+                               server_event_context(),
+                               server_messaging_context(),
+                               &conn,
+                               -1,
+                               "/",
+                               NULL);
 
-       if (!(conn->params = talloc(conn, struct share_params))) {
-               DEBUG(0, ("talloc failed\n"));
-               TALLOC_FREE(frame);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       conn->params->service = -1;
-
-       set_conn_connectpath(conn, "/");
-
-       if (!smbd_vfs_init(conn)) {
-               DEBUG(0,("smbd_vfs_init() failed!\n"));
+       if (!NT_STATUS_IS_OK(status)) {
+               DEBUG(0,("create_conn_struct returned %s.\n",
+                       nt_errstr(status)));
                TALLOC_FREE(frame);
-               return NT_STATUS_INTERNAL_ERROR;
+               return status;
        }
 
        status = SMB_VFS_GET_NT_ACL(conn, fname, security_info_wanted, ctx, sd);