s3:libsmb/clitrans: move MID handling to the end of cli_trans_send() and add a comment
[metze/samba/wip.git] / source3 / libsmb / clitrans.c
index 120b6c0e293b83c7dc7a25d130608de134b15540..82a73ee2c077ee52ded716fe8880fcdb10fd6994 100644 (file)
 */
 
 #include "includes.h"
-
-
-/****************************************************************************
- Send a SMB trans or trans2 request.
-****************************************************************************/
-
-bool cli_send_trans(struct cli_state *cli, int trans,
-                   const char *pipe_name,
-                   int fid, int flags,
-                   uint16 *setup, unsigned int lsetup, unsigned int msetup,
-                   const char *param, unsigned int lparam, unsigned int mparam,
-                   const char *data, unsigned int ldata, unsigned int mdata)
-{
-       unsigned int i;
-       unsigned int this_ldata,this_lparam;
-       unsigned int tot_data=0,tot_param=0;
-       char *outdata,*outparam;
-       char *p;
-       int pipe_name_len=0;
-       uint16 mid;
-
-       this_lparam = MIN(lparam,cli->max_xmit - (500+lsetup*2)); /* hack */
-       this_ldata = MIN(ldata,cli->max_xmit - (500+lsetup*2+this_lparam));
-
-       memset(cli->outbuf,'\0',smb_size);
-       cli_set_message(cli->outbuf,14+lsetup,0,True);
-       SCVAL(cli->outbuf,smb_com,trans);
-       SSVAL(cli->outbuf,smb_tid, cli->cnum);
-       cli_setup_packet(cli);
-
-       /*
-        * Save the mid we're using. We need this for finding
-        * signing replies.
-        */
-
-       mid = cli->mid;
-
-       if (pipe_name) {
-               pipe_name_len = clistr_push(cli, smb_buf(cli->outbuf), pipe_name, -1, STR_TERMINATE);
-       }
-
-       outparam = smb_buf(cli->outbuf)+(trans==SMBtrans ? pipe_name_len : 3);
-       outdata = outparam+this_lparam;
-
-       /* primary request */
-       SSVAL(cli->outbuf,smb_tpscnt,lparam);   /* tpscnt */
-       SSVAL(cli->outbuf,smb_tdscnt,ldata);    /* tdscnt */
-       SSVAL(cli->outbuf,smb_mprcnt,mparam);   /* mprcnt */
-       SSVAL(cli->outbuf,smb_mdrcnt,mdata);    /* mdrcnt */
-       SCVAL(cli->outbuf,smb_msrcnt,msetup);   /* msrcnt */
-       SSVAL(cli->outbuf,smb_flags,flags);     /* flags */
-       SIVAL(cli->outbuf,smb_timeout,0);               /* timeout */
-       SSVAL(cli->outbuf,smb_pscnt,this_lparam);       /* pscnt */
-       SSVAL(cli->outbuf,smb_psoff,smb_offset(outparam,cli->outbuf)); /* psoff */
-       SSVAL(cli->outbuf,smb_dscnt,this_ldata);        /* dscnt */
-       SSVAL(cli->outbuf,smb_dsoff,smb_offset(outdata,cli->outbuf)); /* dsoff */
-       SCVAL(cli->outbuf,smb_suwcnt,lsetup);   /* suwcnt */
-       for (i=0;i<lsetup;i++)          /* setup[] */
-               SSVAL(cli->outbuf,smb_setup+i*2,setup[i]);
-       p = smb_buf(cli->outbuf);
-       if (trans != SMBtrans) {
-               *p++ = 0;  /* put in a null smb_name */
-               *p++ = 'D'; *p++ = ' '; /* observed in OS/2 */
-       }
-       if (this_lparam)                        /* param[] */
-               memcpy(outparam,param,this_lparam);
-       if (this_ldata)                 /* data[] */
-               memcpy(outdata,data,this_ldata);
-       cli_setup_bcc(cli, outdata+this_ldata);
-
-       show_msg(cli->outbuf);
-
-       if (!cli_send_smb(cli)) {
-               return False;
-       }
-
-       /* Note we're in a trans state. Save the sequence
-        * numbers for replies. */
-       client_set_trans_sign_state_on(cli, mid);
-
-       if (this_ldata < ldata || this_lparam < lparam) {
-               /* receive interim response */
-               if (!cli_receive_smb(cli) || cli_is_error(cli)) {
-                       client_set_trans_sign_state_off(cli, mid);
-                       return(False);
-               }
-
-               tot_data = this_ldata;
-               tot_param = this_lparam;
-
-               while (tot_data < ldata || tot_param < lparam)  {
-                       this_lparam = MIN(lparam-tot_param,cli->max_xmit - 500); /* hack */
-                       this_ldata = MIN(ldata-tot_data,cli->max_xmit - (500+this_lparam));
-
-                       client_set_trans_sign_state_off(cli, mid);
-                       client_set_trans_sign_state_on(cli, mid);
-
-                       cli_set_message(cli->outbuf,trans==SMBtrans?8:9,0,True);
-                       SCVAL(cli->outbuf,smb_com,(trans==SMBtrans ? SMBtranss : SMBtranss2));
-
-                       outparam = smb_buf(cli->outbuf);
-                       outdata = outparam+this_lparam;
-
-                       /* secondary request */
-                       SSVAL(cli->outbuf,smb_tpscnt,lparam);   /* tpscnt */
-                       SSVAL(cli->outbuf,smb_tdscnt,ldata);    /* tdscnt */
-                       SSVAL(cli->outbuf,smb_spscnt,this_lparam);      /* pscnt */
-                       SSVAL(cli->outbuf,smb_spsoff,smb_offset(outparam,cli->outbuf)); /* psoff */
-                       SSVAL(cli->outbuf,smb_spsdisp,tot_param);       /* psdisp */
-                       SSVAL(cli->outbuf,smb_sdscnt,this_ldata);       /* dscnt */
-                       SSVAL(cli->outbuf,smb_sdsoff,smb_offset(outdata,cli->outbuf)); /* dsoff */
-                       SSVAL(cli->outbuf,smb_sdsdisp,tot_data);        /* dsdisp */
-                       if (trans==SMBtrans2)
-                               SSVALS(cli->outbuf,smb_sfid,fid);               /* fid */
-                       if (this_lparam)                        /* param[] */
-                               memcpy(outparam,param+tot_param,this_lparam);
-                       if (this_ldata)                 /* data[] */
-                               memcpy(outdata,data+tot_data,this_ldata);
-                       cli_setup_bcc(cli, outdata+this_ldata);
-
-                       /*
-                        * Save the mid we're using. We need this for finding
-                        * signing replies.
-                        */
-                       mid = cli->mid;
-
-                       show_msg(cli->outbuf);
-                       if (!cli_send_smb(cli)) {
-                               client_set_trans_sign_state_off(cli, mid);
-                               return False;
-                       }
-
-                       /* Ensure we use the same mid for the secondaries. */
-                       cli->mid = mid;
-
-                       tot_data += this_ldata;
-                       tot_param += this_lparam;
-               }
-       }
-
-       return(True);
-}
-
-/****************************************************************************
- Receive a SMB trans or trans2 response allocating the necessary memory.
-****************************************************************************/
-
-bool cli_receive_trans(struct cli_state *cli,int trans,
-                              char **param, unsigned int *param_len,
-                              char **data, unsigned int *data_len)
-{
-       unsigned int total_data=0;
-       unsigned int total_param=0;
-       unsigned int this_data,this_param;
-       NTSTATUS status;
-       bool ret = False;
-
-       *data_len = *param_len = 0;
-
-       if (!cli_receive_smb(cli)) {
-               return False;
-       }
-
-       show_msg(cli->inbuf);
-
-       /* sanity check */
-       if (CVAL(cli->inbuf,smb_com) != trans) {
-               DEBUG(0,("Expected %s response, got command 0x%02x\n",
-                        trans==SMBtrans?"SMBtrans":"SMBtrans2",
-                        CVAL(cli->inbuf,smb_com)));
-               return False;
-       }
-
-       /*
-        * An NT RPC pipe call can return ERRDOS, ERRmoredata
-        * to a trans call. This is not an error and should not
-        * be treated as such. Note that STATUS_NO_MORE_FILES is
-        * returned when a trans2 findfirst/next finishes.
-        * When setting up an encrypted transport we can also
-        * see NT_STATUS_MORE_PROCESSING_REQUIRED here.
-         *
-         * Vista returns NT_STATUS_INACCESSIBLE_SYSTEM_SHORTCUT if the folder
-         * "<share>/Users/All Users" is enumerated.  This is a special pseudo
-         * folder, and the response does not have parameters (nor a parameter
-         * length).
-        */
-       status = cli_nt_error(cli);
-
-       if (!NT_STATUS_EQUAL(status, NT_STATUS_MORE_PROCESSING_REQUIRED)) {
-               if (NT_STATUS_IS_ERR(status) ||
-                    NT_STATUS_EQUAL(status,STATUS_NO_MORE_FILES) ||
-                    NT_STATUS_EQUAL(status,NT_STATUS_INACCESSIBLE_SYSTEM_SHORTCUT)) {
-                       goto out;
-               }
-       }
-
-       /* parse out the lengths */
-       total_data = SVAL(cli->inbuf,smb_tdrcnt);
-       total_param = SVAL(cli->inbuf,smb_tprcnt);
-
-       /* allocate it */
-       if (total_data!=0) {
-               /* We know adding 2 is safe as total_data is an
-                * SVAL <= 0xFFFF. */
-               *data = (char *)SMB_REALLOC(*data,total_data+2);
-               if (!(*data)) {
-                       DEBUG(0,("cli_receive_trans: failed to enlarge data buffer\n"));
-                       goto out;
-               }
-       }
-
-       if (total_param!=0) {
-               /* We know adding 2 is safe as total_param is an
-                * SVAL <= 0xFFFF. */
-               *param = (char *)SMB_REALLOC(*param,total_param+2);
-               if (!(*param)) {
-                       DEBUG(0,("cli_receive_trans: failed to enlarge param buffer\n"));
-                       goto out;
-               }
-       }
-
-       for (;;)  {
-               this_data = SVAL(cli->inbuf,smb_drcnt);
-               this_param = SVAL(cli->inbuf,smb_prcnt);
-
-               if (this_data + *data_len > total_data ||
-                   this_param + *param_len > total_param) {
-                       DEBUG(1,("Data overflow in cli_receive_trans\n"));
-                       goto out;
-               }
-
-               if (this_data + *data_len < this_data ||
-                               this_data + *data_len < *data_len ||
-                               this_param + *param_len < this_param ||
-                               this_param + *param_len < *param_len) {
-                       DEBUG(1,("Data overflow in cli_receive_trans\n"));
-                       goto out;
-               }
-
-               if (this_data) {
-                       unsigned int data_offset_out = SVAL(cli->inbuf,smb_drdisp);
-                       unsigned int data_offset_in = SVAL(cli->inbuf,smb_droff);
-
-                       if (data_offset_out > total_data ||
-                                       data_offset_out + this_data > total_data ||
-                                       data_offset_out + this_data < data_offset_out ||
-                                       data_offset_out + this_data < this_data) {
-                               DEBUG(1,("Data overflow in cli_receive_trans\n"));
-                               goto out;
-                       }
-                       if (data_offset_in > cli->bufsize ||
-                                       data_offset_in + this_data >  cli->bufsize ||
-                                       data_offset_in + this_data < data_offset_in ||
-                                       data_offset_in + this_data < this_data) {
-                               DEBUG(1,("Data overflow in cli_receive_trans\n"));
-                               goto out;
-                       }
-
-                       memcpy(*data + data_offset_out, smb_base(cli->inbuf) + data_offset_in, this_data);
-               }
-               if (this_param) {
-                       unsigned int param_offset_out = SVAL(cli->inbuf,smb_prdisp);
-                       unsigned int param_offset_in = SVAL(cli->inbuf,smb_proff);
-
-                       if (param_offset_out > total_param ||
-                                       param_offset_out + this_param > total_param ||
-                                       param_offset_out + this_param < param_offset_out ||
-                                       param_offset_out + this_param < this_param) {
-                               DEBUG(1,("Param overflow in cli_receive_trans\n"));
-                               goto out;
-                       }
-                       if (param_offset_in > cli->bufsize ||
-                                       param_offset_in + this_param >  cli->bufsize ||
-                                       param_offset_in + this_param < param_offset_in ||
-                                       param_offset_in + this_param < this_param) {
-                               DEBUG(1,("Param overflow in cli_receive_trans\n"));
-                               goto out;
-                       }
-
-                       memcpy(*param + param_offset_out, smb_base(cli->inbuf) + param_offset_in, this_param);
-               }
-               *data_len += this_data;
-               *param_len += this_param;
-
-               if (total_data <= *data_len && total_param <= *param_len) {
-                       ret = True;
-                       break;
-               }
-
-               if (!cli_receive_smb(cli)) {
-                       goto out;
-               }
-
-               show_msg(cli->inbuf);
-
-               /* sanity check */
-               if (CVAL(cli->inbuf,smb_com) != trans) {
-                       DEBUG(0,("Expected %s response, got command 0x%02x\n",
-                                trans==SMBtrans?"SMBtrans":"SMBtrans2", 
-                                CVAL(cli->inbuf,smb_com)));
-                       goto out;
-               }
-               if (!NT_STATUS_EQUAL(status, NT_STATUS_MORE_PROCESSING_REQUIRED)) {
-                       if (NT_STATUS_IS_ERR(cli_nt_error(cli))) {
-                               goto out;
-                       }
-               }
-
-               /* parse out the total lengths again - they can shrink! */
-               if (SVAL(cli->inbuf,smb_tdrcnt) < total_data)
-                       total_data = SVAL(cli->inbuf,smb_tdrcnt);
-               if (SVAL(cli->inbuf,smb_tprcnt) < total_param)
-                       total_param = SVAL(cli->inbuf,smb_tprcnt);
-
-               if (total_data <= *data_len && total_param <= *param_len) {
-                       ret = True;
-                       break;
-               }
-       }
-
-  out:
-
-       if (ret) {
-               /* Ensure the last 2 bytes of param and data are 2 null
-                * bytes. These are malloc'ed, but not included in any
-                * length counts. This allows cli_XX string reading functions
-                * to safely null terminate. */
-               if (total_data) {
-                       SSVAL(*data,total_data,0);
-               }
-               if (total_param) {
-                       SSVAL(*param,total_param,0);
-               }
-       }
-
-       client_set_trans_sign_state_off(cli, SVAL(cli->inbuf,smb_mid));
-       return ret;
-}
-
-/****************************************************************************
- Send a SMB nttrans request.
-****************************************************************************/
-
-bool cli_send_nt_trans(struct cli_state *cli,
-                      int function,
-                      int flags,
-                      uint16 *setup, unsigned int lsetup, unsigned int msetup,
-                      char *param, unsigned int lparam, unsigned int mparam,
-                      char *data, unsigned int ldata, unsigned int mdata)
-{
-       unsigned int i;
-       unsigned int this_ldata,this_lparam;
-       unsigned int tot_data=0,tot_param=0;
-       uint16 mid;
-       char *outdata,*outparam;
-
-       this_lparam = MIN(lparam,cli->max_xmit - (500+lsetup*2)); /* hack */
-       this_ldata = MIN(ldata,cli->max_xmit - (500+lsetup*2+this_lparam));
-
-       memset(cli->outbuf,'\0',smb_size);
-       cli_set_message(cli->outbuf,19+lsetup,0,True);
-       SCVAL(cli->outbuf,smb_com,SMBnttrans);
-       SSVAL(cli->outbuf,smb_tid, cli->cnum);
-       cli_setup_packet(cli);
-
-       /*
-        * Save the mid we're using. We need this for finding
-        * signing replies.
-        */
-
-       mid = cli->mid;
-
-       outparam = smb_buf(cli->outbuf)+3;
-       outdata = outparam+this_lparam;
-
-       /* primary request */
-       SCVAL(cli->outbuf,smb_nt_MaxSetupCount,msetup);
-       SCVAL(cli->outbuf,smb_nt_Flags,flags);
-       SIVAL(cli->outbuf,smb_nt_TotalParameterCount, lparam);
-       SIVAL(cli->outbuf,smb_nt_TotalDataCount, ldata);
-       SIVAL(cli->outbuf,smb_nt_MaxParameterCount, mparam);
-       SIVAL(cli->outbuf,smb_nt_MaxDataCount, mdata);
-       SIVAL(cli->outbuf,smb_nt_ParameterCount, this_lparam);
-       SIVAL(cli->outbuf,smb_nt_ParameterOffset, smb_offset(outparam,cli->outbuf));
-       SIVAL(cli->outbuf,smb_nt_DataCount, this_ldata);
-       SIVAL(cli->outbuf,smb_nt_DataOffset, smb_offset(outdata,cli->outbuf));
-       SIVAL(cli->outbuf,smb_nt_SetupCount, lsetup);
-       SIVAL(cli->outbuf,smb_nt_Function, function);
-       for (i=0;i<lsetup;i++)          /* setup[] */
-               SSVAL(cli->outbuf,smb_nt_SetupStart+i*2,setup[i]);
-
-       if (this_lparam)                        /* param[] */
-               memcpy(outparam,param,this_lparam);
-       if (this_ldata)                 /* data[] */
-               memcpy(outdata,data,this_ldata);
-
-       cli_setup_bcc(cli, outdata+this_ldata);
-
-       show_msg(cli->outbuf);
-       if (!cli_send_smb(cli)) {
-               return False;
-       }
-
-       /* Note we're in a trans state. Save the sequence
-        * numbers for replies. */
-       client_set_trans_sign_state_on(cli, mid);
-
-       if (this_ldata < ldata || this_lparam < lparam) {
-               /* receive interim response */
-               if (!cli_receive_smb(cli) || cli_is_error(cli)) {
-                       client_set_trans_sign_state_off(cli, mid);
-                       return(False);
-               }
-
-               tot_data = this_ldata;
-               tot_param = this_lparam;
-
-               while (tot_data < ldata || tot_param < lparam)  {
-                       this_lparam = MIN(lparam-tot_param,cli->max_xmit - 500); /* hack */
-                       this_ldata = MIN(ldata-tot_data,cli->max_xmit - (500+this_lparam));
-
-                       cli_set_message(cli->outbuf,18,0,True);
-                       SCVAL(cli->outbuf,smb_com,SMBnttranss);
-
-                       /* XXX - these should probably be aligned */
-                       outparam = smb_buf(cli->outbuf);
-                       outdata = outparam+this_lparam;
-
-                       /* secondary request */
-                       SIVAL(cli->outbuf,smb_nts_TotalParameterCount,lparam);
-                       SIVAL(cli->outbuf,smb_nts_TotalDataCount,ldata);
-                       SIVAL(cli->outbuf,smb_nts_ParameterCount,this_lparam);
-                       SIVAL(cli->outbuf,smb_nts_ParameterOffset,smb_offset(outparam,cli->outbuf));
-                       SIVAL(cli->outbuf,smb_nts_ParameterDisplacement,tot_param);
-                       SIVAL(cli->outbuf,smb_nts_DataCount,this_ldata);
-                       SIVAL(cli->outbuf,smb_nts_DataOffset,smb_offset(outdata,cli->outbuf));
-                       SIVAL(cli->outbuf,smb_nts_DataDisplacement,tot_data);
-                       if (this_lparam)                        /* param[] */
-                               memcpy(outparam,param+tot_param,this_lparam);
-                       if (this_ldata)                 /* data[] */
-                               memcpy(outdata,data+tot_data,this_ldata);
-                       cli_setup_bcc(cli, outdata+this_ldata);
-
-                       /*
-                        * Save the mid we're using. We need this for finding
-                        * signing replies.
-                        */
-                       mid = cli->mid;
-
-                       show_msg(cli->outbuf);
-
-                       if (!cli_send_smb(cli)) {
-                               client_set_trans_sign_state_off(cli, mid);
-                               return False;
-                       }
-
-                       /* Ensure we use the same mid for the secondaries. */
-                       cli->mid = mid;
-
-                       tot_data += this_ldata;
-                       tot_param += this_lparam;
-               }
-       }
-
-       return(True);
-}
-
-/****************************************************************************
- Receive a SMB nttrans response allocating the necessary memory.
-****************************************************************************/
-
-bool cli_receive_nt_trans(struct cli_state *cli,
-                         char **param, unsigned int *param_len,
-                         char **data, unsigned int *data_len)
-{
-       unsigned int total_data=0;
-       unsigned int total_param=0;
-       unsigned int this_data,this_param;
-       uint8 eclass;
-       uint32 ecode;
-       bool ret = False;
-
-       *data_len = *param_len = 0;
-
-       if (!cli_receive_smb(cli)) {
-               return False;
-       }
-
-       show_msg(cli->inbuf);
-
-       /* sanity check */
-       if (CVAL(cli->inbuf,smb_com) != SMBnttrans) {
-               DEBUG(0,("Expected SMBnttrans response, got command 0x%02x\n",
-                        CVAL(cli->inbuf,smb_com)));
-               return(False);
-       }
-
-       /*
-        * An NT RPC pipe call can return ERRDOS, ERRmoredata
-        * to a trans call. This is not an error and should not
-        * be treated as such.
-        */
-       if (cli_is_dos_error(cli)) {
-                cli_dos_error(cli, &eclass, &ecode);
-               if (!(eclass == ERRDOS && ecode == ERRmoredata)) {
-                       goto out;
-               }
-       }
-
-       /*
-        * Likewise for NT_STATUS_BUFFER_TOO_SMALL
-        */
-       if (cli_is_nt_error(cli)) {
-               if (!NT_STATUS_EQUAL(cli_nt_error(cli),
-                                    NT_STATUS_BUFFER_TOO_SMALL)) {
-                       goto out;
-               }
-       }
-
-       /* parse out the lengths */
-       total_data = IVAL(cli->inbuf,smb_ntr_TotalDataCount);
-       total_param = IVAL(cli->inbuf,smb_ntr_TotalParameterCount);
-       /* Only allow 16 megs. */
-       if (total_param > 16*1024*1024) {
-               DEBUG(0,("cli_receive_nt_trans: param buffer too large %d\n",
-                                       total_param));
-               goto out;
-       }
-       if (total_data > 16*1024*1024) {
-               DEBUG(0,("cli_receive_nt_trans: data buffer too large %d\n",
-                                       total_data));
-               goto out;
-       }
-
-       /* allocate it */
-       if (total_data) {
-               /* We know adding 2 is safe as total_data is less
-                * than 16mb (above). */
-               *data = (char *)SMB_REALLOC(*data,total_data+2);
-               if (!(*data)) {
-                       DEBUG(0,("cli_receive_nt_trans: failed to enlarge data buffer to %d\n",total_data));
-                       goto out;
-               }
-       }
-
-       if (total_param) {
-               /* We know adding 2 is safe as total_param is less
-                * than 16mb (above). */
-               *param = (char *)SMB_REALLOC(*param,total_param+2);
-               if (!(*param)) {
-                       DEBUG(0,("cli_receive_nt_trans: failed to enlarge param buffer to %d\n", total_param));
-                       goto out;
-               }
-       }
-
-       while (1)  {
-               this_data = SVAL(cli->inbuf,smb_ntr_DataCount);
-               this_param = SVAL(cli->inbuf,smb_ntr_ParameterCount);
-
-               if (this_data + *data_len > total_data ||
-                   this_param + *param_len > total_param) {
-                       DEBUG(1,("Data overflow in cli_receive_nt_trans\n"));
-                       goto out;
-               }
-
-               if (this_data + *data_len < this_data ||
-                               this_data + *data_len < *data_len ||
-                               this_param + *param_len < this_param ||
-                               this_param + *param_len < *param_len) {
-                       DEBUG(1,("Data overflow in cli_receive_nt_trans\n"));
-                       goto out;
-               }
-
-               if (this_data) {
-                       unsigned int data_offset_out = SVAL(cli->inbuf,smb_ntr_DataDisplacement);
-                       unsigned int data_offset_in = SVAL(cli->inbuf,smb_ntr_DataOffset);
-
-                       if (data_offset_out > total_data ||
-                                       data_offset_out + this_data > total_data ||
-                                       data_offset_out + this_data < data_offset_out ||
-                                       data_offset_out + this_data < this_data) {
-                               DEBUG(1,("Data overflow in cli_receive_nt_trans\n"));
-                               goto out;
-                       }
-                       if (data_offset_in > cli->bufsize ||
-                                       data_offset_in + this_data >  cli->bufsize ||
-                                       data_offset_in + this_data < data_offset_in ||
-                                       data_offset_in + this_data < this_data) {
-                               DEBUG(1,("Data overflow in cli_receive_nt_trans\n"));
-                               goto out;
-                       }
-
-                       memcpy(*data + data_offset_out, smb_base(cli->inbuf) + data_offset_in, this_data);
-               }
-
-               if (this_param) {
-                       unsigned int param_offset_out = SVAL(cli->inbuf,smb_ntr_ParameterDisplacement);
-                       unsigned int param_offset_in = SVAL(cli->inbuf,smb_ntr_ParameterOffset);
-
-                       if (param_offset_out > total_param ||
-                                       param_offset_out + this_param > total_param ||
-                                       param_offset_out + this_param < param_offset_out ||
-                                       param_offset_out + this_param < this_param) {
-                               DEBUG(1,("Param overflow in cli_receive_nt_trans\n"));
-                               goto out;
-                       }
-                       if (param_offset_in > cli->bufsize ||
-                                       param_offset_in + this_param >  cli->bufsize ||
-                                       param_offset_in + this_param < param_offset_in ||
-                                       param_offset_in + this_param < this_param) {
-                               DEBUG(1,("Param overflow in cli_receive_nt_trans\n"));
-                               goto out;
-                       }
-
-                       memcpy(*param + param_offset_out, smb_base(cli->inbuf) + param_offset_in, this_param);
-               }
-
-               *data_len += this_data;
-               *param_len += this_param;
-
-               if (total_data <= *data_len && total_param <= *param_len) {
-                       ret = True;
-                       break;
-               }
-
-               if (!cli_receive_smb(cli)) {
-                       goto out;
-               }
-
-               show_msg(cli->inbuf);
-
-               /* sanity check */
-               if (CVAL(cli->inbuf,smb_com) != SMBnttrans) {
-                       DEBUG(0,("Expected SMBnttrans response, got command 0x%02x\n",
-                                CVAL(cli->inbuf,smb_com)));
-                       goto out;
-               }
-               if (cli_is_dos_error(cli)) {
-                        cli_dos_error(cli, &eclass, &ecode);
-                       if(!(eclass == ERRDOS && ecode == ERRmoredata)) {
-                               goto out;
-                       }
-               }
-               /*
-                * Likewise for NT_STATUS_BUFFER_TOO_SMALL
-                */
-               if (cli_is_nt_error(cli)) {
-                       if (!NT_STATUS_EQUAL(cli_nt_error(cli),
-                                            NT_STATUS_BUFFER_TOO_SMALL)) {
-                               goto out;
-                       }
-               }
-
-               /* parse out the total lengths again - they can shrink! */
-               if (IVAL(cli->inbuf,smb_ntr_TotalDataCount) < total_data)
-                       total_data = IVAL(cli->inbuf,smb_ntr_TotalDataCount);
-               if (IVAL(cli->inbuf,smb_ntr_TotalParameterCount) < total_param)
-                       total_param = IVAL(cli->inbuf,smb_ntr_TotalParameterCount);
-
-               if (total_data <= *data_len && total_param <= *param_len) {
-                       ret = True;
-                       break;
-               }
-       }
-
-  out:
-
-       if (ret) {
-               /* Ensure the last 2 bytes of param and data are 2 null
-                * bytes. These are malloc'ed, but not included in any
-                * length counts. This allows cli_XX string reading functions
-                * to safely null terminate. */
-               if (total_data) {
-                       SSVAL(*data,total_data,0);
-               }
-               if (total_param) {
-                       SSVAL(*param,total_param,0);
-               }
-       }
-
-       client_set_trans_sign_state_off(cli, SVAL(cli->inbuf,smb_mid));
-       return ret;
-}
+#include "libsmb/libsmb.h"
+#include "../lib/util/tevent_ntstatus.h"
+#include "async_smb.h"
 
 struct trans_recvblob {
        uint8_t *data;
@@ -712,7 +32,10 @@ struct cli_trans_state {
        struct event_context *ev;
        uint8_t cmd;
        uint16_t mid;
+       uint32_t seqnum;
        const char *pipe_name;
+       uint8_t *pipe_name_conv;
+       size_t pipe_name_conv_len;
        uint16_t fid;
        uint16_t function;
        int flags;
@@ -727,283 +50,25 @@ struct cli_trans_state {
        uint16_t *rsetup;
        struct trans_recvblob rparam;
        struct trans_recvblob rdata;
+       uint16_t recv_flags2;
 
-       TALLOC_CTX *secondary_request_ctx;
+       struct iovec iov[6];
+       uint8_t pad[4];
+       uint8_t zero_pad[4];
+       uint16_t vwv[32];
 };
 
-static void cli_trans_recv_helper(struct async_req *req);
-
-static struct async_req *cli_ship_trans(TALLOC_CTX *mem_ctx,
-                                       struct cli_trans_state *state)
-{
-       TALLOC_CTX *frame;
-       struct async_req *result = NULL;
-       struct cli_request *cli_req;
-       uint8_t wct;
-       uint16_t *vwv;
-       uint8_t *bytes = NULL;
-       uint16_t param_offset;
-       uint16_t this_param = 0;
-       uint16_t this_data = 0;
-       uint32_t useable_space;
-       uint8_t cmd;
-
-       frame = talloc_stackframe();
-
-       cmd = state->cmd;
-
-       if ((state->param_sent != 0) || (state->data_sent != 0)) {
-               /* The secondary commands are one after the primary ones */
-               cmd += 1;
-       }
-
-       param_offset = smb_size - 4;
-
-       switch (cmd) {
-       case SMBtrans:
-               bytes = TALLOC_ZERO_P(talloc_tos(), uint8_t); /* padding */
-               if (bytes == NULL) {
-                       goto fail;
-               }
-               bytes = smb_bytes_push_str(
-                       bytes, (state->cli->capabilities & CAP_UNICODE) != 0,
-                       state->pipe_name);
-               if (bytes == NULL) {
-                       goto fail;
-               }
-               wct = 14 + state->num_setup;
-               param_offset += talloc_get_size(bytes);
-               break;
-       case SMBtrans2:
-               bytes = TALLOC_ARRAY(talloc_tos(), uint8_t, 3); /* padding */
-               if (bytes == NULL) {
-                       goto fail;
-               }
-               bytes[0] = 0;
-               bytes[1] = 'D'; /* Copy this from "old" 3.0 behaviour */
-               bytes[2] = ' ';
-               wct = 14 + state->num_setup;
-               param_offset += talloc_get_size(bytes);
-               break;
-       case SMBtranss:
-               wct = 8;
-               break;
-       case SMBtranss2:
-               wct = 9;
-               break;
-       case SMBnttrans:
-               wct = 19 + state->num_setup;
-               break;
-       case SMBnttranss:
-               wct = 18;
-               break;
-       default:
-               goto fail;
-       }
-
-       useable_space = state->cli->max_xmit - smb_size - sizeof(uint16_t)*wct;
-
-       if (state->param_sent < state->num_param) {
-               this_param = MIN(state->num_param - state->param_sent,
-                                useable_space);
-       }
-
-       if (state->data_sent < state->num_data) {
-               this_data = MIN(state->num_data - state->data_sent,
-                               useable_space - this_param);
-       }
-
-       vwv = TALLOC_ARRAY(talloc_tos(), uint16_t, wct);
-       if (vwv == NULL) {
-               goto fail;
-       }
-       param_offset += wct * sizeof(uint16_t);
-
-       DEBUG(10, ("num_setup=%u, max_setup=%u, "
-                  "param_total=%u, this_param=%u, max_param=%u, "
-                  "data_total=%u, this_data=%u, max_data=%u, "
-                  "param_offset=%u, param_disp=%u, data_disp=%u\n",
-                  (unsigned)state->num_setup, (unsigned)state->max_setup,
-                  (unsigned)state->num_param, (unsigned)this_param,
-                  (unsigned)state->rparam.max,
-                  (unsigned)state->num_data, (unsigned)this_data,
-                  (unsigned)state->rdata.max,
-                  (unsigned)param_offset,
-                  (unsigned)state->param_sent, (unsigned)state->data_sent));
-
-       switch (cmd) {
-       case SMBtrans:
-       case SMBtrans2:
-               SSVAL(vwv + 0, 0, state->num_param);
-               SSVAL(vwv + 1, 0, state->num_data);
-               SSVAL(vwv + 2, 0, state->rparam.max);
-               SSVAL(vwv + 3, 0, state->rdata.max);
-               SCVAL(vwv + 4, 0, state->max_setup);
-               SCVAL(vwv + 4, 1, 0);   /* reserved */
-               SSVAL(vwv + 5, 0, state->flags);
-               SIVAL(vwv + 6, 0, 0);   /* timeout */
-               SSVAL(vwv + 8, 0, 0);   /* reserved */
-               SSVAL(vwv + 9, 0, this_param);
-               SSVAL(vwv +10, 0, param_offset);
-               SSVAL(vwv +11, 0, this_data);
-               SSVAL(vwv +12, 0, param_offset + this_param);
-               SCVAL(vwv +13, 0, state->num_setup);
-               SCVAL(vwv +13, 1, 0);   /* reserved */
-               memcpy(vwv + 14, state->setup,
-                      sizeof(uint16_t) * state->num_setup);
-               break;
-       case SMBtranss:
-       case SMBtranss2:
-               SSVAL(vwv + 0, 0, state->num_param);
-               SSVAL(vwv + 1, 0, state->num_data);
-               SSVAL(vwv + 2, 0, this_param);
-               SSVAL(vwv + 3, 0, param_offset);
-               SSVAL(vwv + 4, 0, state->param_sent);
-               SSVAL(vwv + 5, 0, this_data);
-               SSVAL(vwv + 6, 0, param_offset + this_param);
-               SSVAL(vwv + 7, 0, state->data_sent);
-               if (cmd == SMBtranss2) {
-                       SSVAL(vwv + 8, 0, state->fid);
-               }
-               break;
-       case SMBnttrans:
-               SCVAL(vwv,  0, state->max_setup);
-               SSVAL(vwv,  1, 0); /* reserved */
-               SIVAL(vwv,  3, state->num_param);
-               SIVAL(vwv,  7, state->num_data);
-               SIVAL(vwv, 11, state->rparam.max);
-               SIVAL(vwv, 15, state->rdata.max);
-               SIVAL(vwv, 19, this_param);
-               SIVAL(vwv, 23, param_offset);
-               SIVAL(vwv, 27, this_data);
-               SIVAL(vwv, 31, param_offset + this_param);
-               SCVAL(vwv, 35, state->num_setup);
-               SSVAL(vwv, 36, state->function);
-               memcpy(vwv + 19, state->setup,
-                      sizeof(uint16_t) * state->num_setup);
-               break;
-       case SMBnttranss:
-               SSVAL(vwv,  0, 0); /* reserved */
-               SCVAL(vwv,  2, 0); /* reserved */
-               SIVAL(vwv,  3, state->num_param);
-               SIVAL(vwv,  7, state->num_data);
-               SIVAL(vwv, 11, this_param);
-               SIVAL(vwv, 15, param_offset);
-               SIVAL(vwv, 19, state->param_sent);
-               SIVAL(vwv, 23, this_data);
-               SIVAL(vwv, 27, param_offset + this_param);
-               SIVAL(vwv, 31, state->data_sent);
-               SCVAL(vwv, 35, 0); /* reserved */
-               break;
-       }
-
-       bytes = (uint8_t *)talloc_append_blob(
-               talloc_tos(), bytes,
-               data_blob_const(state->param + state->param_sent, this_param));
-       if (bytes == NULL) {
-               goto fail;
-       }
-       state->param_sent += this_param;
-
-       bytes = (uint8_t *)talloc_append_blob(
-               talloc_tos(), bytes,
-               data_blob_const(state->data + state->data_sent, this_data));
-       if (bytes == NULL) {
-               goto fail;
-       }
-       state->data_sent += this_data;
-
-       if ((cmd == SMBtrans) || (cmd == SMBtrans2) || (cmd == SMBnttrans)) {
-               /*
-                * Primary request, retrieve our mid
-                */
-               result = cli_request_send(mem_ctx, state->ev, state->cli,
-                                         cmd, 0, wct, vwv,
-                                         talloc_get_size(bytes), bytes);
-               if (result == NULL) {
-                       goto fail;
-               }
-               cli_req = talloc_get_type_abort(result->private_data,
-                                               struct cli_request);
-               state->mid = cli_req->mid;
-       } else {
-               uint16_t num_bytes = talloc_get_size(bytes);
-               /*
-                * Secondary request, we have to fix up the mid. Thus we do
-                * the chain_cork/chain/uncork ourselves.
-                */
-               if (!cli_chain_cork(state->cli, state->ev,
-                                   wct * sizeof(uint16_t) + num_bytes + 3)) {
-                       goto fail;
-               }
-               result = cli_request_send(mem_ctx, state->ev, state->cli,
-                                         cmd, 0, wct, vwv, num_bytes, bytes);
-               if (result == NULL) {
-                       goto fail;
-               }
-               cli_req = talloc_get_type_abort(result->private_data,
-                                               struct cli_request);
-               cli_req->recv_helper.fn = cli_trans_recv_helper;
-               cli_req->recv_helper.priv = state;
-               cli_req->mid = state->mid;
-               client_set_trans_sign_state_off(state->cli, state->mid);
-               cli_chain_uncork(state->cli);
-       }
-
-       client_set_trans_sign_state_on(state->cli, state->mid);
-
- fail:
-       TALLOC_FREE(frame);
-       return result;
-}
-
-static void cli_trans_ship_rest(struct async_req *req,
-                               struct cli_trans_state *state)
-{
-       state->secondary_request_ctx = talloc_new(state);
-       if (state->secondary_request_ctx == NULL) {
-               async_req_error(req, NT_STATUS_NO_MEMORY);
-               return;
-       }
-
-       while ((state->param_sent < state->num_param)
-              || (state->data_sent < state->num_data)) {
-               struct async_req *cli_req;
-
-               cli_req = cli_ship_trans(state->secondary_request_ctx, state);
-               if (cli_req == NULL) {
-                       async_req_error(req, NT_STATUS_NO_MEMORY);
-                       return;
-               }
-       }
-}
-
-static NTSTATUS cli_pull_trans(struct async_req *req,
-                              struct cli_request *cli_req,
-                              uint8_t smb_cmd, bool expect_first_reply,
-                              uint8_t *pnum_setup, uint16_t **psetup,
-                              uint32_t *ptotal_param, uint32_t *pnum_param,
-                              uint32_t *pparam_disp, uint8_t **pparam,
-                              uint32_t *ptotal_data, uint32_t *pnum_data,
-                              uint32_t *pdata_disp, uint8_t **pdata)
-{
-       uint32_t param_ofs, data_ofs;
-       uint8_t wct;
-       uint16_t *vwv;
-       uint16_t num_bytes;
-       uint8_t *bytes;
-       NTSTATUS status;
-
-       status = cli_pull_reply(req, &wct, &vwv, &num_bytes, &bytes);
-
-       /*
-        * We can receive something like STATUS_MORE_ENTRIES, so don't use
-        * !NT_STATUS_IS_OK(status) here.
-        */
-
-       if (NT_STATUS_IS_ERR(status)) {
-               return status;
-       }
+static NTSTATUS cli_pull_trans(uint8_t *inbuf,
+                              uint8_t wct, uint16_t *vwv,
+                              uint16_t num_bytes, uint8_t *bytes,
+                              uint8_t smb_cmd, bool expect_first_reply,
+                              uint8_t *pnum_setup, uint16_t **psetup,
+                              uint32_t *ptotal_param, uint32_t *pnum_param,
+                              uint32_t *pparam_disp, uint8_t **pparam,
+                              uint32_t *ptotal_data, uint32_t *pnum_data,
+                              uint32_t *pdata_disp, uint8_t **pdata)
+{
+       uint32_t param_ofs, data_ofs;
 
        if (expect_first_reply) {
                if ((wct != 0) || (num_bytes != 0)) {
@@ -1059,15 +124,15 @@ static NTSTATUS cli_pull_trans(struct async_req *req,
         * length. Likewise for param_ofs/param_disp.
         */
 
-       if (trans_oob(smb_len(cli_req->inbuf), param_ofs, *pnum_param)
+       if (trans_oob(smb_len(inbuf), param_ofs, *pnum_param)
            || trans_oob(*ptotal_param, *pparam_disp, *pnum_param)
-           || trans_oob(smb_len(cli_req->inbuf), data_ofs, *pnum_data)
+           || trans_oob(smb_len(inbuf), data_ofs, *pnum_data)
            || trans_oob(*ptotal_data, *pdata_disp, *pnum_data)) {
                return NT_STATUS_INVALID_NETWORK_RESPONSE;
        }
 
-       *pparam = (uint8_t *)cli_req->inbuf + 4 + param_ofs;
-       *pdata = (uint8_t *)cli_req->inbuf + 4 + data_ofs;
+       *pparam = (uint8_t *)inbuf + 4 + param_ofs;
+       *pdata = (uint8_t *)inbuf + 4 + data_ofs;
 
        return NT_STATUS_OK;
 }
@@ -1082,7 +147,7 @@ static NTSTATUS cli_trans_pull_blob(TALLOC_CTX *mem_ctx,
                        return NT_STATUS_INVALID_NETWORK_RESPONSE;
                }
                blob->total = total;
-               blob->data = TALLOC_ARRAY(mem_ctx, uint8_t, total);
+               blob->data = talloc_array(mem_ctx, uint8_t, total);
                if (blob->data == NULL) {
                        return NT_STATUS_NO_MEMORY;
                }
@@ -1100,114 +165,218 @@ static NTSTATUS cli_trans_pull_blob(TALLOC_CTX *mem_ctx,
        return NT_STATUS_OK;
 }
 
-static void cli_trans_recv_helper(struct async_req *req)
+static void cli_trans_format(struct cli_trans_state *state, uint8_t *pwct,
+                            int *piov_count)
 {
-       struct cli_request *cli_req = talloc_get_type_abort(
-               req->private_data, struct cli_request);
-       struct cli_trans_state *state = talloc_get_type_abort(
-               cli_req->recv_helper.priv, struct cli_trans_state);
-       uint8_t num_setup       = 0;
-       uint16_t *setup         = NULL;
-       uint32_t total_param    = 0;
-       uint32_t num_param      = 0;
-       uint32_t param_disp     = 0;
-       uint32_t total_data     = 0;
-       uint32_t num_data       = 0;
-       uint32_t data_disp      = 0;
-       uint8_t *param          = NULL;
-       uint8_t *data           = NULL;
-       bool sent_all;
-       NTSTATUS status;
+       uint8_t wct = 0;
+       struct iovec *iov = state->iov;
+       uint8_t *pad = state->pad;
+       uint16_t *vwv = state->vwv;
+       uint32_t param_offset;
+       uint32_t this_param = 0;
+       uint32_t param_pad;
+       uint32_t data_offset;
+       uint32_t this_data = 0;
+       uint32_t data_pad;
+       uint32_t useable_space;
+       uint8_t cmd;
 
-       sent_all = (state->param_sent == state->num_param)
-               && (state->data_sent == state->num_data);
+       cmd = state->cmd;
 
-       status = cli_pull_trans(
-               req, cli_req, state->cmd, !sent_all, &num_setup, &setup,
-               &total_param, &num_param, &param_disp, &param,
-               &total_data, &num_data, &data_disp, &data);
+       if ((state->param_sent != 0) || (state->data_sent != 0)) {
+               /* The secondary commands are one after the primary ones */
+               cmd += 1;
+       }
 
-       /*
-        * We can receive something like STATUS_MORE_ENTRIES, so don't use
-        * !NT_STATUS_IS_OK(status) here.
-        */
+       param_offset = smb_size - 4;
 
-       if (NT_STATUS_IS_ERR(status)) {
-               async_req_error(req, status);
-               return;
+       switch (cmd) {
+       case SMBtrans:
+               pad[0] = 0;
+               iov[0].iov_base = (void *)pad;
+               iov[0].iov_len = 1;
+               iov[1].iov_base = (void *)state->pipe_name_conv;
+               iov[1].iov_len = state->pipe_name_conv_len;
+               wct = 14 + state->num_setup;
+               param_offset += iov[0].iov_len + iov[1].iov_len;
+               iov += 2;
+               break;
+       case SMBtrans2:
+               pad[0] = 0;
+               pad[1] = 'D'; /* Copy this from "old" 3.0 behaviour */
+               pad[2] = ' ';
+               iov[0].iov_base = (void *)pad;
+               iov[0].iov_len = 3;
+               wct = 14 + state->num_setup;
+               param_offset += 3;
+               iov += 1;
+               break;
+       case SMBtranss:
+               wct = 8;
+               break;
+       case SMBtranss2:
+               wct = 9;
+               break;
+       case SMBnttrans:
+               wct = 19 + state->num_setup;
+               break;
+       case SMBnttranss:
+               wct = 18;
+               break;
        }
 
-       if (!sent_all) {
-               cli_trans_ship_rest(req, state);
-               return;
-       }
+       param_offset += wct * sizeof(uint16_t);
+       useable_space = state->cli->max_xmit - param_offset;
 
-       /*
-        * We've just received a real response. This means that we don't need
-        * the secondary cli_request structures anymore, they have all been
-        * shipped to the server.
-        */
-       TALLOC_FREE(state->secondary_request_ctx);
+       param_pad = param_offset % 4;
+       if (param_pad > 0) {
+               param_pad = MIN(param_pad, useable_space);
+               iov[0].iov_base = (void *)state->zero_pad;
+               iov[0].iov_len = param_pad;
+               iov += 1;
+               param_offset += param_pad;
+       }
+       useable_space = state->cli->max_xmit - param_offset;
 
-       if (num_setup != 0) {
-               TALLOC_FREE(state->rsetup);
-               state->rsetup = (uint16_t *)TALLOC_MEMDUP(
-                       state, setup, sizeof(uint16_t) * num_setup);
-               if (state->rsetup == NULL) {
-                       async_req_error(req, NT_STATUS_NO_MEMORY);
-                       return;
-               }
+       if (state->param_sent < state->num_param) {
+               this_param = MIN(state->num_param - state->param_sent,
+                                useable_space);
+               iov[0].iov_base = (void *)(state->param + state->param_sent);
+               iov[0].iov_len = this_param;
+               iov += 1;
        }
 
-       status = cli_trans_pull_blob(
-               state, &state->rparam, total_param, num_param, param,
-               param_disp);
+       data_offset = param_offset + this_param;
+       useable_space = state->cli->max_xmit - data_offset;
 
-       if (!NT_STATUS_IS_OK(status)) {
-               DEBUG(10, ("Pulling params failed: %s\n", nt_errstr(status)));
-               async_req_error(req, status);
-               return;
+       data_pad = data_offset % 4;
+       if (data_pad > 0) {
+               data_pad = MIN(data_pad, useable_space);
+               iov[0].iov_base = (void *)state->zero_pad;
+               iov[0].iov_len = data_pad;
+               iov += 1;
+               data_offset += data_pad;
        }
+       useable_space = state->cli->max_xmit - data_offset;
 
-       status = cli_trans_pull_blob(
-               state, &state->rdata, total_data, num_data, data,
-               data_disp);
-
-       if (!NT_STATUS_IS_OK(status)) {
-               DEBUG(10, ("Pulling data failed: %s\n", nt_errstr(status)));
-               async_req_error(req, status);
-               return;
+       if (state->data_sent < state->num_data) {
+               this_data = MIN(state->num_data - state->data_sent,
+                               useable_space);
+               iov[0].iov_base = (void *)(state->data + state->data_sent);
+               iov[0].iov_len = this_data;
+               iov += 1;
        }
 
-       if ((state->rparam.total == state->rparam.received)
-           && (state->rdata.total == state->rdata.received)) {
-               client_set_trans_sign_state_off(state->cli, state->mid);
-               async_req_done(req);
+       DEBUG(10, ("num_setup=%u, max_setup=%u, "
+                  "param_total=%u, this_param=%u, max_param=%u, "
+                  "data_total=%u, this_data=%u, max_data=%u, "
+                  "param_offset=%u, param_pad=%u, param_disp=%u, "
+                  "data_offset=%u, data_pad=%u, data_disp=%u\n",
+                  (unsigned)state->num_setup, (unsigned)state->max_setup,
+                  (unsigned)state->num_param, (unsigned)this_param,
+                  (unsigned)state->rparam.max,
+                  (unsigned)state->num_data, (unsigned)this_data,
+                  (unsigned)state->rdata.max,
+                  (unsigned)param_offset, (unsigned)param_pad,
+                  (unsigned)state->param_sent,
+                  (unsigned)data_offset, (unsigned)data_pad,
+                  (unsigned)state->data_sent));
+
+       switch (cmd) {
+       case SMBtrans:
+       case SMBtrans2:
+               SSVAL(vwv + 0, 0, state->num_param);
+               SSVAL(vwv + 1, 0, state->num_data);
+               SSVAL(vwv + 2, 0, state->rparam.max);
+               SSVAL(vwv + 3, 0, state->rdata.max);
+               SCVAL(vwv + 4, 0, state->max_setup);
+               SCVAL(vwv + 4, 1, 0);   /* reserved */
+               SSVAL(vwv + 5, 0, state->flags);
+               SIVAL(vwv + 6, 0, 0);   /* timeout */
+               SSVAL(vwv + 8, 0, 0);   /* reserved */
+               SSVAL(vwv + 9, 0, this_param);
+               SSVAL(vwv +10, 0, param_offset);
+               SSVAL(vwv +11, 0, this_data);
+               SSVAL(vwv +12, 0, data_offset);
+               SCVAL(vwv +13, 0, state->num_setup);
+               SCVAL(vwv +13, 1, 0);   /* reserved */
+               memcpy(vwv + 14, state->setup,
+                      sizeof(uint16_t) * state->num_setup);
+               break;
+       case SMBtranss:
+       case SMBtranss2:
+               SSVAL(vwv + 0, 0, state->num_param);
+               SSVAL(vwv + 1, 0, state->num_data);
+               SSVAL(vwv + 2, 0, this_param);
+               SSVAL(vwv + 3, 0, param_offset);
+               SSVAL(vwv + 4, 0, state->param_sent);
+               SSVAL(vwv + 5, 0, this_data);
+               SSVAL(vwv + 6, 0, data_offset);
+               SSVAL(vwv + 7, 0, state->data_sent);
+               if (cmd == SMBtranss2) {
+                       SSVAL(vwv + 8, 0, state->fid);
+               }
+               break;
+       case SMBnttrans:
+               SCVAL(vwv + 0, 0, state->max_setup);
+               SSVAL(vwv + 0, 1, 0); /* reserved */
+               SIVAL(vwv + 1, 1, state->num_param);
+               SIVAL(vwv + 3, 1, state->num_data);
+               SIVAL(vwv + 5, 1, state->rparam.max);
+               SIVAL(vwv + 7, 1, state->rdata.max);
+               SIVAL(vwv + 9, 1, this_param);
+               SIVAL(vwv +11, 1, param_offset);
+               SIVAL(vwv +13, 1, this_data);
+               SIVAL(vwv +15, 1, data_offset);
+               SCVAL(vwv +17, 1, state->num_setup);
+               SSVAL(vwv +18, 0, state->function);
+               memcpy(vwv + 19, state->setup,
+                      sizeof(uint16_t) * state->num_setup);
+               break;
+       case SMBnttranss:
+               SSVAL(vwv + 0, 0, 0); /* reserved */
+               SCVAL(vwv + 1, 0, 0); /* reserved */
+               SIVAL(vwv + 1, 1, state->num_param);
+               SIVAL(vwv + 3, 1, state->num_data);
+               SIVAL(vwv + 5, 1, this_param);
+               SIVAL(vwv + 7, 1, param_offset);
+               SIVAL(vwv + 9, 1, state->param_sent);
+               SIVAL(vwv +11, 1, this_data);
+               SIVAL(vwv +13, 1, data_offset);
+               SIVAL(vwv +15, 1, state->data_sent);
+               SCVAL(vwv +17, 1, 0); /* reserved */
+               break;
        }
+
+       state->param_sent += this_param;
+       state->data_sent += this_data;
+
+       *pwct = wct;
+       *piov_count = iov - state->iov;
 }
 
-struct async_req *cli_trans_send(
+static void cli_trans_done(struct tevent_req *subreq);
+
+struct tevent_req *cli_trans_send(
        TALLOC_CTX *mem_ctx, struct event_context *ev,
-       struct cli_state *cli, uint8_t trans_cmd,
+       struct cli_state *cli, uint8_t cmd,
        const char *pipe_name, uint16_t fid, uint16_t function, int flags,
        uint16_t *setup, uint8_t num_setup, uint8_t max_setup,
        uint8_t *param, uint32_t num_param, uint32_t max_param,
        uint8_t *data, uint32_t num_data, uint32_t max_data)
 {
-       struct async_req *req;
-       struct cli_request *cli_req;
+       struct tevent_req *req, *subreq;
        struct cli_trans_state *state;
+       int iov_count;
+       uint8_t wct;
+       NTSTATUS status;
 
-       /*
-        * We can't use it in a chained request chain, we'd get the offset
-        * calculations wrong.
-        */
-
-       if (cli_in_chain(cli)) {
+       req = tevent_req_create(mem_ctx, &state, struct cli_trans_state);
+       if (req == NULL) {
                return NULL;
        }
 
-       if ((trans_cmd == SMBtrans) || (trans_cmd == SMBtrans2)) {
+       if ((cmd == SMBtrans) || (cmd == SMBtrans2)) {
                if ((num_param > 0xffff) || (max_param > 0xffff)
                    || (num_data > 0xffff) || (max_data > 0xffff)) {
                        DEBUG(3, ("Attempt to send invalid trans2 request "
@@ -1215,113 +384,228 @@ struct async_req *cli_trans_send(
                                  (unsigned)num_setup,
                                  (unsigned)num_param, (unsigned)max_param,
                                  (unsigned)num_data, (unsigned)max_data));
-                       return NULL;
+                       tevent_req_nterror(req, NT_STATUS_INVALID_PARAMETER);
+                       return tevent_req_post(req, ev);
                }
        }
 
-       state = talloc(mem_ctx, struct cli_trans_state);
-       if (state == NULL) {
-               goto nomem;
+       /*
+        * The largest wct will be for nttrans (19+num_setup). Make sure we
+        * don't overflow state->vwv in cli_trans_format.
+        */
+
+       if ((num_setup + 19) > ARRAY_SIZE(state->vwv)) {
+               tevent_req_nterror(req, NT_STATUS_INVALID_PARAMETER);
+               return tevent_req_post(req, ev);
        }
 
        state->cli = cli;
        state->ev = ev;
-       state->cmd = trans_cmd;
+       state->cmd = cmd;
+       state->flags = flags;
        state->num_rsetup = 0;
        state->rsetup = NULL;
        ZERO_STRUCT(state->rparam);
        ZERO_STRUCT(state->rdata);
-       state->secondary_request_ctx = NULL;
 
-       if (trans_cmd == SMBtrans) {
-               state->pipe_name = talloc_strdup(state, pipe_name);
-               if (state->pipe_name == NULL) {
-                       goto nomem;
-               }
+       if ((pipe_name != NULL)
+           && (!convert_string_talloc(state, CH_UNIX,
+                                      cli_ucs2(cli) ? CH_UTF16LE : CH_DOS,
+                                      pipe_name, strlen(pipe_name) + 1,
+                                      &state->pipe_name_conv,
+                                      &state->pipe_name_conv_len))) {
+               tevent_req_nterror(req, NT_STATUS_NO_MEMORY);
+               return tevent_req_post(req, ev);
        }
-       if (trans_cmd == SMBtrans2) {
-               state->fid = fid;
+       state->fid = fid;       /* trans2 */
+       state->function = function; /* nttrans */
+
+       state->setup = setup;
+       state->num_setup = num_setup;
+       state->max_setup = max_setup;
+
+       state->param = param;
+       state->num_param = num_param;
+       state->param_sent = 0;
+       state->rparam.max = max_param;
+
+       state->data = data;
+       state->num_data = num_data;
+       state->data_sent = 0;
+       state->rdata.max = max_data;
+
+       cli_trans_format(state, &wct, &iov_count);
+
+       subreq = cli_smb_req_create(state, ev, cli, cmd, 0, wct, state->vwv,
+                                   iov_count, state->iov);
+       if (tevent_req_nomem(subreq, req)) {
+               return tevent_req_post(req, ev);
        }
-       if (trans_cmd == SMBnttrans) {
-               state->function = function;
+       status = cli_smb_req_send(subreq);
+       if (!NT_STATUS_IS_OK(status)) {
+               tevent_req_nterror(req, status);
+               return tevent_req_post(req, state->ev);
        }
+       tevent_req_set_callback(subreq, cli_trans_done, req);
 
-       state->flags = flags;
+       /*
+        * Now get the MID of the primary request
+        * and mark it as persistent. This means
+        * we will able to send and receive multiple
+        * SMB pdus using this MID in both directions
+        * (including correct SMB signing).
+        */
+       state->mid = cli_smb_req_mid(subreq);
+       cli_state_seqnum_persistent(cli, state->mid);
 
-       if (setup != NULL) {
-               state->setup = (uint16_t *)TALLOC_MEMDUP(
-                       state, setup, sizeof(*setup) * num_setup);
-               if (state->setup == NULL) {
-                       goto nomem;
-               }
-               state->num_setup = num_setup;
-       } else {
-               state->setup = NULL;
-               state->num_setup = 0;
+       return req;
+}
+
+static void cli_trans_done(struct tevent_req *subreq)
+{
+       struct tevent_req *req = tevent_req_callback_data(
+               subreq, struct tevent_req);
+       struct cli_trans_state *state = tevent_req_data(
+               req, struct cli_trans_state);
+       NTSTATUS status;
+       bool sent_all;
+       uint8_t wct;
+       uint16_t *vwv;
+       uint32_t num_bytes;
+       uint8_t *bytes;
+       uint8_t *inbuf;
+       uint8_t num_setup       = 0;
+       uint16_t *setup         = NULL;
+       uint32_t total_param    = 0;
+       uint32_t num_param      = 0;
+       uint32_t param_disp     = 0;
+       uint32_t total_data     = 0;
+       uint32_t num_data       = 0;
+       uint32_t data_disp      = 0;
+       uint8_t *param          = NULL;
+       uint8_t *data           = NULL;
+
+       status = cli_smb_recv(subreq, state, &inbuf, 0, &wct, &vwv,
+                             &num_bytes, &bytes);
+       /*
+        * Do not TALLOC_FREE(subreq) here, we might receive more than
+        * one response for the same mid.
+        */
+
+       /*
+        * We can receive something like STATUS_MORE_ENTRIES, so don't use
+        * !NT_STATUS_IS_OK(status) here.
+        */
+
+       if (NT_STATUS_IS_ERR(status)) {
+               goto fail;
        }
 
-       state->max_setup = max_setup;
+       sent_all = ((state->param_sent == state->num_param)
+                   && (state->data_sent == state->num_data));
 
-       if (param != NULL) {
-               state->param = (uint8_t *)TALLOC_MEMDUP(state, param,
-                                                       num_param);
-               if (state->param == NULL) {
-                       goto nomem;
-               }
-               state->num_param = num_param;
-       } else {
-               state->param = NULL;
-               state->num_param = 0;
+       status = cli_pull_trans(
+               inbuf, wct, vwv, num_bytes, bytes,
+               state->cmd, !sent_all, &num_setup, &setup,
+               &total_param, &num_param, &param_disp, &param,
+               &total_data, &num_data, &data_disp, &data);
+
+       if (!NT_STATUS_IS_OK(status)) {
+               goto fail;
        }
 
-       state->param_sent = 0;
-       state->rparam.max = max_param;
+       if (!sent_all) {
+               int iov_count;
 
-       if (data != NULL) {
-               state->data = (uint8_t *)TALLOC_MEMDUP(state, data, num_data);
-               if (state->data == NULL) {
-                       goto nomem;
+               TALLOC_FREE(subreq);
+
+               cli_trans_format(state, &wct, &iov_count);
+
+               subreq = cli_smb_req_create(state, state->ev, state->cli,
+                                           state->cmd + 1, 0, wct, state->vwv,
+                                           iov_count, state->iov);
+               if (tevent_req_nomem(subreq, req)) {
+                       return;
                }
-               state->num_data = num_data;
-       } else {
-               state->data = NULL;
-               state->num_data = 0;
+               cli_smb_req_set_mid(subreq, state->mid);
+
+               status = cli_smb_req_send(subreq);
+
+               if (!NT_STATUS_IS_OK(status)) {
+                       goto fail;
+               }
+               tevent_req_set_callback(subreq, cli_trans_done, req);
+               return;
        }
 
-       state->data_sent = 0;
-       state->rdata.max = max_data;
+       status = cli_trans_pull_blob(
+               state, &state->rparam, total_param, num_param, param,
+               param_disp);
 
-       req = cli_ship_trans(state, state);
-       if (req == NULL) {
-               goto nomem;
+       if (!NT_STATUS_IS_OK(status)) {
+               DEBUG(10, ("Pulling params failed: %s\n", nt_errstr(status)));
+               goto fail;
        }
 
-       cli_req = talloc_get_type_abort(req->private_data, struct cli_request);
-       cli_req->recv_helper.fn = cli_trans_recv_helper;
-       cli_req->recv_helper.priv = state;
+       status = cli_trans_pull_blob(
+               state, &state->rdata, total_data, num_data, data,
+               data_disp);
 
-       return req;
+       if (!NT_STATUS_IS_OK(status)) {
+               DEBUG(10, ("Pulling data failed: %s\n", nt_errstr(status)));
+               goto fail;
+       }
 
- nomem:
-       TALLOC_FREE(state);
-       return NULL;
+       if ((state->rparam.total == state->rparam.received)
+           && (state->rdata.total == state->rdata.received)) {
+               state->recv_flags2 = SVAL(inbuf, smb_flg2);
+               TALLOC_FREE(subreq);
+               cli_state_seqnum_remove(state->cli, state->mid);
+               tevent_req_done(req);
+               return;
+       }
+
+       TALLOC_FREE(inbuf);
+
+       if (!cli_smb_req_set_pending(subreq)) {
+               status = NT_STATUS_NO_MEMORY;
+               goto fail;
+       }
+       return;
+
+ fail:
+       cli_state_seqnum_remove(state->cli, state->mid);
+       TALLOC_FREE(subreq);
+       tevent_req_nterror(req, status);
 }
 
-NTSTATUS cli_trans_recv(struct async_req *req, TALLOC_CTX *mem_ctx,
-                       uint16_t **setup, uint8_t *num_setup,
-                       uint8_t **param, uint32_t *num_param,
-                       uint8_t **data, uint32_t *num_data)
+NTSTATUS cli_trans_recv(struct tevent_req *req, TALLOC_CTX *mem_ctx,
+                       uint16_t *recv_flags2,
+                       uint16_t **setup, uint8_t min_setup,
+                       uint8_t *num_setup,
+                       uint8_t **param, uint32_t min_param,
+                       uint32_t *num_param,
+                       uint8_t **data, uint32_t min_data,
+                       uint32_t *num_data)
 {
-       struct cli_request *cli_req = talloc_get_type_abort(
-               req->private_data, struct cli_request);
-       struct cli_trans_state *state = talloc_get_type_abort(
-               cli_req->recv_helper.priv, struct cli_trans_state);
+       struct cli_trans_state *state = tevent_req_data(
+               req, struct cli_trans_state);
        NTSTATUS status;
 
-       if (async_req_is_error(req, &status)) {
+       if (tevent_req_is_nterror(req, &status)) {
                return status;
        }
 
+       if ((state->num_rsetup < min_setup)
+           || (state->rparam.total < min_param)
+           || (state->rdata.total < min_data)) {
+               return NT_STATUS_INVALID_NETWORK_RESPONSE;
+       }
+
+       if (recv_flags2 != NULL) {
+               *recv_flags2 = state->recv_flags2;
+       }
+
        if (setup != NULL) {
                *setup = talloc_move(mem_ctx, &state->rsetup);
                *num_setup = state->num_rsetup;
@@ -1353,25 +637,27 @@ NTSTATUS cli_trans(TALLOC_CTX *mem_ctx, struct cli_state *cli,
                   uint16_t *setup, uint8_t num_setup, uint8_t max_setup,
                   uint8_t *param, uint32_t num_param, uint32_t max_param,
                   uint8_t *data, uint32_t num_data, uint32_t max_data,
-                  uint16_t **rsetup, uint8_t *num_rsetup,
-                  uint8_t **rparam, uint32_t *num_rparam,
-                  uint8_t **rdata, uint32_t *num_rdata)
+                  uint16_t *recv_flags2,
+                  uint16_t **rsetup, uint8_t min_rsetup, uint8_t *num_rsetup,
+                  uint8_t **rparam, uint32_t min_rparam, uint32_t *num_rparam,
+                  uint8_t **rdata, uint32_t min_rdata, uint32_t *num_rdata)
 {
        TALLOC_CTX *frame = talloc_stackframe();
        struct event_context *ev;
-       struct async_req *req;
-       NTSTATUS status = NT_STATUS_NO_MEMORY;
+       struct tevent_req *req;
+       NTSTATUS status = NT_STATUS_OK;
 
-       if (cli->fd_event != NULL) {
+       if (cli_has_async_calls(cli)) {
                /*
                 * Can't use sync call while an async call is in flight
                 */
-               cli_set_error(cli, NT_STATUS_INVALID_PARAMETER);
+               status = NT_STATUS_INVALID_PARAMETER;
                goto fail;
        }
 
        ev = event_context_init(frame);
        if (ev == NULL) {
+               status = NT_STATUS_NO_MEMORY;
                goto fail;
        }
 
@@ -1381,15 +667,19 @@ NTSTATUS cli_trans(TALLOC_CTX *mem_ctx, struct cli_state *cli,
                             param, num_param, max_param,
                             data, num_data, max_data);
        if (req == NULL) {
+               status = NT_STATUS_NO_MEMORY;
                goto fail;
        }
 
-       while (req->state < ASYNC_REQ_DONE) {
-               event_loop_once(ev);
+       if (!tevent_req_poll(req, ev)) {
+               status = map_nt_error_from_unix(errno);
+               goto fail;
        }
 
-       status = cli_trans_recv(req, mem_ctx, rsetup, num_rsetup,
-                               rparam, num_rparam, rdata, num_rdata);
+       status = cli_trans_recv(req, mem_ctx, recv_flags2,
+                               rsetup, min_rsetup, num_rsetup,
+                               rparam, min_rparam, num_rparam,
+                               rdata, min_rdata, num_rdata);
  fail:
        TALLOC_FREE(frame);
        return status;