s3-netlogon: make _netr_ServerAuthenticate a callback to _netr_ServerAuthenticate3.
[ira/wip.git] / source3 / rpc_server / srv_netlog_nt.c
index 2b4921921fbd35d9c8edddfcec51c0555def42bb..de898569f20f44fbcd0d27c9258d7285bf8fee65 100644 (file)
@@ -6,6 +6,7 @@
  *  Copyright (C) Paul Ashton                       1997.
  *  Copyright (C) Jeremy Allison               1998-2001.
  *  Copyright (C) Andrew Bartlett                   2001.
+ *  Copyright (C) Guenther Deschner                2008.
  *
  *  This program is free software; you can redistribute it and/or modify
  *  it under the terms of the GNU General Public License as published by
@@ -24,6 +25,7 @@
 /* This is the implementation of the netlogon pipe. */
 
 #include "includes.h"
+#include "../libcli/auth/libcli_auth.h"
 
 extern userdom_struct current_user_info;
 
@@ -35,7 +37,7 @@ extern userdom_struct current_user_info;
  *************************************************************************/
 
 static void init_net_r_req_chal(struct netr_Credential *r,
-                               DOM_CHAL *srv_chal)
+                               struct netr_Credential *srv_chal)
 {
        DEBUG(6,("init_net_r_req_chal: %d\n", __LINE__));
 
@@ -89,28 +91,24 @@ static void init_netlogon_info3(struct netr_NETLOGON_INFO_3 *r,
 WERROR _netr_LogonControl(pipes_struct *p,
                          struct netr_LogonControl *r)
 {
-       struct netr_NETLOGON_INFO_1 *info1;
-       uint32_t flags = 0x0;
-       uint32_t pdc_connection_status = W_ERROR_V(WERR_OK);
-
-       /* Setup the Logon Control response */
+       struct netr_LogonControl2Ex l;
 
        switch (r->in.level) {
-               case 1:
-                       info1 = TALLOC_ZERO_P(p->mem_ctx, struct netr_NETLOGON_INFO_1);
-                       if (!info1) {
-                               return WERR_NOMEM;
-                       }
-                       init_netlogon_info1(info1,
-                                           flags,
-                                           pdc_connection_status);
-                       r->out.info->info1 = info1;
-                       break;
-               default:
-                       return WERR_UNKNOWN_LEVEL;
+       case 1:
+               break;
+       case 2:
+               return WERR_NOT_SUPPORTED;
+       default:
+               return WERR_UNKNOWN_LEVEL;
        }
 
-       return WERR_OK;
+       l.in.logon_server       = r->in.logon_server;
+       l.in.function_code      = r->in.function_code;
+       l.in.level              = r->in.level;
+       l.in.data               = NULL;
+       l.out.query             = r->out.query;
+
+       return _netr_LogonControl2Ex(p, &l);
 }
 
 /****************************************************************************
@@ -130,20 +128,53 @@ static void send_sync_message(void)
 
 WERROR _netr_LogonControl2(pipes_struct *p,
                           struct netr_LogonControl2 *r)
+{
+       struct netr_LogonControl2Ex l;
+
+       l.in.logon_server       = r->in.logon_server;
+       l.in.function_code      = r->in.function_code;
+       l.in.level              = r->in.level;
+       l.in.data               = r->in.data;
+       l.out.query             = r->out.query;
+
+       return _netr_LogonControl2Ex(p, &l);
+}
+
+/****************************************************************
+ _netr_LogonControl2Ex
+****************************************************************/
+
+WERROR _netr_LogonControl2Ex(pipes_struct *p,
+                            struct netr_LogonControl2Ex *r)
 {
         uint32 flags = 0x0;
         uint32 pdc_connection_status = 0x0;
         uint32 logon_attempts = 0x0;
         uint32 tc_status;
-       fstring dc_name, dc_name2;
+       fstring dc_name2;
+       const char *dc_name = NULL;
        struct sockaddr_storage dc_ss;
        const char *domain = NULL;
        struct netr_NETLOGON_INFO_1 *info1;
        struct netr_NETLOGON_INFO_2 *info2;
        struct netr_NETLOGON_INFO_3 *info3;
+       const char *fn;
+
+       switch (p->hdr_req.opnum) {
+               case NDR_NETR_LOGONCONTROL:
+                       fn = "_netr_LogonControl";
+                       break;
+               case NDR_NETR_LOGONCONTROL2:
+                       fn = "_netr_LogonControl2";
+                       break;
+               case NDR_NETR_LOGONCONTROL2EX:
+                       fn = "_netr_LogonControl2Ex";
+                       break;
+               default:
+                       return WERR_INVALID_PARAM;
+       }
 
        tc_status = W_ERROR_V(WERR_NO_SUCH_DOMAIN);
-       fstrcpy( dc_name, "" );
 
        switch (r->in.function_code) {
                case NETLOGON_CONTROL_TC_QUERY:
@@ -157,7 +188,10 @@ WERROR _netr_LogonControl2(pipes_struct *p,
                                break;
                        }
 
-                       fstr_sprintf( dc_name, "\\\\%s", dc_name2 );
+                       dc_name = talloc_asprintf(p->mem_ctx, "\\\\%s", dc_name2);
+                       if (!dc_name) {
+                               return WERR_NOMEM;
+                       }
 
                        tc_status = W_ERROR_V(WERR_OK);
 
@@ -174,7 +208,10 @@ WERROR _netr_LogonControl2(pipes_struct *p,
                                break;
                        }
 
-                       fstr_sprintf( dc_name, "\\\\%s", dc_name2 );
+                       dc_name = talloc_asprintf(p->mem_ctx, "\\\\%s", dc_name2);
+                       if (!dc_name) {
+                               return WERR_NOMEM;
+                       }
 
                        tc_status = W_ERROR_V(WERR_OK);
 
@@ -182,8 +219,8 @@ WERROR _netr_LogonControl2(pipes_struct *p,
 
                default:
                        /* no idea what this should be */
-                       DEBUG(0,("_netr_LogonControl2: unimplemented function level [%d]\n",
-                               r->in.function_code));
+                       DEBUG(0,("%s: unimplemented function level [%d]\n",
+                               fn, r->in.function_code));
                        return WERR_UNKNOWN_LEVEL;
        }
 
@@ -259,7 +296,8 @@ WERROR _netr_NetrEnumerateTrustedDomains(pipes_struct *p,
  gets a machine password entry.  checks access rights of the host.
  ******************************************************************/
 
-static NTSTATUS get_md4pw(char *md4pw, const char *mach_acct, uint16 sec_chan_type)
+static NTSTATUS get_md4pw(char *md4pw, const char *mach_acct,
+                         uint16_t sec_chan_type, uint32_t *rid)
 {
        struct samu *sampass = NULL;
        const uint8 *pass;
@@ -355,6 +393,10 @@ static NTSTATUS get_md4pw(char *md4pw, const char *mach_acct, uint16 sec_chan_ty
        memcpy(md4pw, pass, 16);
        dump_data(5, (uint8 *)md4pw, 16);
 
+       if (rid) {
+               *rid = pdb_get_user_rid(sampass);
+       }
+
        TALLOC_FREE(sampass);
 
        return NT_STATUS_OK;
@@ -370,7 +412,7 @@ NTSTATUS _netr_ServerReqChallenge(pipes_struct *p,
                                  struct netr_ServerReqChallenge *r)
 {
        if (!p->dc) {
-               p->dc = TALLOC_ZERO_P(p->pipe_state_mem_ctx, struct dcinfo);
+               p->dc = TALLOC_ZERO_P(p, struct dcinfo);
                if (!p->dc) {
                        return NT_STATUS_NO_MEMORY;
                }
@@ -390,7 +432,7 @@ NTSTATUS _netr_ServerReqChallenge(pipes_struct *p,
        generate_random_buffer(p->dc->srv_chal.data, 8);
 
        /* set up the LSA REQUEST CHALLENGE response */
-       init_net_r_req_chal(r->out.credentials, &p->dc->srv_chal);
+       init_net_r_req_chal(r->out.return_credentials, &p->dc->srv_chal);
 
        p->dc->challenge_sent = True;
 
@@ -405,115 +447,120 @@ NTSTATUS _netr_ServerReqChallenge(pipes_struct *p,
 NTSTATUS _netr_ServerAuthenticate(pipes_struct *p,
                                  struct netr_ServerAuthenticate *r)
 {
-       NTSTATUS status;
-       DOM_CHAL srv_chal_out;
+       struct netr_ServerAuthenticate3 a;
+       uint32_t negotiate_flags = 0;
+       uint32_t rid;
 
-       if (!p->dc || !p->dc->challenge_sent) {
-               return NT_STATUS_ACCESS_DENIED;
-       }
+       a.in.server_name                = r->in.server_name;
+       a.in.account_name               = r->in.account_name;
+       a.in.secure_channel_type        = r->in.secure_channel_type;
+       a.in.computer_name              = r->in.computer_name;
+       a.in.credentials                = r->in.credentials;
+       a.in.negotiate_flags            = &negotiate_flags;
 
-       status = get_md4pw((char *)p->dc->mach_pw,
-                          r->in.account_name,
-                          r->in.secure_channel_type);
-       if (!NT_STATUS_IS_OK(status)) {
-               DEBUG(0,("_netr_ServerAuthenticate: get_md4pw failed. Failed to "
-                       "get password for machine account %s "
-                       "from client %s: %s\n",
-                       r->in.account_name,
-                       r->in.computer_name,
-                       nt_errstr(status) ));
-               /* always return NT_STATUS_ACCESS_DENIED */
-               return NT_STATUS_ACCESS_DENIED;
-       }
-
-       /* From the client / server challenges and md4 password, generate sess key */
-       creds_server_init(0,                    /* No neg flags. */
-                       p->dc,
-                       &p->dc->clnt_chal,      /* Stored client chal. */
-                       &p->dc->srv_chal,       /* Stored server chal. */
-                       p->dc->mach_pw,
-                       &srv_chal_out);
-
-       /* Check client credentials are valid. */
-       if (!netlogon_creds_server_check(p->dc, r->in.credentials)) {
-               DEBUG(0,("_netr_ServerAuthenticate: netlogon_creds_server_check failed. Rejecting auth "
-                       "request from client %s machine account %s\n",
-                       r->in.computer_name,
-                       r->in.account_name));
-               return NT_STATUS_ACCESS_DENIED;
-       }
-
-       fstrcpy(p->dc->mach_acct, r->in.account_name);
-       fstrcpy(p->dc->remote_machine, r->in.computer_name);
-       p->dc->authenticated = True;
+       a.out.return_credentials        = r->out.return_credentials;
+       a.out.rid                       = &rid;
+       a.out.negotiate_flags           = &negotiate_flags;
 
-       /* set up the LSA AUTH response */
-       /* Return the server credentials. */
+       return _netr_ServerAuthenticate3(p, &a);
 
-       memcpy(r->out.credentials->data, &srv_chal_out.data,
-              sizeof(r->out.credentials->data));
-
-       return NT_STATUS_OK;
 }
 
 /*************************************************************************
- init_net_r_auth_2:
+ _netr_ServerAuthenticate3
  *************************************************************************/
 
-static void init_net_r_auth_2(NET_R_AUTH_2 *r_a,
-                              DOM_CHAL *resp_cred, NEG_FLAGS *flgs, NTSTATUS status)
-{
-       memcpy(r_a->srv_chal.data, resp_cred->data, sizeof(resp_cred->data));
-       memcpy(&r_a->srv_flgs, flgs, sizeof(r_a->srv_flgs));
-       r_a->status = status;
-}
-
-/*************************************************************************
- _net_auth_2
- *************************************************************************/
-
-NTSTATUS _net_auth_2(pipes_struct *p, NET_Q_AUTH_2 *q_u, NET_R_AUTH_2 *r_u)
+NTSTATUS _netr_ServerAuthenticate3(pipes_struct *p,
+                                  struct netr_ServerAuthenticate3 *r)
 {
        NTSTATUS status;
-       NEG_FLAGS srv_flgs;
-       fstring mach_acct;
-       fstring remote_machine;
-       DOM_CHAL srv_chal_out;
+       uint32_t srv_flgs;
+       /* r->in.negotiate_flags is an aliased pointer to r->out.negotiate_flags,
+        * so use a copy to avoid destroying the client values. */
+       uint32_t in_neg_flags = *r->in.negotiate_flags;
+       struct netr_Credential srv_chal_out;
+       const char *fn;
+       uint32_t rid;
+
+       /* According to Microsoft (see bugid #6099)
+        * Windows 7 looks at the negotiate_flags
+        * returned in this structure *even if the
+        * call fails with access denied* ! So in order
+        * to allow Win7 to connect to a Samba NT style
+        * PDC we set the flags before we know if it's
+        * an error or not.
+        */
+
+       /* 0x000001ff */
+       srv_flgs = NETLOGON_NEG_ACCOUNT_LOCKOUT |
+                  NETLOGON_NEG_PERSISTENT_SAMREPL |
+                  NETLOGON_NEG_ARCFOUR |
+                  NETLOGON_NEG_PROMOTION_COUNT |
+                  NETLOGON_NEG_CHANGELOG_BDC |
+                  NETLOGON_NEG_FULL_SYNC_REPL |
+                  NETLOGON_NEG_MULTIPLE_SIDS |
+                  NETLOGON_NEG_REDO |
+                  NETLOGON_NEG_PASSWORD_CHANGE_REFUSAL;
+
+       /* Ensure we support strong (128-bit) keys. */
+       if (in_neg_flags & NETLOGON_NEG_STRONG_KEYS) {
+               srv_flgs |= NETLOGON_NEG_STRONG_KEYS;
+       }
 
-       rpcstr_pull(mach_acct, q_u->clnt_id.uni_acct_name.buffer,sizeof(fstring),
-                               q_u->clnt_id.uni_acct_name.uni_str_len*2,0);
+       if (lp_server_schannel() != false) {
+               srv_flgs |= NETLOGON_NEG_SCHANNEL;
+       }
 
-       /* We use this as the key to store the creds. */
-       rpcstr_pull(remote_machine, q_u->clnt_id.uni_comp_name.buffer,sizeof(fstring),
-                               q_u->clnt_id.uni_comp_name.uni_str_len*2,0);
+       switch (p->hdr_req.opnum) {
+               case NDR_NETR_SERVERAUTHENTICATE:
+                       fn = "_netr_ServerAuthenticate";
+                       break;
+               case NDR_NETR_SERVERAUTHENTICATE2:
+                       fn = "_netr_ServerAuthenticate2";
+                       break;
+               case NDR_NETR_SERVERAUTHENTICATE3:
+                       fn = "_netr_ServerAuthenticate3";
+                       break;
+               default:
+                       return NT_STATUS_INTERNAL_ERROR;
+       }
+
+       /* We use this as the key to store the creds: */
+       /* r->in.computer_name */
 
        if (!p->dc || !p->dc->challenge_sent) {
-               DEBUG(0,("_net_auth2: no challenge sent to client %s\n",
-                       remote_machine ));
-               return NT_STATUS_ACCESS_DENIED;
+               DEBUG(0,("%s: no challenge sent to client %s\n", fn,
+                       r->in.computer_name));
+               status = NT_STATUS_ACCESS_DENIED;
+               goto out;
        }
 
-       if ( (lp_server_schannel() == True) &&
-            ((q_u->clnt_flgs.neg_flags & NETLOGON_NEG_SCHANNEL) == 0) ) {
+       if ( (lp_server_schannel() == true) &&
+            ((in_neg_flags & NETLOGON_NEG_SCHANNEL) == 0) ) {
 
                /* schannel must be used, but client did not offer it. */
-               DEBUG(0,("_net_auth2: schannel required but client failed "
+               DEBUG(0,("%s: schannel required but client failed "
                        "to offer it. Client was %s\n",
-                       mach_acct ));
-               return NT_STATUS_ACCESS_DENIED;
+                       fn, r->in.account_name));
+               status = NT_STATUS_ACCESS_DENIED;
+               goto out;
        }
 
-       status = get_md4pw((char *)p->dc->mach_pw, mach_acct, q_u->clnt_id.sec_chan);
+       status = get_md4pw((char *)p->dc->mach_pw,
+                          r->in.account_name,
+                          r->in.secure_channel_type,
+                          &rid);
        if (!NT_STATUS_IS_OK(status)) {
-               DEBUG(0,("_net_auth2: failed to get machine password for "
+               DEBUG(0,("%s: failed to get machine password for "
                        "account %s: %s\n",
-                       mach_acct, nt_errstr(status) ));
+                       fn, r->in.account_name, nt_errstr(status) ));
                /* always return NT_STATUS_ACCESS_DENIED */
-               return NT_STATUS_ACCESS_DENIED;
+               status = NT_STATUS_ACCESS_DENIED;
+               goto out;
        }
 
        /* From the client / server challenges and md4 password, generate sess key */
-       creds_server_init(q_u->clnt_flgs.neg_flags,
+       creds_server_init(in_neg_flags,
                        p->dc,
                        &p->dc->clnt_chal,      /* Stored client chal. */
                        &p->dc->srv_chal,       /* Stored server chal. */
@@ -521,24 +568,20 @@ NTSTATUS _net_auth_2(pipes_struct *p, NET_Q_AUTH_2 *q_u, NET_R_AUTH_2 *r_u)
                        &srv_chal_out);
 
        /* Check client credentials are valid. */
-       if (!creds_server_check(p->dc, &q_u->clnt_chal)) {
-               DEBUG(0,("_net_auth2: creds_server_check failed. Rejecting auth "
+       if (!netlogon_creds_server_check(p->dc, r->in.credentials)) {
+               DEBUG(0,("%s: netlogon_creds_server_check failed. Rejecting auth "
                        "request from client %s machine account %s\n",
-                       remote_machine, mach_acct ));
-               return NT_STATUS_ACCESS_DENIED;
-       }
-
-       srv_flgs.neg_flags = 0x000001ff;
-
-       if (lp_server_schannel() != False) {
-               srv_flgs.neg_flags |= NETLOGON_NEG_SCHANNEL;
+                       fn, r->in.computer_name,
+                       r->in.account_name));
+               status = NT_STATUS_ACCESS_DENIED;
+               goto out;
        }
-
        /* set up the LSA AUTH 2 response */
-       init_net_r_auth_2(r_u, &srv_chal_out, &srv_flgs, NT_STATUS_OK);
+       memcpy(r->out.return_credentials->data, &srv_chal_out.data,
+              sizeof(r->out.return_credentials->data));
 
-       fstrcpy(p->dc->mach_acct, mach_acct);
-       fstrcpy(p->dc->remote_machine, remote_machine);
+       fstrcpy(p->dc->mach_acct, r->in.account_name);
+       fstrcpy(p->dc->remote_machine, r->in.computer_name);
        fstrcpy(p->dc->domain, lp_workgroup() );
 
        p->dc->authenticated = True;
@@ -546,11 +589,42 @@ NTSTATUS _net_auth_2(pipes_struct *p, NET_Q_AUTH_2 *q_u, NET_R_AUTH_2 *r_u)
        /* Store off the state so we can continue after client disconnect. */
        become_root();
        secrets_store_schannel_session_info(p->mem_ctx,
-                                       remote_machine,
-                                       p->dc);
+                                           r->in.computer_name,
+                                           p->dc);
        unbecome_root();
 
-       return r_u->status;
+       *r->out.rid = rid;
+
+       status = NT_STATUS_OK;
+
+  out:
+
+       *r->out.negotiate_flags = srv_flgs;
+       return status;
+}
+
+/*************************************************************************
+ _netr_ServerAuthenticate2
+ *************************************************************************/
+
+NTSTATUS _netr_ServerAuthenticate2(pipes_struct *p,
+                                  struct netr_ServerAuthenticate2 *r)
+{
+       struct netr_ServerAuthenticate3 a;
+       uint32_t rid;
+
+       a.in.server_name                = r->in.server_name;
+       a.in.account_name               = r->in.account_name;
+       a.in.secure_channel_type        = r->in.secure_channel_type;
+       a.in.computer_name              = r->in.computer_name;
+       a.in.credentials                = r->in.credentials;
+       a.in.negotiate_flags            = r->in.negotiate_flags;
+
+       a.out.return_credentials        = r->out.return_credentials;
+       a.out.rid                       = &rid;
+       a.out.negotiate_flags           = r->out.negotiate_flags;
+
+       return _netr_ServerAuthenticate3(p, &a);
 }
 
 /*************************************************************************
@@ -561,7 +635,6 @@ NTSTATUS _netr_ServerPasswordSet(pipes_struct *p,
                                 struct netr_ServerPasswordSet *r)
 {
        NTSTATUS status = NT_STATUS_OK;
-       fstring remote_machine;
        struct samu *sampass=NULL;
        bool ret = False;
        unsigned char pwd[16];
@@ -572,24 +645,20 @@ NTSTATUS _netr_ServerPasswordSet(pipes_struct *p,
 
        DEBUG(5,("_netr_ServerPasswordSet: %d\n", __LINE__));
 
-       /* We need the remote machine name for the creds lookup. */
-       fstrcpy(remote_machine, r->in.computer_name);
-
        if ( (lp_server_schannel() == True) && (p->auth.auth_type != PIPE_AUTH_TYPE_SCHANNEL) ) {
                /* 'server schannel = yes' should enforce use of
                   schannel, the client did offer it in auth2, but
                   obviously did not use it. */
                DEBUG(0,("_netr_ServerPasswordSet: client %s not using schannel for netlogon\n",
-                       remote_machine ));
+                       r->in.computer_name));
                return NT_STATUS_ACCESS_DENIED;
        }
 
        if (!p->dc) {
                /* Restore the saved state of the netlogon creds. */
                become_root();
-               ret = secrets_restore_schannel_session_info(p->pipe_state_mem_ctx,
-                                                       remote_machine,
-                                                       &p->dc);
+               ret = secrets_restore_schannel_session_info(p, r->in.computer_name,
+                                                           &p->dc);
                unbecome_root();
                if (!ret) {
                        return NT_STATUS_INVALID_HANDLE;
@@ -601,13 +670,13 @@ NTSTATUS _netr_ServerPasswordSet(pipes_struct *p,
        }
 
        DEBUG(3,("_netr_ServerPasswordSet: Server Password Set by remote machine:[%s] on account [%s]\n",
-                       remote_machine, p->dc->mach_acct));
+                       r->in.computer_name, p->dc->mach_acct));
 
        /* Step the creds chain forward. */
        if (!netlogon_creds_server_step(p->dc, r->in.credential, &cred_out)) {
                DEBUG(2,("_netr_ServerPasswordSet: netlogon_creds_server_step failed. Rejecting auth "
                        "request from client %s machine account %s\n",
-                       remote_machine, p->dc->mach_acct ));
+                       r->in.computer_name, p->dc->mach_acct ));
                return NT_STATUS_INVALID_PARAMETER;
        }
 
@@ -618,9 +687,7 @@ NTSTATUS _netr_ServerPasswordSet(pipes_struct *p,
        }
 
        become_root();
-       secrets_store_schannel_session_info(p->pipe_state_mem_ctx,
-                                               remote_machine,
-                                               p->dc);
+       secrets_store_schannel_session_info(p, r->in.computer_name, p->dc);
        ret = pdb_getsampwnam(sampass, p->dc->mach_acct);
        unbecome_root();
 
@@ -687,45 +754,39 @@ NTSTATUS _netr_ServerPasswordSet(pipes_struct *p,
        /* set up the LSA Server Password Set response */
 
        memcpy(r->out.return_authenticator, &cred_out,
-              sizeof(r->out.return_authenticator));
+              sizeof(*(r->out.return_authenticator)));
 
        TALLOC_FREE(sampass);
        return status;
 }
 
 /*************************************************************************
- _net_sam_logoff:
+ _netr_LogonSamLogoff
  *************************************************************************/
 
-NTSTATUS _net_sam_logoff(pipes_struct *p, NET_Q_SAM_LOGOFF *q_u, NET_R_SAM_LOGOFF *r_u)
+NTSTATUS _netr_LogonSamLogoff(pipes_struct *p,
+                             struct netr_LogonSamLogoff *r)
 {
-       fstring remote_machine;
-
        if ( (lp_server_schannel() == True) && (p->auth.auth_type != PIPE_AUTH_TYPE_SCHANNEL) ) {
                /* 'server schannel = yes' should enforce use of
                   schannel, the client did offer it in auth2, but
                   obviously did not use it. */
-               DEBUG(0,("_net_sam_logoff: client %s not using schannel for netlogon\n",
+               DEBUG(0,("_netr_LogonSamLogoff: client %s not using schannel for netlogon\n",
                        get_remote_machine_name() ));
                return NT_STATUS_ACCESS_DENIED;
        }
 
 
-       if (!get_valid_user_struct(p->vuid))
-               return NT_STATUS_NO_SUCH_USER;
-
-       /* Get the remote machine name for the creds store. */
-       rpcstr_pull(remote_machine,q_u->sam_id.client.login.uni_comp_name.buffer,
-                   sizeof(remote_machine),q_u->sam_id.client.login.uni_comp_name.uni_str_len*2,0);
+       /* Using the remote machine name for the creds store: */
+       /* r->in.computer_name */
 
        if (!p->dc) {
                /* Restore the saved state of the netlogon creds. */
                bool ret;
 
                become_root();
-               ret = secrets_restore_schannel_session_info(p->pipe_state_mem_ctx,
-                                               remote_machine,
-                                               &p->dc);
+               ret = secrets_restore_schannel_session_info(
+                       p, r->in.computer_name, &p->dc);
                unbecome_root();
                if (!ret) {
                        return NT_STATUS_INVALID_HANDLE;
@@ -736,126 +797,95 @@ NTSTATUS _net_sam_logoff(pipes_struct *p, NET_Q_SAM_LOGOFF *q_u, NET_R_SAM_LOGOF
                return NT_STATUS_INVALID_HANDLE;
        }
 
-       r_u->buffer_creds = 1; /* yes, we have valid server credentials */
-
        /* checks and updates credentials.  creates reply credentials */
-       if (!creds_server_step(p->dc, &q_u->sam_id.client.cred, &r_u->srv_creds)) {
-               DEBUG(2,("_net_sam_logoff: creds_server_step failed. Rejecting auth "
+       if (!netlogon_creds_server_step(p->dc, r->in.credential, r->out.return_authenticator)) {
+               DEBUG(2,("_netr_LogonSamLogoff: netlogon_creds_server_step failed. Rejecting auth "
                        "request from client %s machine account %s\n",
-                       remote_machine, p->dc->mach_acct ));
+                       r->in.computer_name, p->dc->mach_acct ));
                return NT_STATUS_INVALID_PARAMETER;
        }
 
        /* We must store the creds state after an update. */
        become_root();
-       secrets_store_schannel_session_info(p->pipe_state_mem_ctx,
-                                       remote_machine,
-                                       p->dc);
+       secrets_store_schannel_session_info(p, r->in.computer_name, p->dc);
        unbecome_root();
 
-       r_u->status = NT_STATUS_OK;
-       return r_u->status;
-}
-
-/*******************************************************************
- gets a domain user's groups from their already-calculated NT_USER_TOKEN
- ********************************************************************/
-
-static NTSTATUS nt_token_to_group_list(TALLOC_CTX *mem_ctx,
-                                      const DOM_SID *domain_sid,
-                                      size_t num_sids,
-                                      const DOM_SID *sids,
-                                      int *numgroups, DOM_GID **pgids)
-{
-       int i;
-
-       *numgroups=0;
-       *pgids = NULL;
-
-       for (i=0; i<num_sids; i++) {
-               DOM_GID gid;
-               if (!sid_peek_check_rid(domain_sid, &sids[i], &gid.g_rid)) {
-                       continue;
-               }
-               gid.attr = (SE_GROUP_MANDATORY|SE_GROUP_ENABLED_BY_DEFAULT|
-                           SE_GROUP_ENABLED);
-               ADD_TO_ARRAY(mem_ctx, DOM_GID, gid, pgids, numgroups);
-               if (*pgids == NULL) {
-                       return NT_STATUS_NO_MEMORY;
-               }
-       }
        return NT_STATUS_OK;
 }
 
 /*************************************************************************
- _net_sam_logon
+ _netr_LogonSamLogon
  *************************************************************************/
 
-static NTSTATUS _net_sam_logon_internal(pipes_struct *p,
-                                       NET_Q_SAM_LOGON *q_u,
-                                       NET_R_SAM_LOGON *r_u,
-                                       bool process_creds)
+NTSTATUS _netr_LogonSamLogon(pipes_struct *p,
+                            struct netr_LogonSamLogon *r)
 {
        NTSTATUS status = NT_STATUS_OK;
-       NET_USER_INFO_3 *usr_info = NULL;
-       NET_ID_INFO_CTR *ctr = q_u->sam_id.ctr;
-       UNISTR2 *uni_samlogon_user = NULL;
-       UNISTR2 *uni_samlogon_domain = NULL;
-       UNISTR2 *uni_samlogon_workstation = NULL;
-       fstring nt_username, nt_domain, nt_workstation;
+       union netr_LogonLevel *logon = r->in.logon;
+       const char *nt_username, *nt_domain, *nt_workstation;
        auth_usersupplied_info *user_info = NULL;
        auth_serversupplied_info *server_info = NULL;
-       struct samu *sampw;
        struct auth_context *auth_context = NULL;
+       uint8_t pipe_session_key[16];
+       bool process_creds = true;
+       const char *fn;
+
+       switch (p->hdr_req.opnum) {
+               case NDR_NETR_LOGONSAMLOGON:
+                       process_creds = true;
+                       fn = "_netr_LogonSamLogon";
+                       break;
+               case NDR_NETR_LOGONSAMLOGONEX:
+                       fn = "_netr_LogonSamLogonEx";
+               default:
+                       fn = "";
+                       process_creds = false;
+       }
 
        if ( (lp_server_schannel() == True) && (p->auth.auth_type != PIPE_AUTH_TYPE_SCHANNEL) ) {
                /* 'server schannel = yes' should enforce use of
                   schannel, the client did offer it in auth2, but
                   obviously did not use it. */
-               DEBUG(0,("_net_sam_logon_internal: client %s not using schannel for netlogon\n",
-                       get_remote_machine_name() ));
+               DEBUG(0,("%s: client %s not using schannel for netlogon\n",
+                       fn, get_remote_machine_name() ));
                return NT_STATUS_ACCESS_DENIED;
        }
 
-       usr_info = TALLOC_P(p->mem_ctx, NET_USER_INFO_3);
-       if (!usr_info) {
-               return NT_STATUS_NO_MEMORY;
-       }
+       *r->out.authoritative = true; /* authoritative response */
 
-       ZERO_STRUCTP(usr_info);
-
-       /* store the user information, if there is any. */
-       r_u->user = usr_info;
-       r_u->auth_resp = 1; /* authoritative response */
-       if (q_u->validation_level != 2 && q_u->validation_level != 3) {
-               DEBUG(0,("_net_sam_logon: bad validation_level value %d.\n", (int)q_u->validation_level ));
-               return NT_STATUS_ACCESS_DENIED;
+       switch (r->in.validation_level) {
+       case 2:
+               r->out.validation->sam2 = TALLOC_ZERO_P(p->mem_ctx, struct netr_SamInfo2);
+               if (!r->out.validation->sam2) {
+                       return NT_STATUS_NO_MEMORY;
+               }
+               break;
+       case 3:
+               r->out.validation->sam3 = TALLOC_ZERO_P(p->mem_ctx, struct netr_SamInfo3);
+               if (!r->out.validation->sam3) {
+                       return NT_STATUS_NO_MEMORY;
+               }
+               break;
+       default:
+               DEBUG(0,("%s: bad validation_level value %d.\n",
+                       fn, (int)r->in.validation_level));
+               return NT_STATUS_INVALID_INFO_CLASS;
        }
-       /* We handle the return of USER_INFO_2 instead of 3 in the parse return. Sucks, I know... */
-       r_u->switch_value = q_u->validation_level; /* indicates type of validation user info */
-       r_u->buffer_creds = 1; /* Ensure we always return server creds. */
-
-       if (!get_valid_user_struct(p->vuid))
-               return NT_STATUS_NO_SUCH_USER;
 
        if (process_creds) {
-               fstring remote_machine;
 
                /* Get the remote machine name for the creds store. */
                /* Note this is the remote machine this request is coming from (member server),
                   not neccessarily the workstation name the user is logging onto.
                */
-               rpcstr_pull(remote_machine,q_u->sam_id.client.login.uni_comp_name.buffer,
-                   sizeof(remote_machine),q_u->sam_id.client.login.uni_comp_name.uni_str_len*2,0);
 
                if (!p->dc) {
                        /* Restore the saved state of the netlogon creds. */
                        bool ret;
 
                        become_root();
-                       ret = secrets_restore_schannel_session_info(p->pipe_state_mem_ctx,
-                                       remote_machine,
-                                       &p->dc);
+                       ret = secrets_restore_schannel_session_info(
+                               p, r->in.computer_name, &p->dc);
                        unbecome_root();
                        if (!ret) {
                                return NT_STATUS_INVALID_HANDLE;
@@ -867,34 +897,31 @@ static NTSTATUS _net_sam_logon_internal(pipes_struct *p,
                }
 
                /* checks and updates credentials.  creates reply credentials */
-               if (!creds_server_step(p->dc, &q_u->sam_id.client.cred,  &r_u->srv_creds)) {
-                       DEBUG(2,("_net_sam_logon: creds_server_step failed. Rejecting auth "
+               if (!netlogon_creds_server_step(p->dc, r->in.credential,  r->out.return_authenticator)) {
+                       DEBUG(2,("%s: creds_server_step failed. Rejecting auth "
                                "request from client %s machine account %s\n",
-                               remote_machine, p->dc->mach_acct ));
+                               fn, r->in.computer_name, p->dc->mach_acct ));
                        return NT_STATUS_INVALID_PARAMETER;
                }
 
                /* We must store the creds state after an update. */
                become_root();
-               secrets_store_schannel_session_info(p->pipe_state_mem_ctx,
-                                       remote_machine,
-                                       p->dc);
+               secrets_store_schannel_session_info(p, r->in.computer_name, p->dc);
                unbecome_root();
        }
 
-       switch (q_u->sam_id.logon_level) {
-       case INTERACTIVE_LOGON_TYPE:
-               uni_samlogon_user = &ctr->auth.id1.uni_user_name;
-               uni_samlogon_domain = &ctr->auth.id1.uni_domain_name;
-
-                uni_samlogon_workstation = &ctr->auth.id1.uni_wksta_name;
+       switch (r->in.logon_level) {
+       case NetlogonInteractiveInformation:
+               nt_username     = logon->password->identity_info.account_name.string;
+               nt_domain       = logon->password->identity_info.domain_name.string;
+               nt_workstation  = logon->password->identity_info.workstation.string;
 
                DEBUG(3,("SAM Logon (Interactive). Domain:[%s].  ", lp_workgroup()));
                break;
-       case NET_LOGON_TYPE:
-               uni_samlogon_user = &ctr->auth.id2.uni_user_name;
-               uni_samlogon_domain = &ctr->auth.id2.uni_domain_name;
-               uni_samlogon_workstation = &ctr->auth.id2.uni_wksta_name;
+       case NetlogonNetworkInformation:
+               nt_username     = logon->network->identity_info.account_name.string;
+               nt_domain       = logon->network->identity_info.domain_name.string;
+               nt_workstation  = logon->network->identity_info.workstation.string;
 
                DEBUG(3,("SAM Logon (Network). Domain:[%s].  ", lp_workgroup()));
                break;
@@ -903,24 +930,23 @@ static NTSTATUS _net_sam_logon_internal(pipes_struct *p,
                return NT_STATUS_INVALID_INFO_CLASS;
        } /* end switch */
 
-       rpcstr_pull(nt_username,uni_samlogon_user->buffer,sizeof(nt_username),uni_samlogon_user->uni_str_len*2,0);
-       rpcstr_pull(nt_domain,uni_samlogon_domain->buffer,sizeof(nt_domain),uni_samlogon_domain->uni_str_len*2,0);
-       rpcstr_pull(nt_workstation,uni_samlogon_workstation->buffer,sizeof(nt_workstation),uni_samlogon_workstation->uni_str_len*2,0);
-
        DEBUG(3,("User:[%s@%s] Requested Domain:[%s]\n", nt_username, nt_workstation, nt_domain));
        fstrcpy(current_user_info.smb_name, nt_username);
        sub_set_smb_name(nt_username);
 
-       DEBUG(5,("Attempting validation level %d for unmapped username %s.\n", q_u->sam_id.ctr->switch_value, nt_username));
+       DEBUG(5,("Attempting validation level %d for unmapped username %s.\n",
+               r->in.validation_level, nt_username));
 
        status = NT_STATUS_OK;
 
-       switch (ctr->switch_value) {
-       case NET_LOGON_TYPE:
+       switch (r->in.logon_level) {
+       case NetlogonNetworkInformation:
        {
                const char *wksname = nt_workstation;
 
-               if (!NT_STATUS_IS_OK(status = make_auth_context_fixed(&auth_context, ctr->auth.id2.lm_chal))) {
+               status = make_auth_context_fixed(&auth_context,
+                                                logon->network->challenge);
+               if (!NT_STATUS_IS_OK(status)) {
                        return status;
                }
 
@@ -934,36 +960,36 @@ static NTSTATUS _net_sam_logon_internal(pipes_struct *p,
                if (!make_user_info_netlogon_network(&user_info,
                                                     nt_username, nt_domain,
                                                     wksname,
-                                                    ctr->auth.id2.param_ctrl,
-                                                    ctr->auth.id2.lm_chal_resp.buffer,
-                                                    ctr->auth.id2.lm_chal_resp.str_str_len,
-                                                    ctr->auth.id2.nt_chal_resp.buffer,
-                                                    ctr->auth.id2.nt_chal_resp.str_str_len)) {
+                                                    logon->network->identity_info.parameter_control,
+                                                    logon->network->lm.data,
+                                                    logon->network->lm.length,
+                                                    logon->network->nt.data,
+                                                    logon->network->nt.length)) {
                        status = NT_STATUS_NO_MEMORY;
                }
                break;
        }
-       case INTERACTIVE_LOGON_TYPE:
+       case NetlogonInteractiveInformation:
                /* 'Interactive' authentication, supplies the password in its
                   MD4 form, encrypted with the session key.  We will convert
                   this to challenge/response for the auth subsystem to chew
                   on */
        {
-               const uint8 *chal;
+               uint8_t chal[8];
 
                if (!NT_STATUS_IS_OK(status = make_auth_context_subsystem(&auth_context))) {
                        return status;
                }
 
-               chal = auth_context->get_ntlm_challenge(auth_context);
+               auth_context->get_ntlm_challenge(auth_context, chal);
 
                if (!make_user_info_netlogon_interactive(&user_info,
                                                         nt_username, nt_domain,
                                                         nt_workstation,
-                                                        ctr->auth.id1.param_ctrl,
+                                                        logon->password->identity_info.parameter_control,
                                                         chal,
-                                                        ctr->auth.id1.lm_owf.data,
-                                                        ctr->auth.id1.nt_owf.data,
+                                                        logon->password->lmpassword.hash,
+                                                        logon->password->ntpassword.hash,
                                                         p->dc->sess_key)) {
                        status = NT_STATUS_NO_MEMORY;
                }
@@ -982,8 +1008,8 @@ static NTSTATUS _net_sam_logon_internal(pipes_struct *p,
        (auth_context->free)(&auth_context);
        free_user_info(&user_info);
 
-       DEBUG(5, ("_net_sam_logon: check_password returned status %s\n",
-                 nt_errstr(status)));
+       DEBUG(5,("%s: check_password returned status %s\n",
+                 fn, nt_errstr(status)));
 
        /* Check account and password */
 
@@ -995,7 +1021,7 @@ static NTSTATUS _net_sam_logon_internal(pipes_struct *p,
                 if ( NT_STATUS_EQUAL(status, NT_STATUS_NO_SUCH_USER)
                     && !strequal(nt_domain, get_global_sam_name())
                     && !is_trusted_domain(nt_domain) )
-                       r_u->auth_resp = 0; /* We are not authoritative */
+                       *r->out.authoritative = false; /* We are not authoritative */
 
                TALLOC_FREE(server_info);
                return status;
@@ -1003,8 +1029,8 @@ static NTSTATUS _net_sam_logon_internal(pipes_struct *p,
 
        if (server_info->guest) {
                /* We don't like guest domain logons... */
-               DEBUG(5,("_net_sam_logon: Attempted domain logon as GUEST "
-                        "denied.\n"));
+               DEBUG(5,("%s: Attempted domain logon as GUEST "
+                        "denied.\n", fn));
                TALLOC_FREE(server_info);
                return NT_STATUS_LOGON_FAILURE;
        }
@@ -1013,181 +1039,65 @@ static NTSTATUS _net_sam_logon_internal(pipes_struct *p,
            the SAM Local Security Authority should record that the user is
            logged in to the domain.  */
 
-       {
-               DOM_GID *gids = NULL;
-               const DOM_SID *user_sid = NULL;
-               const DOM_SID *group_sid = NULL;
-               DOM_SID domain_sid;
-               uint32 user_rid, group_rid;
-
-               int num_gids = 0;
-               const char *my_name;
-               unsigned char user_session_key[16];
-               unsigned char lm_session_key[16];
-               unsigned char pipe_session_key[16];
-
-               sampw = server_info->sam_account;
-
-               /* set up pointer indicating user/password failed to be
-                * found */
-               usr_info->ptr_user_info = 0;
-
-               user_sid = pdb_get_user_sid(sampw);
-               group_sid = pdb_get_group_sid(sampw);
-
-               if ((user_sid == NULL) || (group_sid == NULL)) {
-                       DEBUG(1, ("_net_sam_logon: User without group or user SID\n"));
-                       return NT_STATUS_UNSUCCESSFUL;
-               }
-
-               sid_copy(&domain_sid, user_sid);
-               sid_split_rid(&domain_sid, &user_rid);
-
-               if (!sid_peek_check_rid(&domain_sid, group_sid, &group_rid)) {
-                       DEBUG(1, ("_net_sam_logon: user %s\\%s has user sid "
-                                 "%s\n but group sid %s.\n"
-                                 "The conflicting domain portions are not "
-                                 "supported for NETLOGON calls\n",
-                                 pdb_get_domain(sampw),
-                                 pdb_get_username(sampw),
-                                 sid_string_dbg(user_sid),
-                                 sid_string_dbg(group_sid)));
-                       return NT_STATUS_UNSUCCESSFUL;
-               }
-
-               if(server_info->login_server) {
-                       my_name = server_info->login_server;
-               } else {
-                       my_name = global_myname();
-               }
-
-               status = nt_token_to_group_list(p->mem_ctx, &domain_sid,
-                                               server_info->num_sids,
-                                               server_info->sids,
-                                               &num_gids, &gids);
-
-               if (!NT_STATUS_IS_OK(status)) {
-                       return status;
-               }
-
-               if (server_info->user_session_key.length) {
-                       memcpy(user_session_key,
-                              server_info->user_session_key.data,
-                              MIN(sizeof(user_session_key),
-                                  server_info->user_session_key.length));
-                       if (process_creds) {
-                               /* Get the pipe session key from the creds. */
-                               memcpy(pipe_session_key, p->dc->sess_key, 16);
-                       } else {
-                               /* Get the pipe session key from the schannel. */
-                               if (p->auth.auth_type != PIPE_AUTH_TYPE_SCHANNEL || p->auth.a_u.schannel_auth == NULL) {
-                                       return NT_STATUS_INVALID_HANDLE;
-                               }
-                               memcpy(pipe_session_key, p->auth.a_u.schannel_auth->sess_key, 16);
-                       }
-                       SamOEMhash(user_session_key, pipe_session_key, 16);
-                       memset(pipe_session_key, '\0', 16);
-               }
-               if (server_info->lm_session_key.length) {
-                       memcpy(lm_session_key,
-                              server_info->lm_session_key.data,
-                              MIN(sizeof(lm_session_key),
-                                  server_info->lm_session_key.length));
-                       if (process_creds) {
-                               /* Get the pipe session key from the creds. */
-                               memcpy(pipe_session_key, p->dc->sess_key, 16);
-                       } else {
-                               /* Get the pipe session key from the schannel. */
-                               if (p->auth.auth_type != PIPE_AUTH_TYPE_SCHANNEL || p->auth.a_u.schannel_auth == NULL) {
-                                       return NT_STATUS_INVALID_HANDLE;
-                               }
-                               memcpy(pipe_session_key, p->auth.a_u.schannel_auth->sess_key, 16);
-                       }
-                       SamOEMhash(lm_session_key, pipe_session_key, 16);
-                       memset(pipe_session_key, '\0', 16);
+       if (process_creds) {
+               /* Get the pipe session key from the creds. */
+               memcpy(pipe_session_key, p->dc->sess_key, 16);
+       } else {
+               /* Get the pipe session key from the schannel. */
+               if ((p->auth.auth_type != PIPE_AUTH_TYPE_SCHANNEL)
+                   || (p->auth.a_u.schannel_auth == NULL)) {
+                       return NT_STATUS_INVALID_HANDLE;
                }
+               memcpy(pipe_session_key, p->auth.a_u.schannel_auth->sess_key, 16);
+       }
 
-               init_net_user_info3(p->mem_ctx, usr_info,
-                                   user_rid,
-                                   group_rid,
-                                   pdb_get_username(sampw),
-                                   pdb_get_fullname(sampw),
-                                   pdb_get_homedir(sampw),
-                                   pdb_get_dir_drive(sampw),
-                                   pdb_get_logon_script(sampw),
-                                   pdb_get_profile_path(sampw),
-                                   pdb_get_logon_time(sampw),
-                                   get_time_t_max(),
-                                   get_time_t_max(),
-                                   pdb_get_pass_last_set_time(sampw),
-                                   pdb_get_pass_can_change_time(sampw),
-                                   pdb_get_pass_must_change_time(sampw),
-                                   0, /* logon_count */
-                                   0, /* bad_pw_count */
-                                   num_gids,    /* uint32 num_groups */
-                                   gids    , /* DOM_GID *gids */
-                                   NETLOGON_EXTRA_SIDS, /* uint32 user_flgs (?) */
-                                   pdb_get_acct_ctrl(sampw),
-                                   server_info->user_session_key.length ? user_session_key : NULL,
-                                   server_info->lm_session_key.length ? lm_session_key : NULL,
-                                   my_name     , /* char *logon_srv */
-                                   pdb_get_domain(sampw),
-                                   &domain_sid);     /* DOM_SID *dom_sid */
-               ZERO_STRUCT(user_session_key);
-               ZERO_STRUCT(lm_session_key);
+       switch (r->in.validation_level) {
+       case 2:
+               status = serverinfo_to_SamInfo2(server_info, pipe_session_key, 16,
+                                               r->out.validation->sam2);
+               break;
+       case 3:
+               status = serverinfo_to_SamInfo3(server_info, pipe_session_key, 16,
+                                               r->out.validation->sam3);
+               break;
        }
-       TALLOC_FREE(server_info);
-       return status;
-}
 
-/*************************************************************************
- _net_sam_logon
- *************************************************************************/
+       TALLOC_FREE(server_info);
 
-NTSTATUS _net_sam_logon(pipes_struct *p, NET_Q_SAM_LOGON *q_u, NET_R_SAM_LOGON *r_u)
-{
-       return _net_sam_logon_internal(p, q_u, r_u, True);
+       return status;
 }
 
 /*************************************************************************
- _net_sam_logon_ex - no credential chaining. Map into net sam logon.
+ _netr_LogonSamLogonEx
+ - no credential chaining. Map into net sam logon.
  *************************************************************************/
 
-NTSTATUS _net_sam_logon_ex(pipes_struct *p, NET_Q_SAM_LOGON_EX *q_u, NET_R_SAM_LOGON_EX *r_u)
+NTSTATUS _netr_LogonSamLogonEx(pipes_struct *p,
+                              struct netr_LogonSamLogonEx *r)
 {
-       NET_Q_SAM_LOGON q;
-       NET_R_SAM_LOGON r;
-
-       ZERO_STRUCT(q);
-       ZERO_STRUCT(r);
+       struct netr_LogonSamLogon q;
 
        /* Only allow this if the pipe is protected. */
        if (p->auth.auth_type != PIPE_AUTH_TYPE_SCHANNEL) {
-               DEBUG(0,("_net_sam_logon_ex: client %s not using schannel for netlogon\n",
+               DEBUG(0,("_netr_LogonSamLogonEx: client %s not using schannel for netlogon\n",
                        get_remote_machine_name() ));
                return NT_STATUS_INVALID_PARAMETER;
         }
 
-       /* Map a NET_Q_SAM_LOGON_EX to NET_Q_SAM_LOGON. */
-       q.validation_level = q_u->validation_level;
-
-       /* Map a DOM_SAM_INFO_EX into a DOM_SAM_INFO with no creds. */
-       q.sam_id.client.login = q_u->sam_id.client;
-       q.sam_id.logon_level = q_u->sam_id.logon_level;
-       q.sam_id.ctr = q_u->sam_id.ctr;
+       q.in.server_name        = r->in.server_name;
+       q.in.computer_name      = r->in.computer_name;
+       q.in.logon_level        = r->in.logon_level;
+       q.in.logon              = r->in.logon;
+       q.in.validation_level   = r->in.validation_level;
+       /* we do not handle the flags */
+       /*                      = r->in.flags; */
 
-       r_u->status = _net_sam_logon_internal(p, &q, &r, False);
-
-       if (!NT_STATUS_IS_OK(r_u->status)) {
-               return r_u->status;
-       }
+       q.out.validation        = r->out.validation;
+       q.out.authoritative     = r->out.authoritative;
+       /* we do not handle the flags */
+       /*                      = r->out.flags; */
 
-       /* Map the NET_R_SAM_LOGON to NET_R_SAM_LOGON_EX. */
-       r_u->switch_value = r.switch_value;
-       r_u->user = r.user;
-       r_u->auth_resp = r.auth_resp;
-       r_u->flags = 0; /* FIXME ! */
-       return r_u->status;
+       return _netr_LogonSamLogon(p, &q);
 }
 
 /*************************************************************************
@@ -1233,26 +1143,6 @@ WERROR _netr_LogonUasLogoff(pipes_struct *p,
 /****************************************************************
 ****************************************************************/
 
-NTSTATUS _netr_LogonSamLogon(pipes_struct *p,
-                            struct netr_LogonSamLogon *r)
-{
-       p->rng_fault_state = true;
-       return NT_STATUS_NOT_IMPLEMENTED;
-}
-
-/****************************************************************
-****************************************************************/
-
-NTSTATUS _netr_LogonSamLogoff(pipes_struct *p,
-                             struct netr_LogonSamLogoff *r)
-{
-       p->rng_fault_state = true;
-       return NT_STATUS_NOT_IMPLEMENTED;
-}
-
-/****************************************************************
-****************************************************************/
-
 NTSTATUS _netr_DatabaseDeltas(pipes_struct *p,
                              struct netr_DatabaseDeltas *r)
 {
@@ -1313,16 +1203,6 @@ WERROR _netr_GetAnyDCName(pipes_struct *p,
 /****************************************************************
 ****************************************************************/
 
-NTSTATUS _netr_ServerAuthenticate2(pipes_struct *p,
-                                  struct netr_ServerAuthenticate2 *r)
-{
-       p->rng_fault_state = true;
-       return NT_STATUS_NOT_IMPLEMENTED;
-}
-
-/****************************************************************
-****************************************************************/
-
 NTSTATUS _netr_DatabaseSync2(pipes_struct *p,
                             struct netr_DatabaseSync2 *r)
 {
@@ -1343,16 +1223,6 @@ NTSTATUS _netr_DatabaseRedo(pipes_struct *p,
 /****************************************************************
 ****************************************************************/
 
-WERROR _netr_LogonControl2Ex(pipes_struct *p,
-                            struct netr_LogonControl2Ex *r)
-{
-       p->rng_fault_state = true;
-       return WERR_NOT_SUPPORTED;
-}
-
-/****************************************************************
-****************************************************************/
-
 WERROR _netr_DsRGetDCName(pipes_struct *p,
                          struct netr_DsRGetDCName *r)
 {
@@ -1363,11 +1233,10 @@ WERROR _netr_DsRGetDCName(pipes_struct *p,
 /****************************************************************
 ****************************************************************/
 
-WERROR _netr_NETRLOGONDUMMYROUTINE1(pipes_struct *p,
-                                   struct netr_NETRLOGONDUMMYROUTINE1 *r)
+NTSTATUS _netr_LogonGetCapabilities(pipes_struct *p,
+                                   struct netr_LogonGetCapabilities *r)
 {
-       p->rng_fault_state = true;
-       return WERR_NOT_SUPPORTED;
+       return NT_STATUS_NOT_IMPLEMENTED;
 }
 
 /****************************************************************
@@ -1413,16 +1282,6 @@ WERROR _netr_NETRLOGONCOMPUTECLIENTDIGEST(pipes_struct *p,
 /****************************************************************
 ****************************************************************/
 
-NTSTATUS _netr_ServerAuthenticate3(pipes_struct *p,
-                                  struct netr_ServerAuthenticate3 *r)
-{
-       p->rng_fault_state = true;
-       return NT_STATUS_NOT_IMPLEMENTED;
-}
-
-/****************************************************************
-****************************************************************/
-
 WERROR _netr_DsRGetDCNameEx(pipes_struct *p,
                            struct netr_DsRGetDCNameEx *r)
 {
@@ -1543,16 +1402,6 @@ WERROR _netr_DsrGetDcSiteCoverageW(pipes_struct *p,
 /****************************************************************
 ****************************************************************/
 
-NTSTATUS _netr_LogonSamLogonEx(pipes_struct *p,
-                              struct netr_LogonSamLogonEx *r)
-{
-       p->rng_fault_state = true;
-       return NT_STATUS_NOT_IMPLEMENTED;
-}
-
-/****************************************************************
-****************************************************************/
-
 WERROR _netr_DsrEnumerateDomainTrusts(pipes_struct *p,
                                      struct netr_DsrEnumerateDomainTrusts *r)
 {
@@ -1613,10 +1462,10 @@ NTSTATUS _netr_LogonSamLogonWithFlags(pipes_struct *p,
 /****************************************************************
 ****************************************************************/
 
-WERROR _netr_NETRSERVERGETTRUSTINFO(pipes_struct *p,
-                                   struct netr_NETRSERVERGETTRUSTINFO *r)
+NTSTATUS _netr_ServerGetTrustInfo(pipes_struct *p,
+                                 struct netr_ServerGetTrustInfo *r)
 {
        p->rng_fault_state = true;
-       return WERR_NOT_SUPPORTED;
+       return NT_STATUS_NOT_IMPLEMENTED;
 }