Fix a bunch of compiler warnings about wrong format types.
[ira/wip.git] / source3 / utils / net_rpc.c
index 5f6c9428b7accb227adfaf2865363d38ae0c05a5..4de4bef837a16847319b48e5cdb50cdbd92c5820 100644 (file)
@@ -22,6 +22,7 @@
 
 #include "includes.h"
 #include "utils/net.h"
+#include "../libcli/auth/libcli_auth.h"
 
 static int net_mode_share;
 static bool sync_files(struct copy_clistate *cp_clistate, const char *mask);
@@ -54,19 +55,20 @@ NTSTATUS net_get_remote_domain_sid(struct cli_state *cli, TALLOC_CTX *mem_ctx,
                                   DOM_SID **domain_sid,
                                   const char **domain_name)
 {
-       struct rpc_pipe_client *lsa_pipe;
-       POLICY_HND pol;
+       struct rpc_pipe_client *lsa_pipe = NULL;
+       struct policy_handle pol;
        NTSTATUS result = NT_STATUS_OK;
        union lsa_PolicyInformation *info = NULL;
 
-       lsa_pipe = cli_rpc_pipe_open_noauth(cli, PI_LSARPC, &result);
-       if (!lsa_pipe) {
+       result = cli_rpc_pipe_open_noauth(cli, &ndr_table_lsarpc.syntax_id,
+                                         &lsa_pipe);
+       if (!NT_STATUS_IS_OK(result)) {
                d_fprintf(stderr, "Could not initialise lsa pipe\n");
                return result;
        }
 
        result = rpccli_lsa_open_policy(lsa_pipe, mem_ctx, false,
-                                    SEC_RIGHTS_MAXIMUM_ALLOWED,
+                                    SEC_FLAG_MAXIMUM_ALLOWED,
                                     &pol);
        if (!NT_STATUS_IS_OK(result)) {
                d_fprintf(stderr, "open_policy failed: %s\n",
@@ -99,15 +101,15 @@ NTSTATUS net_get_remote_domain_sid(struct cli_state *cli, TALLOC_CTX *mem_ctx,
  * @param pipe_name the pipe to connect to (usually a PIPE_ constant)
  * @param conn_flag a NET_FLAG_ combination.  Passed to
  *                   net_make_ipc_connection.
- * @param argc  Standard main() style argc
- * @param argc  Standard main() style argv.  Initial components are already
- *              stripped
- * @return A shell status integer (0 for success)
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
+ * @return A shell status integer (0 for success).
  */
 
 int run_rpc_command(struct net_context *c,
                        struct cli_state *cli_arg,
-                       const int pipe_idx,
+                       const struct ndr_syntax_id *interface,
                        int conn_flags,
                        rpc_command_fn fn,
                        int argc,
@@ -119,6 +121,7 @@ int run_rpc_command(struct net_context *c,
        NTSTATUS nt_status;
        DOM_SID *domain_sid;
        const char *domain_name;
+       int ret = -1;
 
        /* make use of cli_state handed over as an argument, if possible */
        if (!cli_arg) {
@@ -140,38 +143,46 @@ int run_rpc_command(struct net_context *c,
 
        if (!(mem_ctx = talloc_init("run_rpc_command"))) {
                DEBUG(0, ("talloc_init() failed\n"));
-               cli_shutdown(cli);
-               return -1;
+               goto fail;
        }
 
        nt_status = net_get_remote_domain_sid(cli, mem_ctx, &domain_sid,
                                              &domain_name);
        if (!NT_STATUS_IS_OK(nt_status)) {
-               cli_shutdown(cli);
-               return -1;
+               goto fail;
        }
 
        if (!(conn_flags & NET_FLAGS_NO_PIPE)) {
-               if (lp_client_schannel() && (pipe_idx == PI_NETLOGON)) {
+               if (lp_client_schannel()
+                   && (ndr_syntax_id_equal(interface,
+                                           &ndr_table_netlogon.syntax_id))) {
                        /* Always try and create an schannel netlogon pipe. */
-                       pipe_hnd = cli_rpc_pipe_open_schannel(cli, pipe_idx,
-                                                       PIPE_AUTH_LEVEL_PRIVACY,
-                                                       domain_name,
-                                                       &nt_status);
-                       if (!pipe_hnd) {
+                       nt_status = cli_rpc_pipe_open_schannel(
+                               cli, interface,
+                               PIPE_AUTH_LEVEL_PRIVACY, domain_name,
+                               &pipe_hnd);
+                       if (!NT_STATUS_IS_OK(nt_status)) {
                                DEBUG(0, ("Could not initialise schannel netlogon pipe. Error was %s\n",
                                        nt_errstr(nt_status) ));
-                               cli_shutdown(cli);
-                               return -1;
+                               goto fail;
                        }
                } else {
-                       pipe_hnd = cli_rpc_pipe_open_noauth(cli, pipe_idx, &nt_status);
-                       if (!pipe_hnd) {
+                       if (conn_flags & NET_FLAGS_SEAL) {
+                               nt_status = cli_rpc_pipe_open_ntlmssp(
+                                       cli, interface,
+                                       PIPE_AUTH_LEVEL_PRIVACY,
+                                       lp_workgroup(), c->opt_user_name,
+                                       c->opt_password, &pipe_hnd);
+                       } else {
+                               nt_status = cli_rpc_pipe_open_noauth(
+                                       cli, interface,
+                                       &pipe_hnd);
+                       }
+                       if (!NT_STATUS_IS_OK(nt_status)) {
                                DEBUG(0, ("Could not initialise pipe %s. Error was %s\n",
-                                       cli_get_pipe_name(pipe_idx),
+                                       get_pipe_name_from_iface(interface),
                                        nt_errstr(nt_status) ));
-                               cli_shutdown(cli);
-                               return -1;
+                               goto fail;
                        }
                }
        }
@@ -181,6 +192,7 @@ int run_rpc_command(struct net_context *c,
        if (!NT_STATUS_IS_OK(nt_status)) {
                DEBUG(1, ("rpc command function failed! (%s)\n", nt_errstr(nt_status)));
        } else {
+               ret = 0;
                DEBUG(5, ("rpc command function succedded\n"));
        }
 
@@ -190,27 +202,28 @@ int run_rpc_command(struct net_context *c,
                }
        }
 
+fail:
        /* close the connection only if it was opened here */
        if (!cli_arg) {
                cli_shutdown(cli);
        }
 
        talloc_destroy(mem_ctx);
-       return (!NT_STATUS_IS_OK(nt_status));
+       return ret;
 }
 
 /**
  * Force a change of the trust acccount password.
  *
  * All parameters are provided by the run_rpc_command function, except for
- * argc, argv which are passes through.
+ * argc, argv which are passed through.
  *
- * @param domain_sid The domain sid aquired from the remote server
+ * @param domain_sid The domain sid acquired from the remote server.
  * @param cli A cli_state connected to the server.
- * @param mem_ctx Talloc context, destoyed on compleation of the function.
- * @param argc  Standard main() style argc
- * @param argc  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param mem_ctx Talloc context, destroyed on completion of the function.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
  * @return Normal NTSTATUS return.
  **/
@@ -231,11 +244,11 @@ static NTSTATUS rpc_changetrustpw_internals(struct net_context *c,
 /**
  * Force a change of the trust acccount password.
  *
- * @param argc  Standard main() style argc
- * @param argc  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  **/
 
 int net_rpc_changetrustpw(struct net_context *c, int argc, const char **argv)
@@ -247,7 +260,8 @@ int net_rpc_changetrustpw(struct net_context *c, int argc, const char **argv)
                return 0;
        }
 
-       return run_rpc_command(c, NULL, PI_NETLOGON, NET_FLAGS_ANONYMOUS | NET_FLAGS_PDC,
+       return run_rpc_command(c, NULL, &ndr_table_netlogon.syntax_id,
+                              NET_FLAGS_ANONYMOUS | NET_FLAGS_PDC,
                               rpc_changetrustpw_internals,
                               argc, argv);
 }
@@ -260,14 +274,14 @@ int net_rpc_changetrustpw(struct net_context *c, int argc, const char **argv)
  * The password should be created with 'server manager' or equiv first.
  *
  * All parameters are provided by the run_rpc_command function, except for
- * argc, argv which are passes through.
+ * argc, argv which are passed through.
  *
- * @param domain_sid The domain sid aquired from the remote server
+ * @param domain_sid The domain sid acquired from the remote server.
  * @param cli A cli_state connected to the server.
- * @param mem_ctx Talloc context, destoyed on compleation of the function.
- * @param argc  Standard main() style argc
- * @param argc  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param mem_ctx Talloc context, destroyed on completion of the function.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
  * @return Normal NTSTATUS return.
  **/
@@ -287,8 +301,9 @@ static NTSTATUS rpc_oldjoin_internals(struct net_context *c,
        NTSTATUS result;
        uint32 sec_channel_type;
 
-       pipe_hnd = cli_rpc_pipe_open_noauth(cli, PI_NETLOGON, &result);
-       if (!pipe_hnd) {
+       result = cli_rpc_pipe_open_noauth(cli, &ndr_table_netlogon.syntax_id,
+                                         &pipe_hnd);
+       if (!NT_STATUS_IS_OK(result)) {
                DEBUG(0,("rpc_oldjoin_internals: netlogon pipe open to machine %s failed. "
                        "error was %s\n",
                        cli->desthost,
@@ -337,16 +352,16 @@ static NTSTATUS rpc_oldjoin_internals(struct net_context *c,
 /**
  * Join a domain, the old way.
  *
- * @param argc  Standard main() style argc
- * @param argc  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  **/
 
 static int net_rpc_perform_oldjoin(struct net_context *c, int argc, const char **argv)
 {
-       return run_rpc_command(c, NULL, PI_NETLOGON,
+       return run_rpc_command(c, NULL, &ndr_table_netlogon.syntax_id,
                               NET_FLAGS_NO_PIPE | NET_FLAGS_ANONYMOUS | NET_FLAGS_PDC,
                               rpc_oldjoin_internals,
                               argc, argv);
@@ -355,13 +370,13 @@ static int net_rpc_perform_oldjoin(struct net_context *c, int argc, const char *
 /**
  * Join a domain, the old way.  This function exists to allow
  * the message to be displayed when oldjoin was explicitly
- * requested, but not when it was implied by "net rpc join"
+ * requested, but not when it was implied by "net rpc join".
  *
- * @param argc  Standard main() style argc
- * @param argc  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  **/
 
 static int net_rpc_oldjoin(struct net_context *c, int argc, const char **argv)
@@ -386,12 +401,12 @@ static int net_rpc_oldjoin(struct net_context *c, int argc, const char **argv)
 
 /**
  * 'net rpc join' entrypoint.
- * @param argc  Standard main() style argc
- * @param argc  Standard main() style argv.  Initial components are already
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
  *              stripped
  *
  * Main 'net_rpc_join()' (where the admin username/password is used) is
- * in net_rpc_join.c
+ * in net_rpc_join.c.
  * Try to just change the password, but if that doesn't work, use/prompt
  * for a username/password.
  **/
@@ -438,10 +453,10 @@ int net_rpc_join(struct net_context *c, int argc, const char **argv)
  *
  * @param domain_sid The domain sid acquired from the remote server
  * @param cli A cli_state connected to the server.
- * @param mem_ctx Talloc context, destoyed on completion of the function.
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param mem_ctx Talloc context, destroyed on completion of the function.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
  * @return Normal NTSTATUS return.
  **/
@@ -455,7 +470,7 @@ NTSTATUS rpc_info_internals(struct net_context *c,
                        int argc,
                        const char **argv)
 {
-       POLICY_HND connect_pol, domain_pol;
+       struct policy_handle connect_pol, domain_pol;
        NTSTATUS result = NT_STATUS_UNSUCCESSFUL;
        union samr_DomainInfo *info = NULL;
        fstring sid_str;
@@ -488,13 +503,13 @@ NTSTATUS rpc_info_internals(struct net_context *c,
                                             2,
                                             &info);
        if (NT_STATUS_IS_OK(result)) {
-               d_printf("Domain Name: %s\n", info->info2.domain_name.string);
+               d_printf("Domain Name: %s\n", info->general.domain_name.string);
                d_printf("Domain SID: %s\n", sid_str);
                d_printf("Sequence number: %llu\n",
-                       (unsigned long long)info->info2.sequence_num);
-               d_printf("Num users: %u\n", info->info2.num_users);
-               d_printf("Num domain groups: %u\n", info->info2.num_groups);
-               d_printf("Num local groups: %u\n", info->info2.num_aliases);
+                       (unsigned long long)info->general.sequence_num);
+               d_printf("Num users: %u\n", info->general.num_users);
+               d_printf("Num domain groups: %u\n", info->general.num_groups);
+               d_printf("Num local groups: %u\n", info->general.num_aliases);
        }
 
  done:
@@ -503,9 +518,9 @@ NTSTATUS rpc_info_internals(struct net_context *c,
 
 /**
  * 'net rpc info' entrypoint.
- * @param argc  Standard main() style argc
- * @param argc  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  **/
 
 int net_rpc_info(struct net_context *c, int argc, const char **argv)
@@ -517,23 +532,23 @@ int net_rpc_info(struct net_context *c, int argc, const char **argv)
                return 0;
        }
 
-       return run_rpc_command(c, NULL, PI_SAMR, NET_FLAGS_PDC,
-                              rpc_info_internals,
+       return run_rpc_command(c, NULL, &ndr_table_samr.syntax_id,
+                              NET_FLAGS_PDC, rpc_info_internals,
                               argc, argv);
 }
 
 /**
- * Fetch domain SID into the local secrets.tdb
+ * Fetch domain SID into the local secrets.tdb.
  *
  * All parameters are provided by the run_rpc_command function, except for
- * argc, argv which are passes through.
+ * argc, argv which are passed through.
  *
- * @param domain_sid The domain sid acquired from the remote server
+ * @param domain_sid The domain sid acquired from the remote server.
  * @param cli A cli_state connected to the server.
- * @param mem_ctx Talloc context, destoyed on completion of thea function.
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param mem_ctx Talloc context, destroyed on completion of the function.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
  * @return Normal NTSTATUS return.
  **/
@@ -563,9 +578,9 @@ static NTSTATUS rpc_getsid_internals(struct net_context *c,
 
 /**
  * 'net rpc getsid' entrypoint.
- * @param argc  Standard main() style argc
- * @param argc  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  **/
 
 int net_rpc_getsid(struct net_context *c, int argc, const char **argv)
@@ -577,7 +592,7 @@ int net_rpc_getsid(struct net_context *c, int argc, const char **argv)
                return 0;
        }
 
-       return run_rpc_command(c, NULL, PI_SAMR,
+       return run_rpc_command(c, NULL, &ndr_table_samr.syntax_id,
                               NET_FLAGS_ANONYMOUS | NET_FLAGS_PDC,
                               rpc_getsid_internals,
                               argc, argv);
@@ -586,9 +601,9 @@ int net_rpc_getsid(struct net_context *c, int argc, const char **argv)
 /****************************************************************************/
 
 /**
- * Basic usage function for 'net rpc user'
+ * Basic usage function for 'net rpc user'.
  * @param argc Standard main() style argc.
- * @param argv Standard main() style argv.  Initial components are already
+ * @param argv Standard main() style argv. Initial components are already
  *             stripped.
  **/
 
@@ -598,13 +613,13 @@ static int rpc_user_usage(struct net_context *c, int argc, const char **argv)
 }
 
 /**
- * Add a new user to a remote RPC server
+ * Add a new user to a remote RPC server.
  *
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  **/
 
 static int rpc_user_add(struct net_context *c, int argc, const char **argv)
@@ -640,147 +655,49 @@ static int rpc_user_add(struct net_context *c, int argc, const char **argv)
 }
 
 /**
- * Rename a user on a remote RPC server
+ * Rename a user on a remote RPC server.
  *
- * All parameters are provided by the run_rpc_command function, except for
- * argc, argv which are passes through.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @param domain_sid The domain sid acquired from the remote server
- * @param cli A cli_state connected to the server.
- * @param mem_ctx Talloc context, destoyed on completion of the function.
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
- *
- * @return Normal NTSTATUS return.
+ * @return A shell status integer (0 for success).
  **/
 
-static NTSTATUS rpc_user_rename_internals(struct net_context *c,
-                                       const DOM_SID *domain_sid,
-                                       const char *domain_name,
-                                       struct cli_state *cli,
-                                       struct rpc_pipe_client *pipe_hnd,
-                                       TALLOC_CTX *mem_ctx,
-                                       int argc,
-                                       const char **argv)
+static int rpc_user_rename(struct net_context *c, int argc, const char **argv)
 {
-       POLICY_HND connect_pol, domain_pol, user_pol;
-       NTSTATUS result = NT_STATUS_UNSUCCESSFUL;
-       uint32 info_level = 7;
-       const char *old_name, *new_name;
-       struct samr_Ids user_rids, name_types;
-       struct lsa_String lsa_acct_name;
-       union samr_UserInfo *info = NULL;
+       NET_API_STATUS status;
+       struct USER_INFO_0 u0;
+       uint32_t parm_err = 0;
 
        if (argc != 2 || c->display_usage) {
                rpc_user_usage(c, argc, argv);
-               return NT_STATUS_OK;
-       }
-
-       old_name = argv[0];
-       new_name = argv[1];
-
-       /* Get sam policy handle */
-
-       result = rpccli_samr_Connect2(pipe_hnd, mem_ctx,
-                                     pipe_hnd->desthost,
-                                     MAXIMUM_ALLOWED_ACCESS,
-                                     &connect_pol);
-
-       if (!NT_STATUS_IS_OK(result)) {
-               goto done;
-       }
-
-       /* Get domain policy handle */
-
-       result = rpccli_samr_OpenDomain(pipe_hnd, mem_ctx,
-                                       &connect_pol,
-                                       MAXIMUM_ALLOWED_ACCESS,
-                                       CONST_DISCARD(struct dom_sid2 *, domain_sid),
-                                       &domain_pol);
-       if (!NT_STATUS_IS_OK(result)) {
-               goto done;
-       }
-
-       init_lsa_String(&lsa_acct_name, old_name);
-
-       result = rpccli_samr_LookupNames(pipe_hnd, mem_ctx,
-                                        &domain_pol,
-                                        1,
-                                        &lsa_acct_name,
-                                        &user_rids,
-                                        &name_types);
-       if (!NT_STATUS_IS_OK(result)) {
-               goto done;
-       }
-
-       /* Open domain user */
-       result = rpccli_samr_OpenUser(pipe_hnd, mem_ctx,
-                                     &domain_pol,
-                                     MAXIMUM_ALLOWED_ACCESS,
-                                     user_rids.ids[0],
-                                     &user_pol);
-
-       if (!NT_STATUS_IS_OK(result)) {
-               goto done;
-       }
-
-       /* Query user info */
-       result = rpccli_samr_QueryUserInfo(pipe_hnd, mem_ctx,
-                                          &user_pol,
-                                          info_level,
-                                          &info);
-
-       if (!NT_STATUS_IS_OK(result)) {
-               goto done;
+               return 0;
        }
 
-       init_samr_user_info7(&info->info7, new_name);
-
-       /* Set new name */
-       result = rpccli_samr_SetUserInfo2(pipe_hnd, mem_ctx,
-                                         &user_pol,
-                                         info_level,
-                                         info);
+       u0.usri0_name = argv[1];
 
-       if (!NT_STATUS_IS_OK(result)) {
-               goto done;
-       }
-
- done:
-       if (!NT_STATUS_IS_OK(result)) {
-               d_fprintf(stderr, "Failed to rename user from %s to %s - %s\n", old_name, new_name, 
-                        nt_errstr(result));
+       status = NetUserSetInfo(c->opt_host, argv[0],
+                               0, (uint8_t *)&u0, &parm_err);
+       if (status) {
+               d_fprintf(stderr, "Failed to rename user from %s to %s - %s\n",
+                         argv[0], argv[1],
+                         libnetapi_get_error_string(c->netapi_ctx, status));
        } else {
-               d_printf("Renamed user from %s to %s\n", old_name, new_name);
+               d_printf("Renamed user from %s to %s\n", argv[0], argv[1]);
        }
-       return result;
-}
 
-/**
- * Rename a user on a remote RPC server
- *
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
- *
- * @return A shell status integer (0 for success)
- **/
-
-static int rpc_user_rename(struct net_context *c, int argc, const char **argv)
-{
-       return run_rpc_command(c, NULL, PI_SAMR, 0, rpc_user_rename_internals,
-                              argc, argv);
+       return status;
 }
 
 /**
- * Delete a user from a remote RPC server
+ * Delete a user from a remote RPC server.
  *
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  **/
 
 static int rpc_user_delete(struct net_context *c, int argc, const char **argv)
@@ -807,378 +724,175 @@ static int rpc_user_delete(struct net_context *c, int argc, const char **argv)
 }
 
 /**
- * Set a password for a user on a remote RPC server
- *
- * All parameters are provided by the run_rpc_command function, except for
- * argc, argv which are passes through.
+ * Set a user's password on a remote RPC server.
  *
- * @param domain_sid The domain sid acquired from the remote server
- * @param cli A cli_state connected to the server.
- * @param mem_ctx Talloc context, destoyed on completion of the function.
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return Normal NTSTATUS return.
+ * @return A shell status integer (0 for success).
  **/
 
-static NTSTATUS rpc_user_password_internals(struct net_context *c,
-                                       const DOM_SID *domain_sid,
-                                       const char *domain_name,
-                                       struct cli_state *cli,
-                                       struct rpc_pipe_client *pipe_hnd,
-                                       TALLOC_CTX *mem_ctx,
-                                       int argc,
-                                       const char **argv)
+static int rpc_user_password(struct net_context *c, int argc, const char **argv)
 {
-       NTSTATUS result = NT_STATUS_UNSUCCESSFUL;
-       POLICY_HND connect_pol, domain_pol, user_pol;
-       uchar pwbuf[516];
-       const char *user;
-       const char *new_password;
+       NET_API_STATUS status;
        char *prompt = NULL;
-       union samr_UserInfo info;
+       struct USER_INFO_1003 u1003;
+       uint32_t parm_err = 0;
 
        if (argc < 1 || c->display_usage) {
                rpc_user_usage(c, argc, argv);
-               return NT_STATUS_OK;
+               return 0;
        }
 
-       user = argv[0];
-
        if (argv[1]) {
-               new_password = argv[1];
+               u1003.usri1003_password = argv[1];
        } else {
-               asprintf(&prompt, "Enter new password for %s:", user);
-               new_password = getpass(prompt);
-               SAFE_FREE(prompt);
-       }
-
-       /* Get sam policy and domain handles */
-
-       result = rpccli_samr_Connect2(pipe_hnd, mem_ctx,
-                                     pipe_hnd->desthost,
-                                     MAXIMUM_ALLOWED_ACCESS,
-                                     &connect_pol);
-
-       if (!NT_STATUS_IS_OK(result)) {
-               goto done;
-       }
-
-       result = rpccli_samr_OpenDomain(pipe_hnd, mem_ctx,
-                                       &connect_pol,
-                                       MAXIMUM_ALLOWED_ACCESS,
-                                       CONST_DISCARD(struct dom_sid2 *, domain_sid),
-                                       &domain_pol);
-
-       if (!NT_STATUS_IS_OK(result)) {
-               goto done;
-       }
-
-       /* Get handle on user */
-
-       {
-               struct samr_Ids user_rids, name_types;
-               struct lsa_String lsa_acct_name;
-
-               init_lsa_String(&lsa_acct_name, user);
-
-               result = rpccli_samr_LookupNames(pipe_hnd, mem_ctx,
-                                                &domain_pol,
-                                                1,
-                                                &lsa_acct_name,
-                                                &user_rids,
-                                                &name_types);
-               if (!NT_STATUS_IS_OK(result)) {
-                       goto done;
-               }
-
-               result = rpccli_samr_OpenUser(pipe_hnd, mem_ctx,
-                                             &domain_pol,
-                                             MAXIMUM_ALLOWED_ACCESS,
-                                             user_rids.ids[0],
-                                             &user_pol);
-
-               if (!NT_STATUS_IS_OK(result)) {
-                       goto done;
+               if (asprintf(&prompt, "Enter new password for %s:", argv[0]) == -1) {
+                       return -1;
                }
+               u1003.usri1003_password = getpass(prompt);
+               SAFE_FREE(prompt);
        }
 
-       /* Set password on account */
-
-       encode_pw_buffer(pwbuf, new_password, STR_UNICODE);
-
-       init_samr_user_info24(&info.info24, pwbuf, 24);
-
-       SamOEMhashBlob(info.info24.password.data, 516,
-                      &cli->user_session_key);
-
-       result = rpccli_samr_SetUserInfo2(pipe_hnd, mem_ctx,
-                                         &user_pol,
-                                         24,
-                                         &info);
-
-       if (!NT_STATUS_IS_OK(result)) {
-               goto done;
-       }
+       status = NetUserSetInfo(c->opt_host, argv[0], 1003, (uint8_t *)&u1003, &parm_err);
 
        /* Display results */
+       if (status != 0) {
+               d_fprintf(stderr, "Failed to set password for '%s' with: %s.\n",
+                       argv[0], libnetapi_get_error_string(c->netapi_ctx,
+                                                           status));
+               return -1;
+       }
 
- done:
-       return result;
-
+       return 0;
 }
 
 /**
- * Set a user's password on a remote RPC server
+ * List a user's groups from a remote RPC server.
  *
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
  * @return A shell status integer (0 for success)
  **/
 
-static int rpc_user_password(struct net_context *c, int argc, const char **argv)
-{
-       return run_rpc_command(c, NULL, PI_SAMR, 0, rpc_user_password_internals,
-                              argc, argv);
-}
-
-/**
- * List user's groups on a remote RPC server
- *
- * All parameters are provided by the run_rpc_command function, except for
- * argc, argv which are passes through.
- *
- * @param domain_sid The domain sid acquired from the remote server
- * @param cli A cli_state connected to the server.
- * @param mem_ctx Talloc context, destoyed on completion of the function.
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
- *
- * @return Normal NTSTATUS return.
- **/
+static int rpc_user_info(struct net_context *c, int argc, const char **argv)
 
-static NTSTATUS rpc_user_info_internals(struct net_context *c,
-                       const DOM_SID *domain_sid,
-                       const char *domain_name,
-                       struct cli_state *cli,
-                       struct rpc_pipe_client *pipe_hnd,
-                       TALLOC_CTX *mem_ctx,
-                       int argc,
-                       const char **argv)
 {
-       POLICY_HND connect_pol, domain_pol, user_pol;
-       NTSTATUS result = NT_STATUS_UNSUCCESSFUL;
+       NET_API_STATUS status;
+       struct GROUP_USERS_INFO_0 *u0 = NULL;
+       uint32_t entries_read = 0;
+       uint32_t total_entries = 0;
        int i;
-       struct samr_RidWithAttributeArray *rid_array = NULL;
-       struct lsa_Strings names;
-       struct samr_Ids types;
-       uint32_t *lrids = NULL;
-       struct samr_Ids rids, name_types;
-       struct lsa_String lsa_acct_name;
 
 
        if (argc < 1 || c->display_usage) {
                rpc_user_usage(c, argc, argv);
-               return NT_STATUS_OK;
+               return 0;
        }
-       /* Get sam policy handle */
 
-       result = rpccli_samr_Connect2(pipe_hnd, mem_ctx,
-                                     pipe_hnd->desthost,
-                                     MAXIMUM_ALLOWED_ACCESS,
-                                     &connect_pol);
-       if (!NT_STATUS_IS_OK(result)) goto done;
-
-       /* Get domain policy handle */
-
-       result = rpccli_samr_OpenDomain(pipe_hnd, mem_ctx,
-                                       &connect_pol,
-                                       MAXIMUM_ALLOWED_ACCESS,
-                                       CONST_DISCARD(struct dom_sid2 *, domain_sid),
-                                       &domain_pol);
-       if (!NT_STATUS_IS_OK(result)) goto done;
-
-       /* Get handle on user */
-
-       init_lsa_String(&lsa_acct_name, argv[0]);
-
-       result = rpccli_samr_LookupNames(pipe_hnd, mem_ctx,
-                                        &domain_pol,
-                                        1,
-                                        &lsa_acct_name,
-                                        &rids,
-                                        &name_types);
-
-       if (!NT_STATUS_IS_OK(result)) goto done;
-
-       result = rpccli_samr_OpenUser(pipe_hnd, mem_ctx,
-                                     &domain_pol,
-                                     MAXIMUM_ALLOWED_ACCESS,
-                                     rids.ids[0],
-                                     &user_pol);
-       if (!NT_STATUS_IS_OK(result)) goto done;
-
-       result = rpccli_samr_GetGroupsForUser(pipe_hnd, mem_ctx,
-                                             &user_pol,
-                                             &rid_array);
-
-       if (!NT_STATUS_IS_OK(result)) goto done;
-
-       /* Look up rids */
-
-       if (rid_array->count) {
-               if ((lrids = TALLOC_ARRAY(mem_ctx, uint32, rid_array->count)) == NULL) {
-                       result = NT_STATUS_NO_MEMORY;
-                       goto done;
-               }
-
-               for (i = 0; i < rid_array->count; i++)
-                       lrids[i] = rid_array->rids[i].rid;
-
-               result = rpccli_samr_LookupRids(pipe_hnd, mem_ctx,
-                                               &domain_pol,
-                                               rid_array->count,
-                                               lrids,
-                                               &names,
-                                               &types);
-
-               if (!NT_STATUS_IS_OK(result)) {
-                       goto done;
-               }
-
-               /* Display results */
-
-               for (i = 0; i < names.count; i++)
-                       printf("%s\n", names.names[i].string);
+       status = NetUserGetGroups(c->opt_host,
+                                 argv[0],
+                                 0,
+                                 (uint8_t **)(void *)&u0,
+                                 (uint32_t)-1,
+                                 &entries_read,
+                                 &total_entries);
+       if (status != 0) {
+               d_fprintf(stderr, "Failed to get groups for '%s' with: %s.\n",
+                       argv[0], libnetapi_get_error_string(c->netapi_ctx,
+                                                           status));
+               return -1;
        }
- done:
-       return result;
-}
 
-/**
- * List a user's groups from a remote RPC server
- *
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
- *
- * @return A shell status integer (0 for success)
- **/
+       for (i=0; i < entries_read; i++) {
+               printf("%s\n", u0->grui0_name);
+               u0++;
+       }
 
-static int rpc_user_info(struct net_context *c, int argc, const char **argv)
-{
-       return run_rpc_command(c, NULL, PI_SAMR, 0, rpc_user_info_internals,
-                              argc, argv);
+       return 0;
 }
 
 /**
- * List users on a remote RPC server
+ * List users on a remote RPC server.
  *
  * All parameters are provided by the run_rpc_command function, except for
- * argc, argv which are passes through.
+ * argc, argv which are passed through.
  *
- * @param domain_sid The domain sid acquired from the remote server
+ * @param domain_sid The domain sid acquired from the remote server.
  * @param cli A cli_state connected to the server.
- * @param mem_ctx Talloc context, destoyed on completion of the function.
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param mem_ctx Talloc context, destroyed on completion of the function.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
  * @return Normal NTSTATUS return.
  **/
 
-static NTSTATUS rpc_user_list_internals(struct net_context *c,
-                                       const DOM_SID *domain_sid,
-                                       const char *domain_name,
-                                       struct cli_state *cli,
-                                       struct rpc_pipe_client *pipe_hnd,
-                                       TALLOC_CTX *mem_ctx,
-                                       int argc,
-                                       const char **argv)
+static int rpc_user_list(struct net_context *c, int argc, const char **argv)
 {
-       POLICY_HND connect_pol, domain_pol;
-       NTSTATUS result = NT_STATUS_UNSUCCESSFUL;
-       uint32 start_idx=0, num_entries, i, loop_count = 0;
-
-       /* Get sam policy handle */
-
-       result = rpccli_samr_Connect2(pipe_hnd, mem_ctx,
-                                     pipe_hnd->desthost,
-                                     MAXIMUM_ALLOWED_ACCESS,
-                                     &connect_pol);
-       if (!NT_STATUS_IS_OK(result)) {
-               goto done;
-       }
-
-       /* Get domain policy handle */
-
-       result = rpccli_samr_OpenDomain(pipe_hnd, mem_ctx,
-                                       &connect_pol,
-                                       MAXIMUM_ALLOWED_ACCESS,
-                                       CONST_DISCARD(struct dom_sid2 *, domain_sid),
-                                       &domain_pol);
-       if (!NT_STATUS_IS_OK(result)) {
-               goto done;
-       }
+       NET_API_STATUS status;
+       uint32_t start_idx=0, num_entries, i, loop_count = 0;
+       struct NET_DISPLAY_USER *info = NULL;
+       void *buffer = NULL;
 
        /* Query domain users */
        if (c->opt_long_list_entries)
                d_printf("\nUser name             Comment"
                         "\n-----------------------------\n");
        do {
-               const char *user = NULL;
-               const char *desc = NULL;
-               uint32 max_entries, max_size;
-               uint32_t total_size, returned_size;
-               union samr_DispInfo info;
+               uint32_t max_entries, max_size;
 
                get_query_dispinfo_params(
                        loop_count, &max_entries, &max_size);
 
-               result = rpccli_samr_QueryDisplayInfo(pipe_hnd, mem_ctx,
-                                                     &domain_pol,
-                                                     1,
-                                                     start_idx,
-                                                     max_entries,
-                                                     max_size,
-                                                     &total_size,
-                                                     &returned_size,
-                                                     &info);
-               loop_count++;
-               start_idx += info.info1.count;
-               num_entries = info.info1.count;
+               status = NetQueryDisplayInformation(c->opt_host,
+                                                   1,
+                                                   start_idx,
+                                                   max_entries,
+                                                   max_size,
+                                                   &num_entries,
+                                                   &buffer);
+               if (status != 0 && status != ERROR_MORE_DATA) {
+                       return status;
+               }
+
+               info = (struct NET_DISPLAY_USER *)buffer;
 
                for (i = 0; i < num_entries; i++) {
-                       user = info.info1.entries[i].account_name.string;
-                       if (c->opt_long_list_entries)
-                               desc = info.info1.entries[i].description.string;
+
                        if (c->opt_long_list_entries)
-                               printf("%-21.21s %s\n", user, desc);
+                               printf("%-21.21s %s\n", info->usri1_name,
+                                       info->usri1_comment);
                        else
-                               printf("%s\n", user);
+                               printf("%s\n", info->usri1_name);
+                       info++;
                }
-       } while (NT_STATUS_EQUAL(result, STATUS_MORE_ENTRIES));
 
- done:
-       return result;
+               NetApiBufferFree(buffer);
+
+               loop_count++;
+               start_idx += num_entries;
+
+       } while (status == ERROR_MORE_DATA);
+
+       return status;
 }
 
 /**
  * 'net rpc user' entrypoint.
- * @param argc  Standard main() style argc
- * @param argc  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  **/
 
 int net_rpc_user(struct net_context *c, int argc, const char **argv)
 {
        NET_API_STATUS status;
 
-       struct functable3 func[] = {
+       struct functable func[] = {
                {
                        "add",
                        rpc_user_add,
@@ -1228,6 +942,9 @@ int net_rpc_user(struct net_context *c, int argc, const char **argv)
        }
        libnetapi_set_username(c->netapi_ctx, c->opt_user_name);
        libnetapi_set_password(c->netapi_ctx, c->opt_password);
+       if (c->opt_kerberos) {
+               libnetapi_set_use_kerberos(c->netapi_ctx);
+       }
 
        if (argc == 0) {
                if (c->display_usage) {
@@ -1238,12 +955,10 @@ int net_rpc_user(struct net_context *c, int argc, const char **argv)
                        return 0;
                }
 
-               return run_rpc_command(c, NULL,PI_SAMR, 0,
-                                      rpc_user_list_internals,
-                                      argc, argv);
+               return rpc_user_list(c, argc, argv);
        }
 
-       return net_run_function3(c, argc, argv, "net rpc user", func);
+       return net_run_function(c, argc, argv, "net rpc user", func);
 }
 
 static NTSTATUS rpc_sh_user_list(struct net_context *c,
@@ -1252,9 +967,7 @@ static NTSTATUS rpc_sh_user_list(struct net_context *c,
                                 struct rpc_pipe_client *pipe_hnd,
                                 int argc, const char **argv)
 {
-       return rpc_user_list_internals(c, ctx->domain_sid, ctx->domain_name,
-                                      ctx->cli, pipe_hnd, mem_ctx,
-                                      argc, argv);
+       return werror_to_ntstatus(W_ERROR(rpc_user_list(c, argc, argv)));
 }
 
 static NTSTATUS rpc_sh_user_info(struct net_context *c,
@@ -1263,9 +976,7 @@ static NTSTATUS rpc_sh_user_info(struct net_context *c,
                                 struct rpc_pipe_client *pipe_hnd,
                                 int argc, const char **argv)
 {
-       return rpc_user_info_internals(c, ctx->domain_sid, ctx->domain_name,
-                                      ctx->cli, pipe_hnd, mem_ctx,
-                                      argc, argv);
+       return werror_to_ntstatus(W_ERROR(rpc_user_info(c, argc, argv)));
 }
 
 static NTSTATUS rpc_sh_handle_user(struct net_context *c,
@@ -1278,10 +989,10 @@ static NTSTATUS rpc_sh_handle_user(struct net_context *c,
                                           TALLOC_CTX *mem_ctx,
                                           struct rpc_sh_ctx *ctx,
                                           struct rpc_pipe_client *pipe_hnd,
-                                          POLICY_HND *user_hnd,
+                                          struct policy_handle *user_hnd,
                                           int argc, const char **argv))
 {
-       POLICY_HND connect_pol, domain_pol, user_pol;
+       struct policy_handle connect_pol, domain_pol, user_pol;
        NTSTATUS result = NT_STATUS_UNSUCCESSFUL;
        DOM_SID sid;
        uint32 rid;
@@ -1362,7 +1073,7 @@ static NTSTATUS rpc_sh_user_show_internals(struct net_context *c,
                                           TALLOC_CTX *mem_ctx,
                                           struct rpc_sh_ctx *ctx,
                                           struct rpc_pipe_client *pipe_hnd,
-                                          POLICY_HND *user_hnd,
+                                          struct policy_handle *user_hnd,
                                           int argc, const char **argv)
 {
        NTSTATUS result;
@@ -1413,7 +1124,7 @@ static NTSTATUS rpc_sh_user_str_edit_internals(struct net_context *c,
                                               TALLOC_CTX *mem_ctx,
                                               struct rpc_sh_ctx *ctx,
                                               struct rpc_pipe_client *pipe_hnd,
-                                              POLICY_HND *user_hnd,
+                                              struct policy_handle *user_hnd,
                                               int argc, const char **argv)
 {
        NTSTATUS result;
@@ -1498,7 +1209,7 @@ static NTSTATUS rpc_sh_user_flag_edit_internals(struct net_context *c,
                                                TALLOC_CTX *mem_ctx,
                                                struct rpc_sh_ctx *ctx,
                                                struct rpc_pipe_client *pipe_hnd,
-                                               POLICY_HND *user_hnd,
+                                               struct policy_handle *user_hnd,
                                                int argc, const char **argv)
 {
        NTSTATUS result;
@@ -1576,34 +1287,34 @@ struct rpc_sh_cmd *net_rpc_user_edit_cmds(struct net_context *c,
 {
        static struct rpc_sh_cmd cmds[] = {
 
-               { "fullname", NULL, PI_SAMR, rpc_sh_user_str_edit,
+               { "fullname", NULL, &ndr_table_samr.syntax_id, rpc_sh_user_str_edit,
                  "Show/Set a user's full name" },
 
-               { "homedir", NULL, PI_SAMR, rpc_sh_user_str_edit,
+               { "homedir", NULL, &ndr_table_samr.syntax_id, rpc_sh_user_str_edit,
                  "Show/Set a user's home directory" },
 
-               { "homedrive", NULL, PI_SAMR, rpc_sh_user_str_edit,
+               { "homedrive", NULL, &ndr_table_samr.syntax_id, rpc_sh_user_str_edit,
                  "Show/Set a user's home drive" },
 
-               { "logonscript", NULL, PI_SAMR, rpc_sh_user_str_edit,
+               { "logonscript", NULL, &ndr_table_samr.syntax_id, rpc_sh_user_str_edit,
                  "Show/Set a user's logon script" },
 
-               { "profilepath", NULL, PI_SAMR, rpc_sh_user_str_edit,
+               { "profilepath", NULL, &ndr_table_samr.syntax_id, rpc_sh_user_str_edit,
                  "Show/Set a user's profile path" },
 
-               { "description", NULL, PI_SAMR, rpc_sh_user_str_edit,
+               { "description", NULL, &ndr_table_samr.syntax_id, rpc_sh_user_str_edit,
                  "Show/Set a user's description" },
 
-               { "disabled", NULL, PI_SAMR, rpc_sh_user_flag_edit,
+               { "disabled", NULL, &ndr_table_samr.syntax_id, rpc_sh_user_flag_edit,
                  "Show/Set whether a user is disabled" },
 
-               { "autolock", NULL, PI_SAMR, rpc_sh_user_flag_edit,
+               { "autolock", NULL, &ndr_table_samr.syntax_id, rpc_sh_user_flag_edit,
                  "Show/Set whether a user locked out" },
 
-               { "pwnotreq", NULL, PI_SAMR, rpc_sh_user_flag_edit,
+               { "pwnotreq", NULL, &ndr_table_samr.syntax_id, rpc_sh_user_flag_edit,
                  "Show/Set whether a user does not need a password" },
 
-               { "pwnoexp", NULL, PI_SAMR, rpc_sh_user_flag_edit,
+               { "pwnoexp", NULL, &ndr_table_samr.syntax_id, rpc_sh_user_flag_edit,
                  "Show/Set whether a user's password does not expire" },
 
                { NULL, NULL, 0, NULL, NULL }
@@ -1618,13 +1329,13 @@ struct rpc_sh_cmd *net_rpc_user_cmds(struct net_context *c,
 {
        static struct rpc_sh_cmd cmds[] = {
 
-               { "list", NULL, PI_SAMR, rpc_sh_user_list,
+               { "list", NULL, &ndr_table_samr.syntax_id, rpc_sh_user_list,
                  "List available users" },
 
-               { "info", NULL, PI_SAMR, rpc_sh_user_info,
+               { "info", NULL, &ndr_table_samr.syntax_id, rpc_sh_user_info,
                  "List the domain groups a user is member of" },
 
-               { "show", NULL, PI_SAMR, rpc_sh_user_show,
+               { "show", NULL, &ndr_table_samr.syntax_id, rpc_sh_user_show,
                  "Show info about a user" },
 
                { "edit", net_rpc_user_edit_cmds, 0, NULL,
@@ -1639,9 +1350,9 @@ struct rpc_sh_cmd *net_rpc_user_cmds(struct net_context *c,
 /****************************************************************************/
 
 /**
- * Basic usage function for 'net rpc group'
+ * Basic usage function for 'net rpc group'.
  * @param argc Standard main() style argc.
- * @param argv Standard main() style argv.  Initial components are already
+ * @param argv Standard main() style argv. Initial components are already
  *             stripped.
  **/
 
@@ -1651,17 +1362,17 @@ static int rpc_group_usage(struct net_context *c, int argc, const char **argv)
 }
 
 /**
- * Delete group on a remote RPC server
+ * Delete group on a remote RPC server.
  *
  * All parameters are provided by the run_rpc_command function, except for
- * argc, argv which are passes through.
+ * argc, argv which are passed through.
  *
- * @param domain_sid The domain sid acquired from the remote server
+ * @param domain_sid The domain sid acquired from the remote server.
  * @param cli A cli_state connected to the server.
- * @param mem_ctx Talloc context, destoyed on completion of the function.
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param mem_ctx Talloc context, destroyed on completion of the function.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
  * @return Normal NTSTATUS return.
  **/
@@ -1675,14 +1386,14 @@ static NTSTATUS rpc_group_delete_internals(struct net_context *c,
                                        int argc,
                                        const char **argv)
 {
-       POLICY_HND connect_pol, domain_pol, group_pol, user_pol;
+       struct policy_handle connect_pol, domain_pol, group_pol, user_pol;
        bool group_is_primary = false;
        NTSTATUS result = NT_STATUS_UNSUCCESSFUL;
        uint32_t group_rid;
        struct samr_RidTypeArray *rids = NULL;
        /* char **names; */
        int i;
-       /* DOM_GID *user_gids; */
+       /* struct samr_RidWithAttribute *user_gids; */
 
        struct samr_Ids group_rids, name_types;
        struct lsa_String lsa_acct_name;
@@ -1862,8 +1573,8 @@ static NTSTATUS rpc_group_delete_internals(struct net_context *c,
 
 static int rpc_group_delete(struct net_context *c, int argc, const char **argv)
 {
-       return run_rpc_command(c, NULL, PI_SAMR, 0, rpc_group_delete_internals,
-                               argc,argv);
+       return run_rpc_command(c, NULL, &ndr_table_samr.syntax_id, 0,
+                              rpc_group_delete_internals, argc,argv);
 }
 
 static int rpc_group_add_internals(struct net_context *c, int argc, const char **argv)
@@ -1898,83 +1609,42 @@ static int rpc_group_add_internals(struct net_context *c, int argc, const char *
        return 0;
 }
 
-static NTSTATUS rpc_alias_add_internals(struct net_context *c,
-                                       const DOM_SID *domain_sid,
-                                       const char *domain_name,
-                                       struct cli_state *cli,
-                                       struct rpc_pipe_client *pipe_hnd,
-                                       TALLOC_CTX *mem_ctx,
-                                       int argc,
-                                       const char **argv)
+static int rpc_alias_add_internals(struct net_context *c, int argc, const char **argv)
 {
-       POLICY_HND connect_pol, domain_pol, alias_pol;
-       NTSTATUS result = NT_STATUS_UNSUCCESSFUL;
-       union samr_AliasInfo alias_info;
-       struct lsa_String alias_name;
-       uint32_t rid = 0;
+       NET_API_STATUS status;
+       struct LOCALGROUP_INFO_1 info1;
+       uint32_t parm_error = 0;
 
        if (argc != 1 || c->display_usage) {
                rpc_group_usage(c, argc, argv);
-               return NT_STATUS_OK;
+               return 0;
        }
 
-       init_lsa_String(&alias_name, argv[0]);
-
-       /* Get sam policy handle */
-
-       result = rpccli_samr_Connect2(pipe_hnd, mem_ctx,
-                                     pipe_hnd->desthost,
-                                     MAXIMUM_ALLOWED_ACCESS,
-                                     &connect_pol);
-       if (!NT_STATUS_IS_OK(result)) goto done;
-
-       /* Get domain policy handle */
-
-       result = rpccli_samr_OpenDomain(pipe_hnd, mem_ctx,
-                                       &connect_pol,
-                                       MAXIMUM_ALLOWED_ACCESS,
-                                       CONST_DISCARD(struct dom_sid2 *, domain_sid),
-                                       &domain_pol);
-       if (!NT_STATUS_IS_OK(result)) goto done;
-
-       /* Create the group */
-
-       result = rpccli_samr_CreateDomAlias(pipe_hnd, mem_ctx,
-                                           &domain_pol,
-                                           &alias_name,
-                                           MAXIMUM_ALLOWED_ACCESS,
-                                           &alias_pol,
-                                           &rid);
-       if (!NT_STATUS_IS_OK(result)) goto done;
-
-       if (strlen(c->opt_comment) == 0) goto done;
-
-       /* We've got a comment to set */
-
-       init_lsa_String(&alias_info.description, c->opt_comment);
+       ZERO_STRUCT(info1);
 
-       result = rpccli_samr_SetAliasInfo(pipe_hnd, mem_ctx,
-                                         &alias_pol,
-                                         3,
-                                         &alias_info);
+       info1.lgrpi1_name = argv[0];
+       if (c->opt_comment && strlen(c->opt_comment) > 0) {
+               info1.lgrpi1_comment = c->opt_comment;
+       }
 
-       if (!NT_STATUS_IS_OK(result)) goto done;
+       status = NetLocalGroupAdd(c->opt_host, 1, (uint8_t *)&info1, &parm_error);
 
- done:
-       if (NT_STATUS_IS_OK(result))
-               DEBUG(5, ("add alias succeeded\n"));
-       else
-               d_fprintf(stderr, "add alias failed: %s\n", nt_errstr(result));
+       if (status != 0) {
+               d_fprintf(stderr, "Failed to add alias '%s' with: %s.\n",
+                       argv[0], libnetapi_get_error_string(c->netapi_ctx,
+                                                           status));
+               return -1;
+       } else {
+               d_printf("Added alias '%s'.\n", argv[0]);
+       }
 
-       return result;
+       return 0;
 }
 
 static int rpc_group_add(struct net_context *c, int argc, const char **argv)
 {
        if (c->opt_localgroup)
-               return run_rpc_command(c, NULL, PI_SAMR, 0,
-                                      rpc_alias_add_internals,
-                                      argc, argv);
+               return rpc_alias_add_internals(c, argc, argv);
 
        return rpc_group_add_internals(c, argc, argv);
 }
@@ -1987,17 +1657,18 @@ static NTSTATUS get_sid_from_name(struct cli_state *cli,
 {
        DOM_SID *sids = NULL;
        enum lsa_SidType *types = NULL;
-       struct rpc_pipe_client *pipe_hnd;
-       POLICY_HND lsa_pol;
+       struct rpc_pipe_client *pipe_hnd = NULL;
+       struct policy_handle lsa_pol;
        NTSTATUS result = NT_STATUS_UNSUCCESSFUL;
 
-       pipe_hnd = cli_rpc_pipe_open_noauth(cli, PI_LSARPC, &result);
-       if (!pipe_hnd) {
+       result = cli_rpc_pipe_open_noauth(cli, &ndr_table_lsarpc.syntax_id,
+                                         &pipe_hnd);
+       if (!NT_STATUS_IS_OK(result)) {
                goto done;
        }
 
        result = rpccli_lsa_open_policy(pipe_hnd, mem_ctx, false,
-                                    SEC_RIGHTS_MAXIMUM_ALLOWED, &lsa_pol);
+                                    SEC_FLAG_MAXIMUM_ALLOWED, &lsa_pol);
 
        if (!NT_STATUS_IS_OK(result)) {
                goto done;
@@ -2039,10 +1710,10 @@ static NTSTATUS rpc_add_groupmem(struct rpc_pipe_client *pipe_hnd,
                                const DOM_SID *group_sid,
                                const char *member)
 {
-       POLICY_HND connect_pol, domain_pol;
+       struct policy_handle connect_pol, domain_pol;
        NTSTATUS result;
        uint32 group_rid;
-       POLICY_HND group_pol;
+       struct policy_handle group_pol;
 
        struct samr_Ids rids, rid_types;
        struct lsa_String lsa_acct_name;
@@ -2113,10 +1784,10 @@ static NTSTATUS rpc_add_aliasmem(struct rpc_pipe_client *pipe_hnd,
                                const DOM_SID *alias_sid,
                                const char *member)
 {
-       POLICY_HND connect_pol, domain_pol;
+       struct policy_handle connect_pol, domain_pol;
        NTSTATUS result;
        uint32 alias_rid;
-       POLICY_HND alias_pol;
+       struct policy_handle alias_pol;
 
        DOM_SID member_sid;
        enum lsa_SidType member_type;
@@ -2236,7 +1907,7 @@ static NTSTATUS rpc_group_addmem_internals(struct net_context *c,
 
 static int rpc_group_addmem(struct net_context *c, int argc, const char **argv)
 {
-       return run_rpc_command(c, NULL, PI_SAMR, 0,
+       return run_rpc_command(c, NULL, &ndr_table_samr.syntax_id, 0,
                               rpc_group_addmem_internals,
                               argc, argv);
 }
@@ -2247,10 +1918,10 @@ static NTSTATUS rpc_del_groupmem(struct net_context *c,
                                const DOM_SID *group_sid,
                                const char *member)
 {
-       POLICY_HND connect_pol, domain_pol;
+       struct policy_handle connect_pol, domain_pol;
        NTSTATUS result;
        uint32 group_rid;
-       POLICY_HND group_pol;
+       struct policy_handle group_pol;
 
        struct samr_Ids rids, rid_types;
        struct lsa_String lsa_acct_name;
@@ -2315,10 +1986,10 @@ static NTSTATUS rpc_del_aliasmem(struct rpc_pipe_client *pipe_hnd,
                                const DOM_SID *alias_sid,
                                const char *member)
 {
-       POLICY_HND connect_pol, domain_pol;
+       struct policy_handle connect_pol, domain_pol;
        NTSTATUS result;
        uint32 alias_rid;
-       POLICY_HND alias_pol;
+       struct policy_handle alias_pol;
 
        DOM_SID member_sid;
        enum lsa_SidType member_type;
@@ -2435,23 +2106,23 @@ static NTSTATUS rpc_group_delmem_internals(struct net_context *c,
 
 static int rpc_group_delmem(struct net_context *c, int argc, const char **argv)
 {
-       return run_rpc_command(c, NULL, PI_SAMR, 0,
+       return run_rpc_command(c, NULL, &ndr_table_samr.syntax_id, 0,
                               rpc_group_delmem_internals,
                               argc, argv);
 }
 
 /**
- * List groups on a remote RPC server
+ * List groups on a remote RPC server.
  *
  * All parameters are provided by the run_rpc_command function, except for
  * argc, argv which are passes through.
  *
- * @param domain_sid The domain sid acquired from the remote server
+ * @param domain_sid The domain sid acquired from the remote server.
  * @param cli A cli_state connected to the server.
- * @param mem_ctx Talloc context, destoyed on completion of the function.
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param mem_ctx Talloc context, destroyed on completion of the function.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
  * @return Normal NTSTATUS return.
  **/
@@ -2465,7 +2136,7 @@ static NTSTATUS rpc_group_list_internals(struct net_context *c,
                                        int argc,
                                        const char **argv)
 {
-       POLICY_HND connect_pol, domain_pol;
+       struct policy_handle connect_pol, domain_pol;
        NTSTATUS result = NT_STATUS_UNSUCCESSFUL;
        uint32 start_idx=0, max_entries=250, num_entries, i, loop_count = 0;
        struct samr_SamArray *groups = NULL;
@@ -2588,7 +2259,7 @@ static NTSTATUS rpc_group_list_internals(struct net_context *c,
 
                        if (c->opt_long_list_entries) {
 
-                               POLICY_HND alias_pol;
+                               struct policy_handle alias_pol;
                                union samr_AliasInfo *info = NULL;
 
                                if ((NT_STATUS_IS_OK(rpccli_samr_OpenAlias(pipe_hnd, mem_ctx,
@@ -2647,7 +2318,7 @@ static NTSTATUS rpc_group_list_internals(struct net_context *c,
 
                        if (c->opt_long_list_entries) {
 
-                               POLICY_HND alias_pol;
+                               struct policy_handle alias_pol;
                                union samr_AliasInfo *info = NULL;
 
                                if ((NT_STATUS_IS_OK(rpccli_samr_OpenAlias(pipe_hnd, mem_ctx,
@@ -2681,7 +2352,7 @@ static NTSTATUS rpc_group_list_internals(struct net_context *c,
 
 static int rpc_group_list(struct net_context *c, int argc, const char **argv)
 {
-       return run_rpc_command(c, NULL, PI_SAMR, 0,
+       return run_rpc_command(c, NULL, &ndr_table_samr.syntax_id, 0,
                               rpc_group_list_internals,
                               argc, argv);
 }
@@ -2691,11 +2362,11 @@ static NTSTATUS rpc_list_group_members(struct net_context *c,
                                        TALLOC_CTX *mem_ctx,
                                        const char *domain_name,
                                        const DOM_SID *domain_sid,
-                                       POLICY_HND *domain_pol,
+                                       struct policy_handle *domain_pol,
                                        uint32 rid)
 {
        NTSTATUS result;
-       POLICY_HND group_pol;
+       struct policy_handle group_pol;
        uint32 num_members, *group_rids;
        int i;
        struct samr_RidTypeArray *rids = NULL;
@@ -2766,12 +2437,12 @@ static NTSTATUS rpc_list_group_members(struct net_context *c,
 static NTSTATUS rpc_list_alias_members(struct net_context *c,
                                        struct rpc_pipe_client *pipe_hnd,
                                        TALLOC_CTX *mem_ctx,
-                                       POLICY_HND *domain_pol,
+                                       struct policy_handle *domain_pol,
                                        uint32 rid)
 {
        NTSTATUS result;
        struct rpc_pipe_client *lsa_pipe;
-       POLICY_HND alias_pol, lsa_pol;
+       struct policy_handle alias_pol, lsa_pol;
        uint32 num_members;
        DOM_SID *alias_sids;
        char **domains;
@@ -2804,16 +2475,17 @@ static NTSTATUS rpc_list_alias_members(struct net_context *c,
                return NT_STATUS_OK;
        }
 
-       lsa_pipe = cli_rpc_pipe_open_noauth(rpc_pipe_np_smb_conn(pipe_hnd),
-                                           PI_LSARPC, &result);
-       if (!lsa_pipe) {
+       result = cli_rpc_pipe_open_noauth(rpc_pipe_np_smb_conn(pipe_hnd),
+                                         &ndr_table_lsarpc.syntax_id,
+                                         &lsa_pipe);
+       if (!NT_STATUS_IS_OK(result)) {
                d_fprintf(stderr, "Couldn't open LSA pipe. Error was %s\n",
                        nt_errstr(result) );
                return result;
        }
 
        result = rpccli_lsa_open_policy(lsa_pipe, mem_ctx, true,
-                                    SEC_RIGHTS_MAXIMUM_ALLOWED, &lsa_pol);
+                                    SEC_FLAG_MAXIMUM_ALLOWED, &lsa_pol);
 
        if (!NT_STATUS_IS_OK(result)) {
                d_fprintf(stderr, "Couldn't open LSA policy handle\n");
@@ -2873,7 +2545,7 @@ static NTSTATUS rpc_group_members_internals(struct net_context *c,
                                        const char **argv)
 {
        NTSTATUS result;
-       POLICY_HND connect_pol, domain_pol;
+       struct policy_handle connect_pol, domain_pol;
        struct samr_Ids rids, rid_types;
        struct lsa_String lsa_acct_name;
 
@@ -2966,7 +2638,7 @@ static int rpc_group_members(struct net_context *c, int argc, const char **argv)
                return rpc_group_usage(c, argc, argv);
        }
 
-       return run_rpc_command(c, NULL, PI_SAMR, 0,
+       return run_rpc_command(c, NULL, &ndr_table_samr.syntax_id, 0,
                               rpc_group_members_internals,
                               argc, argv);
 }
@@ -3011,16 +2683,16 @@ static int rpc_group_rename(struct net_context *c, int argc, const char **argv)
 
 /**
  * 'net rpc group' entrypoint.
- * @param argc  Standard main() style argc
- * @param argc  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  **/
 
 int net_rpc_group(struct net_context *c, int argc, const char **argv)
 {
        NET_API_STATUS status;
 
-       struct functable3 func[] = {
+       struct functable func[] = {
                {
                        "add",
                        rpc_group_add,
@@ -3086,6 +2758,9 @@ int net_rpc_group(struct net_context *c, int argc, const char **argv)
        }
        libnetapi_set_username(c->netapi_ctx, c->opt_user_name);
        libnetapi_set_password(c->netapi_ctx, c->opt_password);
+       if (c->opt_kerberos) {
+               libnetapi_set_use_kerberos(c->netapi_ctx);
+       }
 
        if (argc == 0) {
                if (c->display_usage) {
@@ -3097,12 +2772,12 @@ int net_rpc_group(struct net_context *c, int argc, const char **argv)
                        return 0;
                }
 
-               return run_rpc_command(c, NULL, PI_SAMR, 0,
+               return run_rpc_command(c, NULL, &ndr_table_samr.syntax_id, 0,
                                       rpc_group_list_internals,
                                       argc, argv);
        }
 
-       return net_run_function3(c, argc, argv, "net rpc group", func);
+       return net_run_function(c, argc, argv, "net rpc group", func);
 }
 
 /****************************************************************************/
@@ -3113,148 +2788,97 @@ static int rpc_share_usage(struct net_context *c, int argc, const char **argv)
 }
 
 /**
- * Add a share on a remote RPC server
- *
- * All parameters are provided by the run_rpc_command function, except for
- * argc, argv which are passes through.
+ * Add a share on a remote RPC server.
  *
- * @param domain_sid The domain sid acquired from the remote server
- * @param cli A cli_state connected to the server.
- * @param mem_ctx Talloc context, destoyed on completion of the function.
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return Normal NTSTATUS return.
+ * @return A shell status integer (0 for success).
  **/
-static NTSTATUS rpc_share_add_internals(struct net_context *c,
-                                       const DOM_SID *domain_sid,
-                                       const char *domain_name,
-                                       struct cli_state *cli,
-                                       struct rpc_pipe_client *pipe_hnd,
-                                       TALLOC_CTX *mem_ctx,int argc,
-                                       const char **argv)
+
+static int rpc_share_add(struct net_context *c, int argc, const char **argv)
 {
-       WERROR result;
-       NTSTATUS status;
+       NET_API_STATUS status;
        char *sharename;
        char *path;
        uint32 type = STYPE_DISKTREE; /* only allow disk shares to be added */
        uint32 num_users=0, perms=0;
        char *password=NULL; /* don't allow a share password */
-       uint32 level = 2;
-       union srvsvc_NetShareInfo info;
-       struct srvsvc_NetShareInfo2 info2;
+       struct SHARE_INFO_2 i2;
        uint32_t parm_error = 0;
 
-       if ((sharename = talloc_strdup(mem_ctx, argv[0])) == NULL) {
-               return NT_STATUS_NO_MEMORY;
+       if ((argc < 1) || !strchr(argv[0], '=') || c->display_usage) {
+               return rpc_share_usage(c, argc, argv);
+       }
+
+       if ((sharename = talloc_strdup(c, argv[0])) == NULL) {
+               return -1;
        }
 
        path = strchr(sharename, '=');
-       if (!path)
-               return NT_STATUS_UNSUCCESSFUL;
-       *path++ = '\0';
+       if (!path) {
+               return -1;
+       }
 
-       info2.name              = sharename;
-       info2.type              = type;
-       info2.comment           = c->opt_comment;
-       info2.permissions       = perms;
-       info2.max_users         = c->opt_maxusers;
-       info2.current_users     = num_users;
-       info2.path              = path;
-       info2.password          = password;
-
-       info.info2 = &info2;
-
-       status = rpccli_srvsvc_NetShareAdd(pipe_hnd, mem_ctx,
-                                          pipe_hnd->desthost,
-                                          level,
-                                          &info,
-                                          &parm_error,
-                                          &result);
-       return status;
-}
+       *path++ = '\0';
 
-static int rpc_share_add(struct net_context *c, int argc, const char **argv)
-{
-       if ((argc < 1) || !strchr(argv[0], '=') || c->display_usage) {
-               return rpc_share_usage(c, argc, argv);
+       i2.shi2_netname         = sharename;
+       i2.shi2_type            = type;
+       i2.shi2_remark          = c->opt_comment;
+       i2.shi2_permissions     = perms;
+       i2.shi2_max_uses        = c->opt_maxusers;
+       i2.shi2_current_uses    = num_users;
+       i2.shi2_path            = path;
+       i2.shi2_passwd          = password;
+
+       status = NetShareAdd(c->opt_host,
+                            2,
+                            (uint8_t *)&i2,
+                            &parm_error);
+       if (status != 0) {
+               printf("NetShareAdd failed with: %s\n",
+                       libnetapi_get_error_string(c->netapi_ctx, status));
        }
-       return run_rpc_command(c, NULL, PI_SRVSVC, 0,
-                              rpc_share_add_internals,
-                              argc, argv);
-}
-
-/**
- * Delete a share on a remote RPC server
- *
- * All parameters are provided by the run_rpc_command function, except for
- * argc, argv which are passes through.
- *
- * @param domain_sid The domain sid acquired from the remote server
- * @param cli A cli_state connected to the server.
- * @param mem_ctx Talloc context, destoyed on completion of the function.
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
- *
- * @return Normal NTSTATUS return.
- **/
-static NTSTATUS rpc_share_del_internals(struct net_context *c,
-                                       const DOM_SID *domain_sid,
-                                       const char *domain_name,
-                                       struct cli_state *cli,
-                                       struct rpc_pipe_client *pipe_hnd,
-                                       TALLOC_CTX *mem_ctx,
-                                       int argc,
-                                       const char **argv)
-{
-       WERROR result;
 
-       return rpccli_srvsvc_NetShareDel(pipe_hnd, mem_ctx,
-                                        pipe_hnd->desthost,
-                                        argv[0],
-                                        0,
-                                        &result);
+       return status;
 }
 
 /**
- * Delete a share on a remote RPC server
+ * Delete a share on a remote RPC server.
  *
- * @param domain_sid The domain sid acquired from the remote server
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param domain_sid The domain sid acquired from the remote server.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  **/
 static int rpc_share_delete(struct net_context *c, int argc, const char **argv)
 {
        if (argc < 1 || c->display_usage) {
                return rpc_share_usage(c, argc, argv);
        }
-       return run_rpc_command(c, NULL, PI_SRVSVC, 0,
-                              rpc_share_del_internals,
-                              argc, argv);
+
+       return NetShareDel(c->opt_host, argv[0], 0);
 }
 
 /**
  * Formatted print of share info
  *
- * @param info1  pointer to SRV_SHARE_INFO_1 to format
+ * @param r  pointer to SHARE_INFO_1 to format
  **/
 
 static void display_share_info_1(struct net_context *c,
-                                struct srvsvc_NetShareInfo1 *r)
+                                struct SHARE_INFO_1 *r)
 {
        if (c->opt_long_list_entries) {
                d_printf("%-12s %-8.8s %-50s\n",
-                        r->name,
-                        c->share_type[r->type & ~(STYPE_TEMPORARY|STYPE_HIDDEN)],
-                        r->comment);
+                        r->shi1_netname,
+                        net_share_type_str(r->shi1_type & ~(STYPE_TEMPORARY|STYPE_HIDDEN)),
+                        r->shi1_remark);
        } else {
-               d_printf("%s\n", r->name);
+               d_printf("%s\n", r->shi1_netname);
        }
 }
 
@@ -3348,83 +2972,56 @@ done:
        return result;
 }
 
-/**
- * List shares on a remote RPC server
- *
- * All parameters are provided by the run_rpc_command function, except for
- * argc, argv which are passes through.
- *
- * @param domain_sid The domain sid acquired from the remote server
- * @param cli A cli_state connected to the server.
- * @param mem_ctx Talloc context, destoyed on completion of the function.
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
- *
- * @return Normal NTSTATUS return.
- **/
-
-static NTSTATUS rpc_share_list_internals(struct net_context *c,
-                                       const DOM_SID *domain_sid,
-                                       const char *domain_name,
-                                       struct cli_state *cli,
-                                       struct rpc_pipe_client *pipe_hnd,
-                                       TALLOC_CTX *mem_ctx,
-                                       int argc,
-                                       const char **argv)
-{
-       struct srvsvc_NetShareInfoCtr info_ctr;
-       struct srvsvc_NetShareCtr1 ctr1;
-       WERROR result;
-       uint32 i, level = 1;
-
-       ZERO_STRUCT(info_ctr);
-       ZERO_STRUCT(ctr1);
-
-       info_ctr.level = 1;
-       info_ctr.ctr.ctr1 = &ctr1;
-
-       result = get_share_info(c, pipe_hnd, mem_ctx, level, argc, argv,
-                               &info_ctr);
-       if (!W_ERROR_IS_OK(result))
-               goto done;
-
-       /* Display results */
-
-       if (c->opt_long_list_entries) {
-               d_printf(
-       "\nEnumerating shared resources (exports) on remote server:\n\n"
-       "\nShare name   Type     Description\n"
-       "----------   ----     -----------\n");
-       }
-       for (i = 0; i < info_ctr.ctr.ctr1->count; i++)
-               display_share_info_1(c, &info_ctr.ctr.ctr1->array[i]);
- done:
-       return W_ERROR_IS_OK(result) ? NT_STATUS_OK : NT_STATUS_UNSUCCESSFUL;
-}
-
 /***
  * 'net rpc share list' entrypoint.
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  **/
 static int rpc_share_list(struct net_context *c, int argc, const char **argv)
 {
+       NET_API_STATUS status;
+       struct SHARE_INFO_1 *i1 = NULL;
+       uint32_t entries_read = 0;
+       uint32_t total_entries = 0;
+       uint32_t resume_handle = 0;
+       uint32_t i, level = 1;
+
        if (c->display_usage) {
                d_printf("Usage\n"
                         "net rpc share list\n"
                         "    List shares on remote server\n");
                return 0;
        }
-
-       return run_rpc_command(c, NULL, PI_SRVSVC, 0, rpc_share_list_internals,
-                              argc, argv);
+
+       status = NetShareEnum(c->opt_host,
+                             level,
+                             (uint8_t **)(void *)&i1,
+                             (uint32_t)-1,
+                             &entries_read,
+                             &total_entries,
+                             &resume_handle);
+       if (status != 0) {
+               goto done;
+       }
+
+       /* Display results */
+
+       if (c->opt_long_list_entries) {
+               d_printf(
+       "\nEnumerating shared resources (exports) on remote server:\n\n"
+       "\nShare name   Type     Description\n"
+       "----------   ----     -----------\n");
+       }
+       for (i = 0; i < entries_read; i++)
+               display_share_info_1(c, &i1[i]);
+ done:
+       return status;
 }
 
 static bool check_share_availability(struct cli_state *cli, const char *netname)
 {
-       if (!cli_send_tconX(cli, netname, "A:", "", 0)) {
+       if (!NT_STATUS_IS_OK(cli_tcon_andx(cli, netname, "A:", "", 0))) {
                d_printf("skipping   [%s]: not a file share.\n", netname);
                return false;
        }
@@ -3459,17 +3056,17 @@ static bool check_share_sanity(struct net_context *c, struct cli_state *cli,
 }
 
 /**
- * Migrate shares from a remote RPC server to the local RPC server
+ * Migrate shares from a remote RPC server to the local RPC server.
  *
  * All parameters are provided by the run_rpc_command function, except for
  * argc, argv which are passed through.
  *
- * @param domain_sid The domain sid acquired from the remote server
+ * @param domain_sid The domain sid acquired from the remote server.
  * @param cli A cli_state connected to the server.
  * @param mem_ctx Talloc context, destroyed on completion of the function.
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
  * @return Normal NTSTATUS return.
  **/
@@ -3498,7 +3095,8 @@ static NTSTATUS rpc_share_migrate_shares_internals(struct net_context *c,
                goto done;
 
        /* connect destination PI_SRVSVC */
-        nt_status = connect_dst_pipe(c, &cli_dst, &srvsvc_pipe, PI_SRVSVC);
+        nt_status = connect_dst_pipe(c, &cli_dst, &srvsvc_pipe,
+                                    &ndr_table_srvsvc.syntax_id);
         if (!NT_STATUS_IS_OK(nt_status))
                 return nt_status;
 
@@ -3529,14 +3127,14 @@ static NTSTATUS rpc_share_migrate_shares_internals(struct net_context *c,
                                                      &parm_error,
                                                      &result);
 
-                if (W_ERROR_V(result) == W_ERROR_V(WERR_ALREADY_EXISTS)) {
+                if (W_ERROR_V(result) == W_ERROR_V(WERR_FILE_EXISTS)) {
                        printf("           [%s] does already exist\n",
                                info502.name);
                        continue;
                }
 
                if (!NT_STATUS_IS_OK(nt_status) || !W_ERROR_IS_OK(result)) {
-                       printf("cannot add share: %s\n", dos_errstr(result));
+                       printf("cannot add share: %s\n", win_errstr(result));
                        goto done;
                }
 
@@ -3554,13 +3152,13 @@ done:
 }
 
 /**
- * Migrate shares from a rpc-server to another
+ * Migrate shares from a RPC server to another.
  *
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  **/
 static int rpc_share_migrate_shares(struct net_context *c, int argc,
                                    const char **argv)
@@ -3577,7 +3175,7 @@ static int rpc_share_migrate_shares(struct net_context *c, int argc,
                return -1;
        }
 
-       return run_rpc_command(c, NULL, PI_SRVSVC, 0,
+       return run_rpc_command(c, NULL, &ndr_table_srvsvc.syntax_id, 0,
                               rpc_share_migrate_shares_internals,
                               argc, argv);
 }
@@ -3701,7 +3299,7 @@ static bool sync_files(struct copy_clistate *cp_clistate, const char *mask)
 
        DEBUG(3,("calling cli_list with mask: %s\n", mask));
 
-       if ( !cli_resolve_path(talloc_tos(), "", cp_clistate->cli_share_src,
+       if ( !cli_resolve_path(talloc_tos(), "", NULL, cp_clistate->cli_share_src,
                                mask, &targetcli, &targetpath ) ) {
                d_fprintf(stderr, "cli_resolve_path %s failed with error: %s\n", 
                        mask, cli_errstr(cp_clistate->cli_share_src));
@@ -3757,17 +3355,17 @@ bool copy_top_level_perms(struct net_context *c,
 }
 
 /**
- * Sync all files inside a remote share to another share (over smb)
+ * Sync all files inside a remote share to another share (over smb).
  *
  * All parameters are provided by the run_rpc_command function, except for
- * argc, argv which are passes through.
+ * argc, argv which are passed through.
  *
- * @param domain_sid The domain sid acquired from the remote server
+ * @param domain_sid The domain sid acquired from the remote server.
  * @param cli A cli_state connected to the server.
- * @param mem_ctx Talloc context, destoyed on completion of the function.
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param mem_ctx Talloc context, destroyed on completion of the function.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
  * @return Normal NTSTATUS return.
  **/
@@ -3901,23 +3499,23 @@ static int rpc_share_migrate_files(struct net_context *c, int argc, const char *
                return -1;
        }
 
-       return run_rpc_command(c, NULL, PI_SRVSVC, 0,
+       return run_rpc_command(c, NULL, &ndr_table_srvsvc.syntax_id, 0,
                               rpc_share_migrate_files_internals,
                               argc, argv);
 }
 
 /**
- * Migrate share-ACLs from a remote RPC server to the local RPC srever
+ * Migrate share-ACLs from a remote RPC server to the local RPC server.
  *
  * All parameters are provided by the run_rpc_command function, except for
- * argc, argv which are passes through.
+ * argc, argv which are passed through.
  *
- * @param domain_sid The domain sid acquired from the remote server
+ * @param domain_sid The domain sid acquired from the remote server.
  * @param cli A cli_state connected to the server.
- * @param mem_ctx Talloc context, destoyed on completion of the function.
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param mem_ctx Talloc context, destroyed on completion of the function.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
  * @return Normal NTSTATUS return.
  **/
@@ -3948,7 +3546,8 @@ static NTSTATUS rpc_share_migrate_security_internals(struct net_context *c,
                goto done;
 
        /* connect destination PI_SRVSVC */
-        nt_status = connect_dst_pipe(c, &cli_dst, &srvsvc_pipe, PI_SRVSVC);
+        nt_status = connect_dst_pipe(c, &cli_dst, &srvsvc_pipe,
+                                    &ndr_table_srvsvc.syntax_id);
         if (!NT_STATUS_IS_OK(nt_status))
                 return nt_status;
 
@@ -3982,7 +3581,7 @@ static NTSTATUS rpc_share_migrate_security_internals(struct net_context *c,
                                                          &parm_error,
                                                          &result);
                if (!NT_STATUS_IS_OK(nt_status) || !W_ERROR_IS_OK(result)) {
-                       printf("cannot set share-acl: %s\n", dos_errstr(result));
+                       printf("cannot set share-acl: %s\n", win_errstr(result));
                        goto done;
                }
 
@@ -4000,13 +3599,13 @@ done:
 }
 
 /**
- * Migrate share-acls from a rpc-server to another
+ * Migrate share-acls from a RPC server to another.
  *
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  **/
 static int rpc_share_migrate_security(struct net_context *c, int argc,
                                      const char **argv)
@@ -4023,20 +3622,20 @@ static int rpc_share_migrate_security(struct net_context *c, int argc,
                return -1;
        }
 
-       return run_rpc_command(c, NULL, PI_SRVSVC, 0,
+       return run_rpc_command(c, NULL, &ndr_table_srvsvc.syntax_id, 0,
                               rpc_share_migrate_security_internals,
                               argc, argv);
 }
 
 /**
  * Migrate shares (including share-definitions, share-acls and files with acls/attrs)
- * from one server to another
+ * from one server to another.
  *
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  *
  **/
 static int rpc_share_migrate_all(struct net_context *c, int argc,
@@ -4059,17 +3658,17 @@ static int rpc_share_migrate_all(struct net_context *c, int argc,
        /* order is important. we don't want to be locked out by the share-acl
         * before copying files - gd */
 
-       ret = run_rpc_command(c, NULL, PI_SRVSVC, 0,
+       ret = run_rpc_command(c, NULL, &ndr_table_srvsvc.syntax_id, 0,
                              rpc_share_migrate_shares_internals, argc, argv);
        if (ret)
                return ret;
 
-       ret = run_rpc_command(c, NULL, PI_SRVSVC, 0,
+       ret = run_rpc_command(c, NULL, &ndr_table_srvsvc.syntax_id, 0,
                              rpc_share_migrate_files_internals, argc, argv);
        if (ret)
                return ret;
 
-       return run_rpc_command(c, NULL, PI_SRVSVC, 0,
+       return run_rpc_command(c, NULL, &ndr_table_srvsvc.syntax_id, 0,
                               rpc_share_migrate_security_internals, argc,
                               argv);
 }
@@ -4077,14 +3676,14 @@ static int rpc_share_migrate_all(struct net_context *c, int argc,
 
 /**
  * 'net rpc share migrate' entrypoint.
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  **/
 static int rpc_share_migrate(struct net_context *c, int argc, const char **argv)
 {
 
-       struct functable3 func[] = {
+       struct functable func[] = {
                {
                        "all",
                        rpc_share_migrate_all,
@@ -4122,7 +3721,7 @@ static int rpc_share_migrate(struct net_context *c, int argc, const char **argv)
 
        net_mode_share = NET_MODE_SHARE_MIGRATE;
 
-       return net_run_function3(c, argc, argv, "net rpc share migrate", func);
+       return net_run_function(c, argc, argv, "net rpc share migrate", func);
 }
 
 struct full_alias {
@@ -4153,13 +3752,13 @@ static void push_alias(TALLOC_CTX *mem_ctx, struct full_alias *alias)
 
 static NTSTATUS rpc_fetch_domain_aliases(struct rpc_pipe_client *pipe_hnd,
                                        TALLOC_CTX *mem_ctx,
-                                       POLICY_HND *connect_pol,
+                                       struct policy_handle *connect_pol,
                                        const DOM_SID *domain_sid)
 {
        uint32 start_idx, max_entries, num_entries, i;
        struct samr_SamArray *groups = NULL;
        NTSTATUS result;
-       POLICY_HND domain_pol;
+       struct policy_handle domain_pol;
 
        /* Get domain policy handle */
 
@@ -4183,7 +3782,7 @@ static NTSTATUS rpc_fetch_domain_aliases(struct rpc_pipe_client *pipe_hnd,
                                                       &num_entries);
                for (i = 0; i < num_entries; i++) {
 
-                       POLICY_HND alias_pol;
+                       struct policy_handle alias_pol;
                        struct full_alias alias;
                        struct lsa_SidArray sid_array;
                        int j;
@@ -4248,10 +3847,10 @@ static NTSTATUS rpc_aliaslist_dump(struct net_context *c,
 {
        int i;
        NTSTATUS result;
-       POLICY_HND lsa_pol;
+       struct policy_handle lsa_pol;
 
        result = rpccli_lsa_open_policy(pipe_hnd, mem_ctx, true,
-                                    SEC_RIGHTS_MAXIMUM_ALLOWED,
+                                    SEC_FLAG_MAXIMUM_ALLOWED,
                                     &lsa_pol);
        if (!NT_STATUS_IS_OK(result))
                return result;
@@ -4313,7 +3912,7 @@ static NTSTATUS rpc_aliaslist_internals(struct net_context *c,
                                        const char **argv)
 {
        NTSTATUS result;
-       POLICY_HND connect_pol;
+       struct policy_handle connect_pol;
 
        result = rpccli_samr_Connect2(pipe_hnd, mem_ctx,
                                      pipe_hnd->desthost,
@@ -4358,17 +3957,6 @@ static void free_user_token(NT_USER_TOKEN *token)
        SAFE_FREE(token->user_sids);
 }
 
-static bool is_sid_in_token(NT_USER_TOKEN *token, DOM_SID *sid)
-{
-       int i;
-
-       for (i=0; i<token->num_sids; i++) {
-               if (sid_compare(sid, &token->user_sids[i]) == 0)
-                       return true;
-       }
-       return false;
-}
-
 static void add_sid_to_token(NT_USER_TOKEN *token, DOM_SID *sid)
 {
        if (is_sid_in_token(token, sid))
@@ -4475,7 +4063,11 @@ static bool get_user_sids(const char *domain, const char *user, NT_USER_TOKEN *t
                return false;
        }
 
-       string_to_sid(&user_sid, sid_str);
+       if (!string_to_sid(&user_sid, sid_str)) {
+               DEBUG(1,("Could not convert sid %s from string\n", sid_str));
+               return false;
+       }
+
        wbcFreeMemory(sid_str);
        sid_str = NULL;
 
@@ -4496,8 +4088,8 @@ static bool get_user_sids(const char *domain, const char *user, NT_USER_TOKEN *t
 
                wbc_status = wbcGidToSid(gid, &wsid);
                if (!WBC_ERROR_IS_OK(wbc_status)) {
-                       DEBUG(1, ("winbind could not find SID of gid %d: %s\n",
-                                 gid, wbcErrorString(wbc_status)));
+                       DEBUG(1, ("winbind could not find SID of gid %u: %s\n",
+                                 (unsigned int)gid, wbcErrorString(wbc_status)));
                        wbcFreeMemory(groups);
                        return false;
                }
@@ -4611,7 +4203,11 @@ static bool get_user_tokens_from_file(FILE *f,
                        /* We have a SID */
 
                        DOM_SID sid;
-                       string_to_sid(&sid, &line[1]);
+                       if(!string_to_sid(&sid, &line[1])) {
+                               DEBUG(1,("get_user_tokens_from_file: Could "
+                                       "not convert sid %s \n",&line[1]));
+                               return false;
+                       }
 
                        if (token == NULL) {
                                DEBUG(0, ("File does not begin with username"));
@@ -4653,7 +4249,7 @@ static void show_userlist(struct rpc_pipe_client *pipe_hnd,
                        int num_tokens,
                        struct user_token *tokens)
 {
-       int fnum;
+       uint16_t fnum;
        SEC_DESC *share_sd = NULL;
        SEC_DESC *root_sd = NULL;
        struct cli_state *cli = rpc_pipe_np_smb_conn(pipe_hnd);
@@ -4684,13 +4280,12 @@ static void show_userlist(struct rpc_pipe_client *pipe_hnd,
 
        cnum = cli->cnum;
 
-       if (!cli_send_tconX(cli, netname, "A:", "", 0)) {
+       if (!NT_STATUS_IS_OK(cli_tcon_andx(cli, netname, "A:", "", 0))) {
                return;
        }
 
-       fnum = cli_nt_create(cli, "\\", READ_CONTROL_ACCESS);
-
-       if (fnum != -1) {
+       if (!NT_STATUS_IS_OK(cli_ntcreate(cli, "\\", 0, READ_CONTROL_ACCESS, 0,
+                       FILE_SHARE_READ|FILE_SHARE_WRITE, FILE_OPEN, 0x0, 0x0, &fnum))) {
                root_sd = cli_query_secdesc(cli, fnum, mem_ctx);
        }
 
@@ -4698,16 +4293,15 @@ static void show_userlist(struct rpc_pipe_client *pipe_hnd,
                uint32 acc_granted;
 
                if (share_sd != NULL) {
-                       if (!se_access_check(share_sd, &tokens[i].token,
-                                            1, &acc_granted, &status)) {
+                       status = se_access_check(share_sd, &tokens[i].token,
+                                            1, &acc_granted);
+
+                       if (!NT_STATUS_IS_OK(status)) {
                                DEBUG(1, ("Could not check share_sd for "
                                          "user %s\n",
                                          tokens[i].name));
                                continue;
                        }
-
-                       if (!NT_STATUS_IS_OK(status))
-                               continue;
                }
 
                if (root_sd == NULL) {
@@ -4715,20 +4309,17 @@ static void show_userlist(struct rpc_pipe_client *pipe_hnd,
                        continue;
                }
 
-               if (!se_access_check(root_sd, &tokens[i].token,
-                                    1, &acc_granted, &status)) {
+               status = se_access_check(root_sd, &tokens[i].token,
+                                    1, &acc_granted);
+               if (!NT_STATUS_IS_OK(status)) {
                        DEBUG(1, ("Could not check root_sd for user %s\n",
                                  tokens[i].name));
                        continue;
                }
-
-               if (!NT_STATUS_IS_OK(status))
-                       continue;
-
                d_printf(" %s\n", tokens[i].name);
        }
 
-       if (fnum != -1)
+       if (fnum != (uint16_t)-1)
                cli_close(cli, fnum);
        cli_tdis(cli);
        cli->cnum = cnum;
@@ -4759,17 +4350,17 @@ static void collect_share(const char *name, uint32 m,
 }
 
 /**
- * List shares on a remote RPC server, including the security descriptors
+ * List shares on a remote RPC server, including the security descriptors.
  *
  * All parameters are provided by the run_rpc_command function, except for
- * argc, argv which are passes through.
+ * argc, argv which are passed through.
  *
- * @param domain_sid The domain sid acquired from the remote server
+ * @param domain_sid The domain sid acquired from the remote server.
  * @param cli A cli_state connected to the server.
- * @param mem_ctx Talloc context, destoyed on completion of the function.
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param mem_ctx Talloc context, destroyed on completion of the function.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
  * @return Normal NTSTATUS return.
  **/
@@ -4785,7 +4376,6 @@ static NTSTATUS rpc_share_allowedusers_internals(struct net_context *c,
 {
        int ret;
        bool r;
-       ENUM_HND hnd;
        uint32 i;
        FILE *f;
 
@@ -4818,8 +4408,6 @@ static NTSTATUS rpc_share_allowedusers_internals(struct net_context *c,
        for (i=0; i<num_tokens; i++)
                collect_alias_memberships(&tokens[i].token);
 
-       init_enum_hnd(&hnd, 0);
-
        share_list.num_shares = 0;
        share_list.shares = NULL;
 
@@ -4864,19 +4452,19 @@ static int rpc_share_allowedusers(struct net_context *c, int argc,
                return 0;
        }
 
-       result = run_rpc_command(c, NULL, PI_SAMR, 0,
+       result = run_rpc_command(c, NULL, &ndr_table_samr.syntax_id, 0,
                                 rpc_aliaslist_internals,
                                 argc, argv);
        if (result != 0)
                return result;
 
-       result = run_rpc_command(c, NULL, PI_LSARPC, 0,
+       result = run_rpc_command(c, NULL, &ndr_table_lsarpc.syntax_id, 0,
                                 rpc_aliaslist_dump,
                                 argc, argv);
        if (result != 0)
                return result;
 
-       return run_rpc_command(c, NULL, PI_SRVSVC, 0,
+       return run_rpc_command(c, NULL, &ndr_table_srvsvc.syntax_id, 0,
                               rpc_share_allowedusers_internals,
                               argc, argv);
 }
@@ -4919,14 +4507,16 @@ int net_usersidlist_usage(struct net_context *c, int argc, const char **argv)
 
 /**
  * 'net rpc share' entrypoint.
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  **/
 
 int net_rpc_share(struct net_context *c, int argc, const char **argv)
 {
-       struct functable3 func[] = {
+       NET_API_STATUS status;
+
+       struct functable func[] = {
                {
                        "add",
                        rpc_share_add,
@@ -4970,6 +4560,15 @@ int net_rpc_share(struct net_context *c, int argc, const char **argv)
                {NULL, NULL, 0, NULL, NULL}
        };
 
+       status = libnetapi_init(&c->netapi_ctx);
+       if (status != 0) {
+               return -1;
+       }
+       libnetapi_set_username(c->netapi_ctx, c->opt_user_name);
+       libnetapi_set_password(c->netapi_ctx, c->opt_password);
+       if (c->opt_kerberos) {
+               libnetapi_set_use_kerberos(c->netapi_ctx);
+       }
 
        if (argc == 0) {
                if (c->display_usage) {
@@ -4981,12 +4580,10 @@ int net_rpc_share(struct net_context *c, int argc, const char **argv)
                        return 0;
                }
 
-               return run_rpc_command(c, NULL, PI_SRVSVC, 0,
-                                      rpc_share_list_internals,
-                                      argc, argv);
+               return rpc_share_list(c, argc, argv);
        }
 
-       return net_run_function3(c, argc, argv, "net rpc share", func);
+       return net_run_function(c, argc, argv, "net rpc share", func);
 }
 
 static NTSTATUS rpc_sh_share_list(struct net_context *c,
@@ -4995,9 +4592,8 @@ static NTSTATUS rpc_sh_share_list(struct net_context *c,
                                  struct rpc_pipe_client *pipe_hnd,
                                  int argc, const char **argv)
 {
-       return rpc_share_list_internals(c, ctx->domain_sid, ctx->domain_name,
-                                       ctx->cli, pipe_hnd, mem_ctx,
-                                       argc, argv);
+
+       return werror_to_ntstatus(W_ERROR(rpc_share_list(c, argc, argv)));
 }
 
 static NTSTATUS rpc_sh_share_add(struct net_context *c,
@@ -5006,11 +4602,9 @@ static NTSTATUS rpc_sh_share_add(struct net_context *c,
                                 struct rpc_pipe_client *pipe_hnd,
                                 int argc, const char **argv)
 {
-       WERROR result;
-       NTSTATUS status;
+       NET_API_STATUS status;
        uint32_t parm_err = 0;
-       union srvsvc_NetShareInfo info;
-       struct srvsvc_NetShareInfo2 info2;
+       struct SHARE_INFO_2 i2;
 
        if ((argc < 2) || (argc > 3)) {
                d_fprintf(stderr, "usage: %s <share> <path> [comment]\n",
@@ -5018,25 +4612,21 @@ static NTSTATUS rpc_sh_share_add(struct net_context *c,
                return NT_STATUS_INVALID_PARAMETER;
        }
 
-       info2.name              = argv[0];
-       info2.type              = STYPE_DISKTREE;
-       info2.comment           = (argc == 3) ? argv[2] : "";
-       info2.permissions       = 0;
-       info2.max_users         = 0;
-       info2.current_users     = 0;
-       info2.path              = argv[1];
-       info2.password          = NULL;
-
-       info.info2 = &info2;
+       i2.shi2_netname         = argv[0];
+       i2.shi2_type            = STYPE_DISKTREE;
+       i2.shi2_remark          = (argc == 3) ? argv[2] : "";
+       i2.shi2_permissions     = 0;
+       i2.shi2_max_uses        = 0;
+       i2.shi2_current_uses    = 0;
+       i2.shi2_path            = argv[1];
+       i2.shi2_passwd          = NULL;
 
-       status = rpccli_srvsvc_NetShareAdd(pipe_hnd, mem_ctx,
-                                          pipe_hnd->desthost,
-                                          2,
-                                          &info,
-                                          &parm_err,
-                                          &result);
+       status = NetShareAdd(pipe_hnd->desthost,
+                            2,
+                            (uint8_t *)&i2,
+                            &parm_err);
 
-       return status;
+       return werror_to_ntstatus(W_ERROR(status));
 }
 
 static NTSTATUS rpc_sh_share_delete(struct net_context *c,
@@ -5045,21 +4635,12 @@ static NTSTATUS rpc_sh_share_delete(struct net_context *c,
                                    struct rpc_pipe_client *pipe_hnd,
                                    int argc, const char **argv)
 {
-       WERROR result;
-       NTSTATUS status;
-
        if (argc != 1) {
                d_fprintf(stderr, "usage: %s <share>\n", ctx->whoami);
                return NT_STATUS_INVALID_PARAMETER;
        }
 
-       status = rpccli_srvsvc_NetShareDel(pipe_hnd, mem_ctx,
-                                          pipe_hnd->desthost,
-                                          argv[0],
-                                          0,
-                                          &result);
-
-       return status;
+       return werror_to_ntstatus(W_ERROR(NetShareDel(pipe_hnd->desthost, argv[0], 0)));
 }
 
 static NTSTATUS rpc_sh_share_info(struct net_context *c,
@@ -5101,16 +4682,16 @@ struct rpc_sh_cmd *net_rpc_share_cmds(struct net_context *c, TALLOC_CTX *mem_ctx
 {
        static struct rpc_sh_cmd cmds[] = {
 
-       { "list", NULL, PI_SRVSVC, rpc_sh_share_list,
+       { "list", NULL, &ndr_table_srvsvc.syntax_id, rpc_sh_share_list,
          "List available shares" },
 
-       { "add", NULL, PI_SRVSVC, rpc_sh_share_add,
+       { "add", NULL, &ndr_table_srvsvc.syntax_id, rpc_sh_share_add,
          "Add a share" },
 
-       { "delete", NULL, PI_SRVSVC, rpc_sh_share_delete,
+       { "delete", NULL, &ndr_table_srvsvc.syntax_id, rpc_sh_share_delete,
          "Delete a share" },
 
-       { "info", NULL, PI_SRVSVC, rpc_sh_share_info,
+       { "info", NULL, &ndr_table_srvsvc.syntax_id, rpc_sh_share_info,
          "Get information about a share" },
 
        { NULL, NULL, 0, NULL, NULL }
@@ -5127,43 +4708,13 @@ static int rpc_file_usage(struct net_context *c, int argc, const char **argv)
 }
 
 /**
- * Close a file on a remote RPC server
- *
- * All parameters are provided by the run_rpc_command function, except for
- * argc, argv which are passes through.
- *
- * @param c    A net_context structure
- * @param domain_sid The domain sid acquired from the remote server
- * @param cli A cli_state connected to the server.
- * @param mem_ctx Talloc context, destoyed on completion of the function.
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
- *
- * @return Normal NTSTATUS return.
- **/
-static NTSTATUS rpc_file_close_internals(struct net_context *c,
-                                       const DOM_SID *domain_sid,
-                                       const char *domain_name,
-                                       struct cli_state *cli,
-                                       struct rpc_pipe_client *pipe_hnd,
-                                       TALLOC_CTX *mem_ctx,
-                                       int argc,
-                                       const char **argv)
-{
-       return rpccli_srvsvc_NetFileClose(pipe_hnd, mem_ctx,
-                                           pipe_hnd->desthost,
-                                           atoi(argv[0]), NULL);
-}
-
-/**
- * Close a file on a remote RPC server
+ * Close a file on a remote RPC server.
  *
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  **/
 static int rpc_file_close(struct net_context *c, int argc, const char **argv)
 {
@@ -5171,80 +4722,64 @@ static int rpc_file_close(struct net_context *c, int argc, const char **argv)
                return rpc_file_usage(c, argc, argv);
        }
 
-       return run_rpc_command(c, NULL, PI_SRVSVC, 0,
-                              rpc_file_close_internals,
-                              argc, argv);
+       return NetFileClose(c->opt_host, atoi(argv[0]));
 }
 
 /**
  * Formatted print of open file info
  *
- * @param r  struct srvsvc_NetFileInfo3 contents
+ * @param r  struct FILE_INFO_3 contents
  **/
 
-static void display_file_info_3(struct srvsvc_NetFileInfo3 *r)
+static void display_file_info_3(struct FILE_INFO_3 *r)
 {
        d_printf("%-7.1d %-20.20s 0x%-4.2x %-6.1d %s\n",
-                r->fid, r->user, r->permissions, r->num_locks, r->path);
+                r->fi3_id, r->fi3_username, r->fi3_permissions,
+                r->fi3_num_locks, r->fi3_pathname);
 }
 
 /**
- * List open files on a remote RPC server
+ * List files for a user on a remote RPC server.
  *
- * All parameters are provided by the run_rpc_command function, except for
- * argc, argv which are passes through.
- *
- * @param c    A net_context structure
- * @param domain_sid The domain sid acquired from the remote server
- * @param cli A cli_state connected to the server.
- * @param mem_ctx Talloc context, destoyed on completion of the function.
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return Normal NTSTATUS return.
+ * @return A shell status integer (0 for success)..
  **/
 
-static NTSTATUS rpc_file_list_internals(struct net_context *c,
-                                       const DOM_SID *domain_sid,
-                                       const char *domain_name,
-                                       struct cli_state *cli,
-                                       struct rpc_pipe_client *pipe_hnd,
-                                       TALLOC_CTX *mem_ctx,
-                                       int argc,
-                                       const char **argv)
+static int rpc_file_user(struct net_context *c, int argc, const char **argv)
 {
-       struct srvsvc_NetFileInfoCtr info_ctr;
-       struct srvsvc_NetFileCtr3 ctr3;
-       WERROR result;
-       NTSTATUS status;
+       NET_API_STATUS status;
        uint32 preferred_len = 0xffffffff, i;
        const char *username=NULL;
        uint32_t total_entries = 0;
+       uint32_t entries_read = 0;
        uint32_t resume_handle = 0;
+       struct FILE_INFO_3 *i3 = NULL;
+
+       if (c->display_usage) {
+               return rpc_file_usage(c, argc, argv);
+       }
 
        /* if argc > 0, must be user command */
-       if (argc > 0)
+       if (argc > 0) {
                username = smb_xstrdup(argv[0]);
+       }
 
-       ZERO_STRUCT(info_ctr);
-       ZERO_STRUCT(ctr3);
-
-       info_ctr.level = 3;
-       info_ctr.ctr.ctr3 = &ctr3;
-
-       status = rpccli_srvsvc_NetFileEnum(pipe_hnd, mem_ctx,
-                                          pipe_hnd->desthost,
-                                          NULL,
-                                          username,
-                                          &info_ctr,
-                                          preferred_len,
-                                          &total_entries,
-                                          &resume_handle,
-                                          &result);
+       status = NetFileEnum(c->opt_host,
+                            NULL,
+                            username,
+                            3,
+                            (uint8_t **)(void *)&i3,
+                            preferred_len,
+                            &entries_read,
+                            &total_entries,
+                            &resume_handle);
 
-       if (!NT_STATUS_IS_OK(status) || !W_ERROR_IS_OK(result))
+       if (status != 0) {
                goto done;
+       }
 
        /* Display results */
 
@@ -5252,43 +4787,25 @@ static NTSTATUS rpc_file_list_internals(struct net_context *c,
                 "\nEnumerating open files on remote server:\n\n"
                 "\nFileId  Opened by            Perms  Locks  Path"
                 "\n------  ---------            -----  -----  ---- \n");
-       for (i = 0; i < total_entries; i++)
-               display_file_info_3(&info_ctr.ctr.ctr3->array[i]);
- done:
-       return W_ERROR_IS_OK(result) ? NT_STATUS_OK : NT_STATUS_UNSUCCESSFUL;
-}
-
-/**
- * List files for a user on a remote RPC server
- *
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
- *
- * @return A shell status integer (0 for success)
- **/
-
-static int rpc_file_user(struct net_context *c, int argc, const char **argv)
-{
-       if (argc < 1 || c->display_usage) {
-               return rpc_file_usage(c, argc, argv);
+       for (i = 0; i < entries_read; i++) {
+               display_file_info_3(&i3[i]);
        }
-
-       return run_rpc_command(c, NULL, PI_SRVSVC, 0,
-                              rpc_file_list_internals,
-                              argc, argv);
+ done:
+       return status;
 }
 
 /**
  * 'net rpc file' entrypoint.
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  **/
 
 int net_rpc_file(struct net_context *c, int argc, const char **argv)
 {
-       struct functable3 func[] = {
+       NET_API_STATUS status;
+
+       struct functable func[] = {
                {
                        "close",
                        rpc_file_close,
@@ -5318,6 +4835,16 @@ int net_rpc_file(struct net_context *c, int argc, const char **argv)
                {NULL, NULL, 0, NULL, NULL}
        };
 
+       status = libnetapi_init(&c->netapi_ctx);
+       if (status != 0) {
+               return -1;
+       }
+       libnetapi_set_username(c->netapi_ctx, c->opt_user_name);
+       libnetapi_set_password(c->netapi_ctx, c->opt_password);
+       if (c->opt_kerberos) {
+               libnetapi_set_use_kerberos(c->netapi_ctx);
+       }
+
        if (argc == 0) {
                if (c->display_usage) {
                        d_printf("Usage:\n");
@@ -5327,27 +4854,25 @@ int net_rpc_file(struct net_context *c, int argc, const char **argv)
                        return 0;
                }
 
-               return run_rpc_command(c, NULL, PI_SRVSVC, 0,
-                                      rpc_file_list_internals,
-                                      argc, argv);
+               return rpc_file_user(c, argc, argv);
        }
 
-       return net_run_function3(c, argc, argv, "net rpc file", func);
+       return net_run_function(c, argc, argv, "net rpc file", func);
 }
 
 /**
- * ABORT the shutdown of a remote RPC Server over, initshutdown pipe
+ * ABORT the shutdown of a remote RPC Server, over initshutdown pipe.
  *
  * All parameters are provided by the run_rpc_command function, except for
  * argc, argv which are passed through.
  *
- * @param c    A net_context structure
- * @param domain_sid The domain sid aquired from the remote server
+ * @param c    A net_context structure.
+ * @param domain_sid The domain sid acquired from the remote server.
  * @param cli A cli_state connected to the server.
- * @param mem_ctx Talloc context, destoyed on compleation of the function.
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param mem_ctx Talloc context, destroyed on completion of the function.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
  * @return Normal NTSTATUS return.
  **/
@@ -5375,18 +4900,18 @@ static NTSTATUS rpc_shutdown_abort_internals(struct net_context *c,
 }
 
 /**
- * ABORT the shutdown of a remote RPC Server,  over winreg pipe
+ * ABORT the shutdown of a remote RPC Server, over winreg pipe.
  *
  * All parameters are provided by the run_rpc_command function, except for
  * argc, argv which are passed through.
  *
- * @param c    A net_context structure
- * @param domain_sid The domain sid aquired from the remote server
+ * @param c    A net_context structure.
+ * @param domain_sid The domain sid acquired from the remote server.
  * @param cli A cli_state connected to the server.
- * @param mem_ctx Talloc context, destoyed on compleation of the function.
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param mem_ctx Talloc context, destroyed on completion of the function.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
  * @return Normal NTSTATUS return.
  **/
@@ -5414,13 +4939,13 @@ static NTSTATUS rpc_reg_shutdown_abort_internals(struct net_context *c,
 }
 
 /**
- * ABORT the Shut down of a remote RPC server
+ * ABORT the shutdown of a remote RPC server.
  *
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  **/
 
 static int rpc_shutdown_abort(struct net_context *c, int argc,
@@ -5435,7 +4960,7 @@ static int rpc_shutdown_abort(struct net_context *c, int argc,
                return 0;
        }
 
-       rc = run_rpc_command(c, NULL, PI_INITSHUTDOWN, 0,
+       rc = run_rpc_command(c, NULL, &ndr_table_initshutdown.syntax_id, 0,
                             rpc_shutdown_abort_internals, argc, argv);
 
        if (rc == 0)
@@ -5443,24 +4968,24 @@ static int rpc_shutdown_abort(struct net_context *c, int argc,
 
        DEBUG(1, ("initshutdown pipe didn't work, trying winreg pipe\n"));
 
-       return run_rpc_command(c, NULL, PI_WINREG, 0,
+       return run_rpc_command(c, NULL, &ndr_table_winreg.syntax_id, 0,
                               rpc_reg_shutdown_abort_internals,
                               argc, argv);
 }
 
 /**
- * Shut down a remote RPC Server via initshutdown pipe
+ * Shut down a remote RPC Server via initshutdown pipe.
  *
  * All parameters are provided by the run_rpc_command function, except for
- * argc, argv which are passes through.
+ * argc, argv which are passed through.
  *
- * @param c    A net_context structure
- * @param domain_sid The domain sid aquired from the remote server
+ * @param c    A net_context structure.
+ * @param domain_sid The domain sid acquired from the remote server.
  * @param cli A cli_state connected to the server.
- * @param mem_ctx Talloc context, destoyed on compleation of the function.
- * @param argc  Standard main() style argc
- * @param argc  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param mem_ctx Talloc context, destroyed on completion of the function.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
  * @return Normal NTSTATUS return.
  **/
@@ -5477,8 +5002,7 @@ NTSTATUS rpc_init_shutdown_internals(struct net_context *c,
        NTSTATUS result = NT_STATUS_UNSUCCESSFUL;
         const char *msg = "This machine will be shutdown shortly";
        uint32 timeout = 20;
-       struct initshutdown_String msg_string;
-       struct initshutdown_String_sub s;
+       struct lsa_StringLarge msg_string;
 
        if (c->opt_comment) {
                msg = c->opt_comment;
@@ -5487,8 +5011,7 @@ NTSTATUS rpc_init_shutdown_internals(struct net_context *c,
                timeout = c->opt_timeout;
        }
 
-       s.name = msg;
-       msg_string.name = &s;
+       msg_string.string = msg;
 
        /* create an entry */
        result = rpccli_initshutdown_Init(pipe_hnd, mem_ctx, NULL,
@@ -5505,18 +5028,18 @@ NTSTATUS rpc_init_shutdown_internals(struct net_context *c,
 }
 
 /**
- * Shut down a remote RPC Server via winreg pipe
+ * Shut down a remote RPC Server via winreg pipe.
  *
  * All parameters are provided by the run_rpc_command function, except for
- * argc, argv which are passes through.
+ * argc, argv which are passed through.
  *
- * @param c    A net_context structure
- * @param domain_sid The domain sid aquired from the remote server
+ * @param c    A net_context structure.
+ * @param domain_sid The domain sid acquired from the remote server.
  * @param cli A cli_state connected to the server.
- * @param mem_ctx Talloc context, destoyed on compleation of the function.
- * @param argc  Standard main() style argc
- * @param argc  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param mem_ctx Talloc context, destroyed on completion of the function.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
  * @return Normal NTSTATUS return.
  **/
@@ -5532,16 +5055,14 @@ NTSTATUS rpc_reg_shutdown_internals(struct net_context *c,
 {
         const char *msg = "This machine will be shutdown shortly";
        uint32 timeout = 20;
-       struct initshutdown_String msg_string;
-       struct initshutdown_String_sub s;
+       struct lsa_StringLarge msg_string;
        NTSTATUS result;
        WERROR werr;
 
        if (c->opt_comment) {
                msg = c->opt_comment;
        }
-       s.name = msg;
-       msg_string.name = &s;
+       msg_string.string = msg;
 
        if (c->opt_timeout) {
                timeout = c->opt_timeout;
@@ -5559,20 +5080,20 @@ NTSTATUS rpc_reg_shutdown_internals(struct net_context *c,
                if ( W_ERROR_EQUAL(werr, WERR_MACHINE_LOCKED) )
                        d_fprintf(stderr, "\nMachine locked, use -f switch to force\n");
                else
-                       d_fprintf(stderr, "\nresult was: %s\n", dos_errstr(werr));
+                       d_fprintf(stderr, "\nresult was: %s\n", win_errstr(werr));
        }
 
        return result;
 }
 
 /**
- * Shut down a remote RPC server
+ * Shut down a remote RPC server.
  *
- * @param argc  Standard main() style argc
- * @param argc  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  **/
 
 static int rpc_shutdown(struct net_context *c, int argc, const char **argv)
@@ -5586,12 +5107,12 @@ static int rpc_shutdown(struct net_context *c, int argc, const char **argv)
                return 0;
        }
 
-       rc = run_rpc_command(c, NULL, PI_INITSHUTDOWN, 0,
+       rc = run_rpc_command(c, NULL, &ndr_table_initshutdown.syntax_id, 0,
                             rpc_init_shutdown_internals, argc, argv);
 
        if (rc) {
                DEBUG(1, ("initshutdown pipe failed, trying winreg pipe\n"));
-               rc = run_rpc_command(c, NULL, PI_WINREG, 0,
+               rc = run_rpc_command(c, NULL, &ndr_table_winreg.syntax_id, 0,
                                     rpc_reg_shutdown_internals, argc, argv);
        }
 
@@ -5607,15 +5128,15 @@ static int rpc_shutdown(struct net_context *c, int argc, const char **argv)
  * All parameters (except for argc and argv) are passed by run_rpc_command
  * function.
  *
- * @param c    A net_context structure
- * @param domain_sid The domain sid acquired from the server
+ * @param c    A net_context structure.
+ * @param domain_sid The domain sid acquired from the server.
  * @param cli A cli_state connected to the server.
- * @param mem_ctx Talloc context, destoyed on completion of the function.
- * @param argc  Standard main() style argc
- * @param argc  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param mem_ctx Talloc context, destroyed on completion of the function.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return normal NTSTATUS return code
+ * @return normal NTSTATUS return code.
  */
 
 static NTSTATUS rpc_trustdom_add_internals(struct net_context *c,
@@ -5627,7 +5148,7 @@ static NTSTATUS rpc_trustdom_add_internals(struct net_context *c,
                                                int argc,
                                                const char **argv)
 {
-       POLICY_HND connect_pol, domain_pol, user_pol;
+       struct policy_handle connect_pol, domain_pol, user_pol;
        NTSTATUS result = NT_STATUS_UNSUCCESSFUL;
        char *acct_name;
        struct lsa_String lsa_acct_name;
@@ -5636,9 +5157,11 @@ static NTSTATUS rpc_trustdom_add_internals(struct net_context *c,
        uint32 user_rid;
        uint32_t access_granted = 0;
        union samr_UserInfo info;
+       unsigned int orig_timeout;
 
        if (argc != 2) {
-               d_printf("Usage: net rpc trustdom add <domain_name> <pw>\n");
+               d_printf("Usage: net rpc trustdom add <domain_name> "
+                        "<trust password>\n");
                return NT_STATUS_INVALID_PARAMETER;
        }
 
@@ -5673,6 +5196,11 @@ static NTSTATUS rpc_trustdom_add_internals(struct net_context *c,
                goto done;
        }
 
+        /* This call can take a long time - allow the server to time out.
+        * 35 seconds should do it. */
+
+        orig_timeout = rpccli_set_timeout(pipe_hnd, 35000);
+
        /* Create trusting domain's account */
        acb_info = ACB_NORMAL;
        acct_flags = SEC_GENERIC_READ | SEC_GENERIC_WRITE | SEC_GENERIC_EXECUTE |
@@ -5689,43 +5217,29 @@ static NTSTATUS rpc_trustdom_add_internals(struct net_context *c,
                                         &user_pol,
                                         &access_granted,
                                         &user_rid);
+
+       /* And restore our original timeout. */
+       rpccli_set_timeout(pipe_hnd, orig_timeout);
+
        if (!NT_STATUS_IS_OK(result)) {
+               d_printf("net rpc trustdom add: create user %s failed %s\n",
+                       acct_name, nt_errstr(result));
                goto done;
        }
 
        {
-               NTTIME notime;
-               struct samr_LogonHours hours;
-               struct lsa_BinaryString parameters;
-               const int units_per_week = 168;
-               uchar pwbuf[516];
+               struct samr_CryptPassword crypt_pwd;
 
-               encode_pw_buffer(pwbuf, argv[1], STR_UNICODE);
+               ZERO_STRUCT(info.info23);
 
-               ZERO_STRUCT(notime);
-               ZERO_STRUCT(hours);
-               ZERO_STRUCT(parameters);
+               init_samr_CryptPassword(argv[1],
+                                       &cli->user_session_key,
+                                       &crypt_pwd);
 
-               hours.bits = talloc_array(mem_ctx, uint8_t, units_per_week);
-               if (!hours.bits) {
-                       result = NT_STATUS_NO_MEMORY;
-                       goto done;
-               }
-               hours.units_per_week = units_per_week;
-               memset(hours.bits, 0xFF, units_per_week);
-
-               init_samr_user_info23(&info.info23,
-                                     notime, notime, notime,
-                                     notime, notime, notime,
-                                     NULL, NULL, NULL, NULL, NULL,
-                                     NULL, NULL, NULL, NULL, &parameters,
-                                     0, 0, ACB_DOMTRUST, SAMR_FIELD_ACCT_FLAGS,
-                                     hours,
-                                     0, 0, 0, 0, 0, 0, 0,
-                                     pwbuf, 24);
-
-               SamOEMhashBlob(info.info23.password.data, 516,
-                              &cli->user_session_key);
+               info.info23.info.fields_present = SAMR_FIELD_ACCT_FLAGS |
+                                                 SAMR_FIELD_NT_PASSWORD_PRESENT;
+               info.info23.info.acct_flags = ACB_DOMTRUST;
+               info.info23.password = crypt_pwd;
 
                result = rpccli_samr_SetUserInfo2(pipe_hnd, mem_ctx,
                                                  &user_pol,
@@ -5747,20 +5261,20 @@ static NTSTATUS rpc_trustdom_add_internals(struct net_context *c,
 /**
  * Create interdomain trust account for a remote domain.
  *
- * @param argc standard argc
- * @param argv standard argv without initial components
+ * @param argc Standard argc.
+ * @param argv Standard argv without initial components.
  *
- * @return Integer status (0 means success)
+ * @return Integer status (0 means success).
  **/
 
 static int rpc_trustdom_add(struct net_context *c, int argc, const char **argv)
 {
        if (argc > 0 && !c->display_usage) {
-               return run_rpc_command(c, NULL, PI_SAMR, 0,
+               return run_rpc_command(c, NULL, &ndr_table_samr.syntax_id, 0,
                                       rpc_trustdom_add_internals, argc, argv);
        } else {
                d_printf("Usage:\n"
-                        "net rpc trustdom add <domain>\n");
+                       "net rpc trustdom add <domain_name> <trust password>\n");
                return -1;
        }
 }
@@ -5771,15 +5285,15 @@ static int rpc_trustdom_add(struct net_context *c, int argc, const char **argv)
  * All parameters (except for argc and argv) are passed by run_rpc_command
  * function.
  *
- * @param c    A net_context structure
- * @param domain_sid The domain sid acquired from the server
+ * @param c    A net_context structure.
+ * @param domain_sid The domain sid acquired from the server.
  * @param cli A cli_state connected to the server.
- * @param mem_ctx Talloc context, destoyed on completion of the function.
- * @param argc  Standard main() style argc
- * @param argc  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param mem_ctx Talloc context, destroyed on completion of the function.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return normal NTSTATUS return code
+ * @return normal NTSTATUS return code.
  */
 
 static NTSTATUS rpc_trustdom_del_internals(struct net_context *c,
@@ -5791,7 +5305,7 @@ static NTSTATUS rpc_trustdom_del_internals(struct net_context *c,
                                        int argc,
                                        const char **argv)
 {
-       POLICY_HND connect_pol, domain_pol, user_pol;
+       struct policy_handle connect_pol, domain_pol, user_pol;
        NTSTATUS result = NT_STATUS_UNSUCCESSFUL;
        char *acct_name;
        DOM_SID trust_acct_sid;
@@ -5842,6 +5356,8 @@ static NTSTATUS rpc_trustdom_del_internals(struct net_context *c,
                                         &name_types);
 
        if (!NT_STATUS_IS_OK(result)) {
+               d_printf("net rpc trustdom del: LookupNames on user %s failed %s\n",
+                       acct_name, nt_errstr(result) );
                goto done;
        }
 
@@ -5852,6 +5368,8 @@ static NTSTATUS rpc_trustdom_del_internals(struct net_context *c,
                                      &user_pol);
 
        if (!NT_STATUS_IS_OK(result)) {
+               d_printf("net rpc trustdom del: OpenUser on user %s failed %s\n",
+                       acct_name, nt_errstr(result) );
                goto done;
        }
 
@@ -5867,6 +5385,8 @@ static NTSTATUS rpc_trustdom_del_internals(struct net_context *c,
                                                           &user_pol,
                                                           &trust_acct_sid);
        if (!NT_STATUS_IS_OK(result)) {
+               d_printf("net rpc trustdom del: RemoveMemberFromForeignDomain on user %s failed %s\n",
+                       acct_name, nt_errstr(result) );
                goto done;
        }
 
@@ -5876,13 +5396,15 @@ static NTSTATUS rpc_trustdom_del_internals(struct net_context *c,
                                        &user_pol);
 
        if (!NT_STATUS_IS_OK(result)) {
+               d_printf("net rpc trustdom del: DeleteUser on user %s failed %s\n",
+                       acct_name, nt_errstr(result) );
                goto done;
        }
 
        if (!NT_STATUS_IS_OK(result)) {
-         DEBUG(0,("Could not set trust account password: %s\n",
-                  nt_errstr(result)));
-         goto done;
+               d_printf("Could not set trust account password: %s\n",
+                  nt_errstr(result));
+               goto done;
        }
 
  done:
@@ -5892,16 +5414,16 @@ static NTSTATUS rpc_trustdom_del_internals(struct net_context *c,
 /**
  * Delete interdomain trust account for a remote domain.
  *
- * @param argc standard argc
- * @param argv standard argv without initial components
+ * @param argc Standard argc.
+ * @param argv Standard argv without initial components.
  *
- * @return Integer status (0 means success)
+ * @return Integer status (0 means success).
  **/
 
 static int rpc_trustdom_del(struct net_context *c, int argc, const char **argv)
 {
        if (argc > 0 && !c->display_usage) {
-               return run_rpc_command(c, NULL, PI_SAMR, 0,
+               return run_rpc_command(c, NULL, &ndr_table_samr.syntax_id, 0,
                                       rpc_trustdom_del_internals, argc, argv);
        } else {
                d_printf("Usage:\n"
@@ -5932,8 +5454,9 @@ static NTSTATUS rpc_trustdom_get_pdc(struct net_context *c,
 
        /* Try netr_GetDcName */
 
-       netr = cli_rpc_pipe_open_noauth(cli, PI_NETLOGON, &status);
-       if (!netr) {
+       status = cli_rpc_pipe_open_noauth(cli, &ndr_table_netlogon.syntax_id,
+                                         &netr);
+       if (!NT_STATUS_IS_OK(status)) {
                return status;
        }
 
@@ -5958,11 +5481,11 @@ static NTSTATUS rpc_trustdom_get_pdc(struct net_context *c,
  * Establish trust relationship to a trusting domain.
  * Interdomain account must already be created on remote PDC.
  *
- * @param c    A net_context structure
- * @param argc standard argc
- * @param argv standard argv without initial components
+ * @param c    A net_context structure.
+ * @param argc Standard argc.
+ * @param argv Standard argv without initial components.
  *
- * @return Integer status (0 means success)
+ * @return Integer status (0 means success).
  **/
 
 static int rpc_trustdom_establish(struct net_context *c, int argc,
@@ -5971,7 +5494,7 @@ static int rpc_trustdom_establish(struct net_context *c, int argc,
        struct cli_state *cli = NULL;
        struct sockaddr_storage server_ss;
        struct rpc_pipe_client *pipe_hnd = NULL;
-       POLICY_HND connect_hnd;
+       struct policy_handle connect_hnd;
        TALLOC_CTX *mem_ctx;
        NTSTATUS nt_status;
        DOM_SID *domain_sid;
@@ -5995,7 +5518,9 @@ static int rpc_trustdom_establish(struct net_context *c, int argc,
        strupper_m(domain_name);
 
        /* account name used at first is our domain's name with '$' */
-       asprintf(&acct_name, "%s$", lp_workgroup());
+       if (asprintf(&acct_name, "%s$", lp_workgroup()) == -1) {
+               return -1;
+       }
        strupper_m(acct_name);
 
        /*
@@ -6061,15 +5586,16 @@ static int rpc_trustdom_establish(struct net_context *c, int argc,
         * Call LsaOpenPolicy and LsaQueryInfo
         */
 
-       pipe_hnd = cli_rpc_pipe_open_noauth(cli, PI_LSARPC, &nt_status);
-       if (!pipe_hnd) {
+       nt_status = cli_rpc_pipe_open_noauth(cli, &ndr_table_lsarpc.syntax_id,
+                                            &pipe_hnd);
+       if (!NT_STATUS_IS_OK(nt_status)) {
                DEBUG(0, ("Could not initialise lsa pipe. Error was %s\n", nt_errstr(nt_status) ));
                cli_shutdown(cli);
                talloc_destroy(mem_ctx);
                return -1;
        }
 
-       nt_status = rpccli_lsa_open_policy2(pipe_hnd, mem_ctx, true, SEC_RIGHTS_QUERY_VALUE,
+       nt_status = rpccli_lsa_open_policy2(pipe_hnd, mem_ctx, true, KEY_QUERY_VALUE,
                                         &connect_hnd);
        if (NT_STATUS_IS_ERR(nt_status)) {
                DEBUG(0, ("Couldn't open policy handle. Error was %s\n",
@@ -6131,13 +5657,13 @@ static int rpc_trustdom_establish(struct net_context *c, int argc,
 }
 
 /**
- * Revoke trust relationship to the remote domain
+ * Revoke trust relationship to the remote domain.
  *
- * @param c    A net_context structure
- * @param argc standard argc
- * @param argv standard argv without initial components
+ * @param c    A net_context structure.
+ * @param argc Standard argc.
+ * @param argv Standard argv without initial components.
  *
- * @return Integer status (0 means success)
+ * @return Integer status (0 means success).
  **/
 
 static int rpc_trustdom_revoke(struct net_context *c, int argc,
@@ -6204,14 +5730,15 @@ static void print_trusted_domain(DOM_SID *dom_sid, const char *trusted_dom_name)
 
 static NTSTATUS vampire_trusted_domain(struct rpc_pipe_client *pipe_hnd,
                                      TALLOC_CTX *mem_ctx,
-                                     POLICY_HND *pol,
+                                     struct policy_handle *pol,
                                      DOM_SID dom_sid,
                                      const char *trusted_dom_name)
 {
        NTSTATUS nt_status;
        union lsa_TrustedDomainInfo *info = NULL;
        char *cleartextpwd = NULL;
-       uint8_t nt_hash[16];
+       uint8_t session_key[16];
+       DATA_BLOB session_key_blob;
        DATA_BLOB data;
 
        nt_status = rpccli_lsa_QueryTrustedDomainInfoBySid(pipe_hnd, mem_ctx,
@@ -6228,12 +5755,13 @@ static NTSTATUS vampire_trusted_domain(struct rpc_pipe_client *pipe_hnd,
        data = data_blob(info->password.password->data,
                         info->password.password->length);
 
-       if (!rpccli_get_pwd_hash(pipe_hnd, nt_hash)) {
+       if (!rpccli_get_pwd_hash(pipe_hnd, session_key)) {
                DEBUG(0, ("Could not retrieve password hash\n"));
                goto done;
        }
 
-       cleartextpwd = decrypt_trustdom_secret(nt_hash, &data);
+       session_key_blob = data_blob_const(session_key, sizeof(session_key));
+       cleartextpwd = sess_decrypt_string(mem_ctx, &data, &session_key_blob);
 
        if (cleartextpwd == NULL) {
                DEBUG(0,("retrieved NULL password\n"));
@@ -6270,7 +5798,7 @@ static int rpc_trustdom_vampire(struct net_context *c, int argc,
        NTSTATUS nt_status;
        const char *domain_name = NULL;
        DOM_SID *queried_dom_sid;
-       POLICY_HND connect_hnd;
+       struct policy_handle connect_hnd;
        union lsa_PolicyInformation *info = NULL;
 
        /* trusted domains listing variables */
@@ -6315,8 +5843,9 @@ static int rpc_trustdom_vampire(struct net_context *c, int argc,
                return -1;
        };
 
-       pipe_hnd = cli_rpc_pipe_open_noauth(cli, PI_LSARPC, &nt_status);
-       if (!pipe_hnd) {
+       nt_status = cli_rpc_pipe_open_noauth(cli, &ndr_table_lsarpc.syntax_id,
+                                            &pipe_hnd);
+       if (!NT_STATUS_IS_OK(nt_status)) {
                DEBUG(0, ("Could not initialise lsa pipe. Error was %s\n",
                        nt_errstr(nt_status) ));
                cli_shutdown(cli);
@@ -6324,7 +5853,7 @@ static int rpc_trustdom_vampire(struct net_context *c, int argc,
                return -1;
        };
 
-       nt_status = rpccli_lsa_open_policy2(pipe_hnd, mem_ctx, false, SEC_RIGHTS_QUERY_VALUE,
+       nt_status = rpccli_lsa_open_policy2(pipe_hnd, mem_ctx, false, KEY_QUERY_VALUE,
                                        &connect_hnd);
        if (NT_STATUS_IS_ERR(nt_status)) {
                DEBUG(0, ("Couldn't open policy handle. Error was %s\n",
@@ -6422,7 +5951,7 @@ static int rpc_trustdom_list(struct net_context *c, int argc, const char **argv)
        DOM_SID *queried_dom_sid;
        fstring padding;
        int ascii_dom_name_len;
-       POLICY_HND connect_hnd;
+       struct policy_handle connect_hnd;
        union lsa_PolicyInformation *info = NULL;
 
        /* trusted domains listing variables */
@@ -6432,7 +5961,7 @@ static int rpc_trustdom_list(struct net_context *c, int argc, const char **argv)
        fstring pdc_name;
 
        /* trusting domains listing variables */
-       POLICY_HND domain_hnd;
+       struct policy_handle domain_hnd;
        struct samr_SamArray *trusts = NULL;
 
        if (c->display_usage) {
@@ -6471,8 +6000,9 @@ static int rpc_trustdom_list(struct net_context *c, int argc, const char **argv)
                return -1;
        };
 
-       pipe_hnd = cli_rpc_pipe_open_noauth(cli, PI_LSARPC, &nt_status);
-       if (!pipe_hnd) {
+       nt_status = cli_rpc_pipe_open_noauth(cli, &ndr_table_lsarpc.syntax_id,
+                                            &pipe_hnd);
+       if (!NT_STATUS_IS_OK(nt_status)) {
                DEBUG(0, ("Could not initialise lsa pipe. Error was %s\n",
                        nt_errstr(nt_status) ));
                cli_shutdown(cli);
@@ -6480,7 +6010,7 @@ static int rpc_trustdom_list(struct net_context *c, int argc, const char **argv)
                return -1;
        };
 
-       nt_status = rpccli_lsa_open_policy2(pipe_hnd, mem_ctx, false, SEC_RIGHTS_QUERY_VALUE,
+       nt_status = rpccli_lsa_open_policy2(pipe_hnd, mem_ctx, false, KEY_QUERY_VALUE,
                                        &connect_hnd);
        if (NT_STATUS_IS_ERR(nt_status)) {
                DEBUG(0, ("Couldn't open policy handle. Error was %s\n",
@@ -6561,8 +6091,9 @@ static int rpc_trustdom_list(struct net_context *c, int argc, const char **argv)
        /*
         * Open \PIPE\samr and get needed policy handles
         */
-       pipe_hnd = cli_rpc_pipe_open_noauth(cli, PI_SAMR, &nt_status);
-       if (!pipe_hnd) {
+       nt_status = cli_rpc_pipe_open_noauth(cli, &ndr_table_samr.syntax_id,
+                                            &pipe_hnd);
+       if (!NT_STATUS_IS_OK(nt_status)) {
                DEBUG(0, ("Could not initialise samr pipe. Error was %s\n", nt_errstr(nt_status)));
                cli_shutdown(cli);
                talloc_destroy(mem_ctx);
@@ -6572,7 +6103,7 @@ static int rpc_trustdom_list(struct net_context *c, int argc, const char **argv)
        /* SamrConnect2 */
        nt_status = rpccli_samr_Connect2(pipe_hnd, mem_ctx,
                                         pipe_hnd->desthost,
-                                        SA_RIGHT_SAM_OPEN_DOMAIN,
+                                        SAMR_ACCESS_LOOKUP_DOMAIN,
                                         &connect_hnd);
        if (!NT_STATUS_IS_OK(nt_status)) {
                DEBUG(0, ("Couldn't open SAMR policy handle. Error was %s\n",
@@ -6586,7 +6117,7 @@ static int rpc_trustdom_list(struct net_context *c, int argc, const char **argv)
           able to enumerate accounts*/
        nt_status = rpccli_samr_OpenDomain(pipe_hnd, mem_ctx,
                                           &connect_hnd,
-                                          SA_RIGHT_DOMAIN_ENUM_ACCOUNTS,
+                                          SAMR_DOMAIN_ACCESS_ENUM_ACCOUNTS,
                                           queried_dom_sid,
                                           &domain_hnd);
        if (!NT_STATUS_IS_OK(nt_status)) {
@@ -6652,9 +6183,11 @@ static int rpc_trustdom_list(struct net_context *c, int argc, const char **argv)
                                        &remote_cli);
                        if (NT_STATUS_IS_OK(nt_status)) {
                                /* query for domain's sid */
-                               if (run_rpc_command(c, remote_cli, PI_LSARPC, 0,
-                                                   rpc_query_domain_sid, argc,
-                                                   argv))
+                               if (run_rpc_command(
+                                           c, remote_cli,
+                                           &ndr_table_lsarpc.syntax_id, 0,
+                                           rpc_query_domain_sid, argc,
+                                           argv))
                                        d_fprintf(stderr, "couldn't get domain's sid\n");
 
                                cli_shutdown(remote_cli);
@@ -6689,17 +6222,17 @@ static int rpc_trustdom_list(struct net_context *c, int argc, const char **argv)
 }
 
 /**
- * Entrypoint for 'net rpc trustdom' code
+ * Entrypoint for 'net rpc trustdom' code.
  *
- * @param argc standard argc
- * @param argv standard argv without initial components
+ * @param argc Standard argc.
+ * @param argv Standard argv without initial components.
  *
- * @return Integer status (0 means success)
+ * @return Integer status (0 means success).
  */
 
 static int rpc_trustdom(struct net_context *c, int argc, const char **argv)
 {
-       struct functable3 func[] = {
+       struct functable func[] = {
                {
                        "add",
                        rpc_trustdom_add,
@@ -6751,7 +6284,7 @@ static int rpc_trustdom(struct net_context *c, int argc, const char **argv)
                {NULL, NULL, 0, NULL, NULL}
        };
 
-       return net_run_function3(c, argc, argv, "net rpc trustdom", func);
+       return net_run_function(c, argc, argv, "net rpc trustdom", func);
 }
 
 /**
@@ -6782,7 +6315,8 @@ bool net_rpc_check(struct net_context *c, unsigned flags)
        if (!attempt_netbios_session_request(&cli, global_myname(),
                                             server_name, &server_ss))
                goto done;
-       if (!cli_negprot(cli))
+       status = cli_negprot(cli);
+       if (!NT_STATUS_IS_OK(status))
                goto done;
        if (cli->protocol < PROTOCOL_NT1)
                goto done;
@@ -6802,36 +6336,73 @@ static int rpc_samdump(struct net_context *c, int argc, const char **argv) {
                return 0;
        }
 
-       return run_rpc_command(c, NULL, PI_NETLOGON, NET_FLAGS_ANONYMOUS,
+       return run_rpc_command(c, NULL, &ndr_table_netlogon.syntax_id,
+                              NET_FLAGS_ANONYMOUS,
                               rpc_samdump_internals, argc, argv);
 }
 
 /* syncronise sam database via samsync rpc calls */
-static int rpc_vampire(struct net_context *c, int argc, const char **argv) {
-       if (c->display_usage) {
-               d_printf("Usage:\n"
-                        "net rpc vampire\n"
-                        "    Vampire remote SAM database\n");
-               return 0;
+static int rpc_vampire(struct net_context *c, int argc, const char **argv)
+{
+       struct functable func[] = {
+               {
+                       "ldif",
+                       rpc_vampire_ldif,
+                       NET_TRANSPORT_RPC,
+                       "Dump remote SAM database to ldif",
+                       "net rpc vampire ldif\n"
+                       "    Dump remote SAM database to LDIF file or stdout"
+               },
+               {
+                       "keytab",
+                       rpc_vampire_keytab,
+                       NET_TRANSPORT_RPC,
+                       "Dump remote SAM database to Kerberos Keytab",
+                       "net rpc vampire keytab\n"
+                       "    Dump remote SAM database to Kerberos keytab file"
+               },
+               {
+                       "passdb",
+                       rpc_vampire_passdb,
+                       NET_TRANSPORT_RPC,
+                       "Dump remote SAM database to passdb",
+                       "net rpc vampire passdb\n"
+                       "    Dump remote SAM database to passdb"
+               },
+
+               {NULL, NULL, 0, NULL, NULL}
+       };
+
+       if (argc == 0) {
+               if (c->display_usage) {
+                       d_printf("Usage:\n"
+                                "net rpc vampire\n"
+                                "    Vampire remote SAM database\n");
+                       return 0;
+               }
+
+               return run_rpc_command(c, NULL, &ndr_table_netlogon.syntax_id,
+                                      NET_FLAGS_ANONYMOUS,
+                                      rpc_vampire_internals,
+                                      argc, argv);
        }
 
-       return run_rpc_command(c, NULL, PI_NETLOGON, NET_FLAGS_ANONYMOUS,
-                              rpc_vampire_internals,  argc, argv);
+       return net_run_function(c, argc, argv, "net rpc vampire", func);
 }
 
 /**
- * Migrate everything from a print-server
+ * Migrate everything from a print server.
  *
- * @param c    A net_context structure
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param c    A net_context structure.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  *
  * The order is important !
- * To successfully add drivers the print-queues have to exist !
- * Applying ACLs should be the last step, because you're easily locked out
+ * To successfully add drivers the print queues have to exist !
+ * Applying ACLs should be the last step, because you're easily locked out.
  *
  **/
 static int rpc_printer_migrate_all(struct net_context *c, int argc,
@@ -6851,44 +6422,44 @@ static int rpc_printer_migrate_all(struct net_context *c, int argc,
                return -1;
        }
 
-       ret = run_rpc_command(c, NULL, PI_SPOOLSS, 0,
+       ret = run_rpc_command(c, NULL, &ndr_table_spoolss.syntax_id, 0,
                              rpc_printer_migrate_printers_internals, argc,
                              argv);
        if (ret)
                return ret;
 
-       ret = run_rpc_command(c, NULL, PI_SPOOLSS, 0,
+       ret = run_rpc_command(c, NULL, &ndr_table_spoolss.syntax_id, 0,
                              rpc_printer_migrate_drivers_internals, argc,
                              argv);
        if (ret)
                return ret;
 
-       ret = run_rpc_command(c, NULL, PI_SPOOLSS, 0,
+       ret = run_rpc_command(c, NULL, &ndr_table_spoolss.syntax_id, 0,
                              rpc_printer_migrate_forms_internals, argc, argv);
        if (ret)
                return ret;
 
-       ret = run_rpc_command(c, NULL, PI_SPOOLSS, 0,
+       ret = run_rpc_command(c, NULL, &ndr_table_spoolss.syntax_id, 0,
                              rpc_printer_migrate_settings_internals, argc,
                              argv);
        if (ret)
                return ret;
 
-       return run_rpc_command(c, NULL, PI_SPOOLSS, 0,
+       return run_rpc_command(c, NULL, &ndr_table_spoolss.syntax_id, 0,
                               rpc_printer_migrate_security_internals, argc,
                               argv);
 
 }
 
 /**
- * Migrate print-drivers from a print-server
+ * Migrate print drivers from a print server.
  *
- * @param c    A net_context structure
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param c    A net_context structure.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  **/
 static int rpc_printer_migrate_drivers(struct net_context *c, int argc,
                                       const char **argv)
@@ -6905,20 +6476,20 @@ static int rpc_printer_migrate_drivers(struct net_context *c, int argc,
                return -1;
        }
 
-       return run_rpc_command(c, NULL, PI_SPOOLSS, 0,
+       return run_rpc_command(c, NULL, &ndr_table_spoolss.syntax_id, 0,
                               rpc_printer_migrate_drivers_internals,
                               argc, argv);
 }
 
 /**
- * Migrate print-forms from a print-server
+ * Migrate print-forms from a print-server.
  *
- * @param c    A net_context structure
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param c    A net_context structure.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  **/
 static int rpc_printer_migrate_forms(struct net_context *c, int argc,
                                     const char **argv)
@@ -6935,20 +6506,20 @@ static int rpc_printer_migrate_forms(struct net_context *c, int argc,
                return -1;
        }
 
-       return run_rpc_command(c, NULL, PI_SPOOLSS, 0,
+       return run_rpc_command(c, NULL, &ndr_table_spoolss.syntax_id, 0,
                               rpc_printer_migrate_forms_internals,
                               argc, argv);
 }
 
 /**
- * Migrate printers from a print-server
+ * Migrate printers from a print-server.
  *
- * @param c    A net_context structure
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param c    A net_context structure.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  **/
 static int rpc_printer_migrate_printers(struct net_context *c, int argc,
                                        const char **argv)
@@ -6965,7 +6536,7 @@ static int rpc_printer_migrate_printers(struct net_context *c, int argc,
                return -1;
        }
 
-       return run_rpc_command(c, NULL, PI_SPOOLSS, 0,
+       return run_rpc_command(c, NULL, &ndr_table_spoolss.syntax_id, 0,
                               rpc_printer_migrate_printers_internals,
                               argc, argv);
 }
@@ -6973,12 +6544,12 @@ static int rpc_printer_migrate_printers(struct net_context *c, int argc,
 /**
  * Migrate printer-ACLs from a print-server
  *
- * @param c    A net_context structure
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param c    A net_context structure.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  **/
 static int rpc_printer_migrate_security(struct net_context *c, int argc,
                                        const char **argv)
@@ -6995,20 +6566,20 @@ static int rpc_printer_migrate_security(struct net_context *c, int argc,
                return -1;
        }
 
-       return run_rpc_command(c, NULL, PI_SPOOLSS, 0,
+       return run_rpc_command(c, NULL, &ndr_table_spoolss.syntax_id, 0,
                               rpc_printer_migrate_security_internals,
                               argc, argv);
 }
 
 /**
- * Migrate printer-settings from a print-server
+ * Migrate printer-settings from a print-server.
  *
- * @param c    A net_context structure
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param c    A net_context structure.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  **/
 static int rpc_printer_migrate_settings(struct net_context *c, int argc,
                                        const char **argv)
@@ -7025,7 +6596,7 @@ static int rpc_printer_migrate_settings(struct net_context *c, int argc,
                return -1;
        }
 
-       return run_rpc_command(c, NULL, PI_SPOOLSS, 0,
+       return run_rpc_command(c, NULL, &ndr_table_spoolss.syntax_id, 0,
                               rpc_printer_migrate_settings_internals,
                               argc, argv);
 }
@@ -7033,10 +6604,10 @@ static int rpc_printer_migrate_settings(struct net_context *c, int argc,
 /**
  * 'net rpc printer' entrypoint.
  *
- * @param c    A net_context structure
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param c    A net_context structure.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  **/
 
 int rpc_printer_migrate(struct net_context *c, int argc, const char **argv)
@@ -7046,7 +6617,7 @@ int rpc_printer_migrate(struct net_context *c, int argc, const char **argv)
           rpc_printer_migrate_drivers_internals, the printer-queue already
           *has* to exist */
 
-       struct functable3 func[] = {
+       struct functable func[] = {
                {
                        "all",
                        rpc_printer_migrate_all,
@@ -7098,19 +6669,19 @@ int rpc_printer_migrate(struct net_context *c, int argc, const char **argv)
                {NULL, NULL, 0, NULL, NULL}
        };
 
-       return net_run_function3(c, argc, argv, "net rpc printer migrate",func);
+       return net_run_function(c, argc, argv, "net rpc printer migrate",func);
 }
 
 
 /**
- * List printers on a remote RPC server
+ * List printers on a remote RPC server.
  *
- * @param c    A net_context structure
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param c    A net_context structure.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  **/
 static int rpc_printer_list(struct net_context *c, int argc, const char **argv)
 {
@@ -7121,20 +6692,20 @@ static int rpc_printer_list(struct net_context *c, int argc, const char **argv)
                return 0;
        }
 
-       return run_rpc_command(c, NULL, PI_SPOOLSS, 0,
+       return run_rpc_command(c, NULL, &ndr_table_spoolss.syntax_id, 0,
                               rpc_printer_list_internals,
                               argc, argv);
 }
 
 /**
- * List printer-drivers on a remote RPC server
+ * List printer-drivers on a remote RPC server.
  *
- * @param c    A net_context structure
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param c    A net_context structure.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  **/
 static int rpc_printer_driver_list(struct net_context *c, int argc,
                                   const char **argv)
@@ -7146,20 +6717,20 @@ static int rpc_printer_driver_list(struct net_context *c, int argc,
                return 0;
        }
 
-       return run_rpc_command(c, NULL, PI_SPOOLSS, 0,
+       return run_rpc_command(c, NULL, &ndr_table_spoolss.syntax_id, 0,
                               rpc_printer_driver_list_internals,
                               argc, argv);
 }
 
 /**
- * Publish printer in ADS via MSRPC
+ * Publish printer in ADS via MSRPC.
  *
- * @param c    A net_context structure
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param c    A net_context structure.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  **/
 static int rpc_printer_publish_publish(struct net_context *c, int argc,
                                       const char **argv)
@@ -7171,20 +6742,20 @@ static int rpc_printer_publish_publish(struct net_context *c, int argc,
                return 0;
        }
 
-       return run_rpc_command(c, NULL, PI_SPOOLSS, 0,
+       return run_rpc_command(c, NULL, &ndr_table_spoolss.syntax_id, 0,
                               rpc_printer_publish_publish_internals,
                               argc, argv);
 }
 
 /**
- * Update printer in ADS via MSRPC
+ * Update printer in ADS via MSRPC.
  *
- * @param c    A net_context structure
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param c    A net_context structure.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  **/
 static int rpc_printer_publish_update(struct net_context *c, int argc, const char **argv)
 {
@@ -7195,20 +6766,20 @@ static int rpc_printer_publish_update(struct net_context *c, int argc, const cha
                return 0;
        }
 
-       return run_rpc_command(c, NULL, PI_SPOOLSS, 0,
+       return run_rpc_command(c, NULL, &ndr_table_spoolss.syntax_id, 0,
                               rpc_printer_publish_update_internals,
                               argc, argv);
 }
 
 /**
- * UnPublish printer in ADS via MSRPC
+ * UnPublish printer in ADS via MSRPC.
  *
- * @param c    A net_context structure
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param c    A net_context structure.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  **/
 static int rpc_printer_publish_unpublish(struct net_context *c, int argc,
                                         const char **argv)
@@ -7220,20 +6791,20 @@ static int rpc_printer_publish_unpublish(struct net_context *c, int argc,
                return 0;
        }
 
-       return run_rpc_command(c, NULL, PI_SPOOLSS, 0,
+       return run_rpc_command(c, NULL, &ndr_table_spoolss.syntax_id, 0,
                               rpc_printer_publish_unpublish_internals,
                               argc, argv);
 }
 
 /**
- * List published printers via MSRPC
+ * List published printers via MSRPC.
  *
- * @param c    A net_context structure
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param c    A net_context structure.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  **/
 static int rpc_printer_publish_list(struct net_context *c, int argc,
                                    const char **argv)
@@ -7245,27 +6816,27 @@ static int rpc_printer_publish_list(struct net_context *c, int argc,
                return 0;
        }
 
-       return run_rpc_command(c, NULL, PI_SPOOLSS, 0,
+       return run_rpc_command(c, NULL, &ndr_table_spoolss.syntax_id, 0,
                               rpc_printer_publish_list_internals,
                               argc, argv);
 }
 
 
 /**
- * Publish printer in ADS
+ * Publish printer in ADS.
  *
- * @param c    A net_context structure
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param c    A net_context structure.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  *
- * @return A shell status integer (0 for success)
+ * @return A shell status integer (0 for success).
  **/
 static int rpc_printer_publish(struct net_context *c, int argc,
                               const char **argv)
 {
 
-       struct functable3 func[] = {
+       struct functable func[] = {
                {
                        "publish",
                        rpc_printer_publish_publish,
@@ -7310,12 +6881,12 @@ static int rpc_printer_publish(struct net_context *c, int argc,
                        net_display_usage_from_functable(func);
                        return 0;
                }
-               return run_rpc_command(c, NULL, PI_SPOOLSS, 0,
+               return run_rpc_command(c, NULL, &ndr_table_spoolss.syntax_id, 0,
                               rpc_printer_publish_list_internals,
                               argc, argv);
        }
 
-       return net_run_function3(c, argc, argv, "net rpc printer publish",func);
+       return net_run_function(c, argc, argv, "net rpc printer publish",func);
 
 }
 
@@ -7323,10 +6894,10 @@ static int rpc_printer_publish(struct net_context *c, int argc,
 /**
  * Display rpc printer help page.
  *
- * @param c    A net_context structure
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param c    A net_context structure.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  **/
 int rpc_printer_usage(struct net_context *c, int argc, const char **argv)
 {
@@ -7362,14 +6933,14 @@ int rpc_printer_usage(struct net_context *c, int argc, const char **argv)
 /**
  * 'net rpc printer' entrypoint.
  *
- * @param c    A net_context structure
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param c    A net_context structure.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  **/
 int net_rpc_printer(struct net_context *c, int argc, const char **argv)
 {
-       struct functable3 func[] = {
+       struct functable func[] = {
                {
                        "list",
                        rpc_printer_list,
@@ -7413,26 +6984,28 @@ int net_rpc_printer(struct net_context *c, int argc, const char **argv)
                        net_display_usage_from_functable(func);
                        return 0;
                }
-               return run_rpc_command(c, NULL, PI_SPOOLSS, 0,
+               return run_rpc_command(c, NULL, &ndr_table_spoolss.syntax_id, 0,
                               rpc_printer_list_internals,
                               argc, argv);
        }
 
-       return net_run_function3(c, argc, argv, "net rpc printer", func);
+       return net_run_function(c, argc, argv, "net rpc printer", func);
 }
 
 /**
  * 'net rpc' entrypoint.
  *
- * @param c    A net_context structure
- * @param argc  Standard main() style argc
- * @param argv  Standard main() style argv.  Initial components are already
- *              stripped
+ * @param c    A net_context structure.
+ * @param argc  Standard main() style argc.
+ * @param argv  Standard main() style argv. Initial components are already
+ *              stripped.
  **/
 
 int net_rpc(struct net_context *c, int argc, const char **argv)
 {
-       struct functable3 func[] = {
+       NET_API_STATUS status;
+
+       struct functable func[] = {
                {
                        "audit",
                        net_rpc_audit,
@@ -7612,5 +7185,16 @@ int net_rpc(struct net_context *c, int argc, const char **argv)
                },
                {NULL, NULL, 0, NULL, NULL}
        };
-       return net_run_function3(c, argc, argv, "net rpc", func);
+
+       status = libnetapi_init(&c->netapi_ctx);
+       if (status != 0) {
+               return -1;
+       }
+       libnetapi_set_username(c->netapi_ctx, c->opt_user_name);
+       libnetapi_set_password(c->netapi_ctx, c->opt_password);
+       if (c->opt_kerberos) {
+               libnetapi_set_use_kerberos(c->netapi_ctx);
+       }
+
+       return net_run_function(c, argc, argv, "net rpc", func);
 }