lib: Split out write_data[_iov]
[obnox/samba/samba-obnox.git] / source3 / smbd / smb2_read.c
index c3cbbe8084ca66668d28bd9cd18d6f593f25a206..4e974a2eee1c5ac0f258338a6068e552c8f36935 100644 (file)
 #include "libcli/security/security.h"
 #include "../lib/util/tevent_ntstatus.h"
 #include "rpc_server/srv_pipe_hnd.h"
+#include "lib/sys_rw_data.h"
 
 static struct tevent_req *smbd_smb2_read_send(TALLOC_CTX *mem_ctx,
                                              struct tevent_context *ev,
                                              struct smbd_smb2_request *smb2req,
-                                             uint32_t in_smbpid,
-                                             uint64_t in_file_id_volatile,
+                                             struct files_struct *in_fsp,
                                              uint32_t in_length,
                                              uint64_t in_offset,
                                              uint32_t in_minimum,
@@ -44,15 +44,14 @@ static NTSTATUS smbd_smb2_read_recv(struct tevent_req *req,
 static void smbd_smb2_request_read_done(struct tevent_req *subreq);
 NTSTATUS smbd_smb2_request_process_read(struct smbd_smb2_request *req)
 {
+       struct smbXsrv_connection *xconn = req->xconn;
        NTSTATUS status;
-       const uint8_t *inhdr;
        const uint8_t *inbody;
-       int i = req->current_idx;
-       uint32_t in_smbpid;
        uint32_t in_length;
        uint64_t in_offset;
        uint64_t in_file_id_persistent;
        uint64_t in_file_id_volatile;
+       struct files_struct *in_fsp;
        uint32_t in_minimum_count;
        uint32_t in_remaining_bytes;
        struct tevent_req *subreq;
@@ -61,10 +60,7 @@ NTSTATUS smbd_smb2_request_process_read(struct smbd_smb2_request *req)
        if (!NT_STATUS_IS_OK(status)) {
                return smbd_smb2_request_error(req, status);
        }
-       inhdr = (const uint8_t *)req->in.vector[i+0].iov_base;
-       inbody = (const uint8_t *)req->in.vector[i+1].iov_base;
-
-       in_smbpid = IVAL(inhdr, SMB2_HDR_PID);
+       inbody = SMBD_SMB2_IN_BODY_PTR(req);
 
        in_length               = IVAL(inbody, 0x04);
        in_offset               = BVAL(inbody, 0x08);
@@ -74,23 +70,25 @@ NTSTATUS smbd_smb2_request_process_read(struct smbd_smb2_request *req)
        in_remaining_bytes      = IVAL(inbody, 0x28);
 
        /* check the max read size */
-       if (in_length > req->sconn->smb2.max_read) {
-               DEBUG(0,("here:%s: 0x%08X: 0x%08X\n",
-                       __location__, in_length, req->sconn->smb2.max_read));
+       if (in_length > xconn->smb2.server.max_read) {
+               DEBUG(2,("smbd_smb2_request_process_read: "
+                        "client ignored max read: %s: 0x%08X: 0x%08X\n",
+                       __location__, in_length, xconn->smb2.server.max_read));
                return smbd_smb2_request_error(req, NT_STATUS_INVALID_PARAMETER);
        }
 
-       if (req->compat_chain_fsp) {
-               /* skip check */
-       } else if (in_file_id_persistent != in_file_id_volatile) {
+       status = smbd_smb2_request_verify_creditcharge(req, in_length);
+       if (!NT_STATUS_IS_OK(status)) {
+               return smbd_smb2_request_error(req, status);
+       }
+
+       in_fsp = file_fsp_smb2(req, in_file_id_persistent, in_file_id_volatile);
+       if (in_fsp == NULL) {
                return smbd_smb2_request_error(req, NT_STATUS_FILE_CLOSED);
        }
 
-       subreq = smbd_smb2_read_send(req,
-                                    req->sconn->smb2.event_ctx,
-                                    req,
-                                    in_smbpid,
-                                    in_file_id_volatile,
+       subreq = smbd_smb2_read_send(req, req->sconn->ev_ctx,
+                                    req, in_fsp,
                                     in_length,
                                     in_offset,
                                     in_minimum_count,
@@ -107,8 +105,6 @@ static void smbd_smb2_request_read_done(struct tevent_req *subreq)
 {
        struct smbd_smb2_request *req = tevent_req_callback_data(subreq,
                                        struct smbd_smb2_request);
-       int i = req->current_idx;
-       uint8_t *outhdr;
        DATA_BLOB outbody;
        DATA_BLOB outdyn;
        uint8_t out_data_offset;
@@ -125,7 +121,7 @@ static void smbd_smb2_request_read_done(struct tevent_req *subreq)
        if (!NT_STATUS_IS_OK(status)) {
                error = smbd_smb2_request_error(req, status);
                if (!NT_STATUS_IS_OK(error)) {
-                       smbd_server_connection_terminate(req->sconn,
+                       smbd_server_connection_terminate(req->xconn,
                                                         nt_errstr(error));
                        return;
                }
@@ -134,13 +130,11 @@ static void smbd_smb2_request_read_done(struct tevent_req *subreq)
 
        out_data_offset = SMB2_HDR_BODY + 0x10;
 
-       outhdr = (uint8_t *)req->out.vector[i].iov_base;
-
-       outbody = data_blob_talloc(req->out.vector, NULL, 0x10);
+       outbody = smbd_smb2_generate_outbody(req, 0x10);
        if (outbody.data == NULL) {
                error = smbd_smb2_request_error(req, NT_STATUS_NO_MEMORY);
                if (!NT_STATUS_IS_OK(error)) {
-                       smbd_server_connection_terminate(req->sconn,
+                       smbd_server_connection_terminate(req->xconn,
                                                         nt_errstr(error));
                        return;
                }
@@ -161,7 +155,7 @@ static void smbd_smb2_request_read_done(struct tevent_req *subreq)
 
        error = smbd_smb2_request_done(req, outbody, &outdyn);
        if (!NT_STATUS_IS_OK(error)) {
-               smbd_server_connection_terminate(req->sconn,
+               smbd_server_connection_terminate(req->xconn,
                                                 nt_errstr(error));
                return;
        }
@@ -169,15 +163,22 @@ static void smbd_smb2_request_read_done(struct tevent_req *subreq)
 
 struct smbd_smb2_read_state {
        struct smbd_smb2_request *smb2req;
+       struct smb_request *smbreq;
        files_struct *fsp;
-       uint64_t in_file_id_volatile;
        uint32_t in_length;
        uint64_t in_offset;
        uint32_t in_minimum;
+       DATA_BLOB out_headers;
+       uint8_t _out_hdr_buf[NBT_HDR_SIZE + SMB2_HDR_BODY + 0x10];
        DATA_BLOB out_data;
        uint32_t out_remaining;
 };
 
+static int smb2_smb2_read_state_deny_destructor(struct smbd_smb2_read_state *state)
+{
+       return -1;
+}
+
 /* struct smbd_smb2_read_state destructor. Send the SMB2_READ data. */
 static int smb2_sendfile_send_data(struct smbd_smb2_read_state *state)
 {
@@ -185,42 +186,59 @@ static int smb2_sendfile_send_data(struct smbd_smb2_read_state *state)
        uint32_t in_length = state->in_length;
        uint64_t in_offset = state->in_offset;
        files_struct *fsp = state->fsp;
+       const DATA_BLOB *hdr = state->smb2req->queue_entry.sendfile_header;
+       NTSTATUS *pstatus = state->smb2req->queue_entry.sendfile_status;
+       struct smbXsrv_connection *xconn = state->smb2req->xconn;
        ssize_t nread;
+       ssize_t ret;
+       int saved_errno;
 
-       nread = SMB_VFS_SENDFILE(fsp->conn->sconn->sock,
-                                       fsp,
-                                       NULL,
-                                       in_offset,
-                                       in_length);
+       nread = SMB_VFS_SENDFILE(xconn->transport.sock,
+                                fsp,
+                                hdr,
+                                in_offset,
+                                in_length);
        DEBUG(10,("smb2_sendfile_send_data: SMB_VFS_SENDFILE returned %d on file %s\n",
                (int)nread,
                fsp_str_dbg(fsp) ));
 
        if (nread == -1) {
-               if (errno == ENOSYS || errno == EINTR) {
+               saved_errno = errno;
+
+               /*
+                * Returning ENOSYS means no data at all was sent.
+                  Do this as a normal read. */
+               if (errno == ENOSYS) {
+                       goto normal_read;
+               }
+
+               if (errno == EINTR) {
                        /*
-                        * Special hack for broken systems with no working
-                        * sendfile. Fake this up by doing read/write calls.
-                       */
+                        * Special hack for broken Linux with no working sendfile. If we
+                        * return EINTR we sent the header but not the rest of the data.
+                        * Fake this up by doing read/write calls.
+                        */
                        set_use_sendfile(SNUM(fsp->conn), false);
-                       nread = fake_sendfile(fsp, in_offset, in_length);
+                       nread = fake_sendfile(xconn, fsp, in_offset, in_length);
                        if (nread == -1) {
-                               DEBUG(0,("smb2_sendfile_send_data: "
-                                       "fake_sendfile failed for "
-                                       "file %s (%s).\n",
-                                       fsp_str_dbg(fsp),
-                                       strerror(errno)));
-                               exit_server_cleanly("smb2_sendfile_send_data: "
-                                       "fake_sendfile failed");
+                               saved_errno = errno;
+                               DEBUG(0,("smb2_sendfile_send_data: fake_sendfile "
+                                        "failed for file %s (%s) for client %s. "
+                                        "Terminating\n",
+                                        fsp_str_dbg(fsp), strerror(saved_errno),
+                                        smbXsrv_connection_dbg(xconn)));
+                               *pstatus = map_nt_error_from_unix_common(saved_errno);
+                               return 0;
                        }
                        goto out;
                }
 
                DEBUG(0,("smb2_sendfile_send_data: sendfile failed for file "
-                       "%s (%s). Terminating\n",
-                       fsp_str_dbg(fsp),
-                       strerror(errno)));
-               exit_server_cleanly("smb2_sendfile_send_data: sendfile failed");
+                        "%s (%s) for client %s. Terminating\n",
+                        fsp_str_dbg(fsp), strerror(saved_errno),
+                        smbXsrv_connection_dbg(xconn)));
+               *pstatus = map_nt_error_from_unix_common(saved_errno);
+               return 0;
        } else if (nread == 0) {
                /*
                 * Some sendfile implementations return 0 to indicate
@@ -232,40 +250,73 @@ static int smb2_sendfile_send_data(struct smbd_smb2_read_state *state)
                DEBUG(3, ("send_file_readX: sendfile sent zero bytes "
                        "falling back to the normal read: %s\n",
                        fsp_str_dbg(fsp)));
+               goto normal_read;
+       }
 
-               nread = fake_sendfile(fsp, in_offset, in_length);
-               if (nread == -1) {
-                       DEBUG(0,("smb2_sendfile_send_data: "
-                               "fake_sendfile failed for file "
-                               "%s (%s). Terminating\n",
-                               fsp_str_dbg(fsp),
-                               strerror(errno)));
-                       exit_server_cleanly("smb2_sendfile_send_data: "
-                               "fake_sendfile failed");
-               }
+       /*
+        * We got a short read
+        */
+       goto out;
+
+normal_read:
+       /* Send out the header. */
+       ret = write_data(xconn->transport.sock,
+                        (const char *)hdr->data, hdr->length);
+       if (ret != hdr->length) {
+               saved_errno = errno;
+               DEBUG(0,("smb2_sendfile_send_data: write_data failed for file "
+                        "%s (%s) for client %s. Terminating\n",
+                        fsp_str_dbg(fsp), strerror(saved_errno),
+                        smbXsrv_connection_dbg(xconn)));
+               *pstatus = map_nt_error_from_unix_common(saved_errno);
+               return 0;
+       }
+       nread = fake_sendfile(xconn, fsp, in_offset, in_length);
+       if (nread == -1) {
+               saved_errno = errno;
+               DEBUG(0,("smb2_sendfile_send_data: fake_sendfile "
+                        "failed for file %s (%s) for client %s. "
+                        "Terminating\n",
+                        fsp_str_dbg(fsp), strerror(saved_errno),
+                        smbXsrv_connection_dbg(xconn)));
+               *pstatus = map_nt_error_from_unix_common(saved_errno);
+               return 0;
        }
 
   out:
 
        if (nread < in_length) {
-               sendfile_short_send(fsp, nread, 0, in_length);
+               ret = sendfile_short_send(xconn, fsp, nread,
+                                         hdr->length, in_length);
+               if (ret == -1) {
+                       saved_errno = errno;
+                       DEBUG(0,("%s: sendfile_short_send "
+                                "failed for file %s (%s) for client %s. "
+                                "Terminating\n",
+                                __func__,
+                                fsp_str_dbg(fsp), strerror(saved_errno),
+                                smbXsrv_connection_dbg(xconn)));
+                       *pstatus = map_nt_error_from_unix_common(saved_errno);
+                       return 0;
+               }
        }
 
        init_strict_lock_struct(fsp,
-                               state->in_file_id_volatile,
+                               fsp->op->global->open_persistent_id,
                                in_offset,
                                in_length,
                                READ_LOCK,
                                &lock);
 
        SMB_VFS_STRICT_UNLOCK(fsp->conn, fsp, &lock);
+
+       *pstatus = NT_STATUS_OK;
        return 0;
 }
 
 static NTSTATUS schedule_smb2_sendfile_read(struct smbd_smb2_request *smb2req,
                                        struct smbd_smb2_read_state *state)
 {
-       struct smbd_smb2_read_state *state_copy = NULL;
        files_struct *fsp = state->fsp;
 
        /*
@@ -283,14 +334,15 @@ static NTSTATUS schedule_smb2_sendfile_read(struct smbd_smb2_request *smb2req,
        */
 
        if (!lp__use_sendfile(SNUM(fsp->conn)) ||
-                       smb2req->do_signing ||
-                       smb2req->in.vector_count != 4 ||
-                       (fsp->base_fsp != NULL) ||
-                       (fsp->wcp != NULL) ||
-                       (!S_ISREG(fsp->fsp_name->st.st_ex_mode)) ||
-                       (state->in_offset >= fsp->fsp_name->st.st_ex_size) ||
-                       (fsp->fsp_name->st.st_ex_size < state->in_offset +
-                               state->in_length)) {
+           smb2req->do_signing ||
+           smb2req->do_encryption ||
+           smb2req->in.vector_count >= (2*SMBD_SMB2_NUM_IOV_PER_REQ) ||
+           (fsp->base_fsp != NULL) ||
+           (fsp->wcp != NULL) ||
+           (!S_ISREG(fsp->fsp_name->st.st_ex_mode)) ||
+           (state->in_offset >= fsp->fsp_name->st.st_ex_size) ||
+           (fsp->fsp_name->st.st_ex_size < state->in_offset + state->in_length))
+       {
                return NT_STATUS_RETRY;
        }
 
@@ -299,15 +351,8 @@ static NTSTATUS schedule_smb2_sendfile_read(struct smbd_smb2_request *smb2req,
        state->out_data.length = state->in_length;
        state->out_remaining = 0;
 
-       /* Make a copy of state attached to the smb2req. Attach
-          the destructor here as this will trigger the sendfile
-          call when the request is destroyed. */
-       state_copy = talloc(smb2req, struct smbd_smb2_read_state);
-       if (!state_copy) {
-               return NT_STATUS_NO_MEMORY;
-       }
-       *state_copy = *state;
-       talloc_set_destructor(state_copy, smb2_sendfile_send_data);
+       state->out_headers = data_blob_const(state->_out_hdr_buf,
+                                            sizeof(state->_out_hdr_buf));
        return NT_STATUS_OK;
 }
 
@@ -351,8 +396,8 @@ NTSTATUS smb2_read_complete(struct tevent_req *req, ssize_t nread, int err)
                return NT_STATUS_END_OF_FILE;
        }
 
-       DEBUG(3,("smbd_smb2_read: fnum=[%d/%s] length=%lu offset=%lu read=%lu\n",
-               fsp->fnum,
+       DEBUG(3,("smbd_smb2_read: %s, file %s, length=%lu offset=%lu read=%lu\n",
+               fsp_fnum_dbg(fsp),
                fsp_str_dbg(fsp),
                (unsigned long)state->in_length,
                (unsigned long)state->in_offset,
@@ -364,11 +409,19 @@ NTSTATUS smb2_read_complete(struct tevent_req *req, ssize_t nread, int err)
        return NT_STATUS_OK;
 }
 
+static bool smbd_smb2_read_cancel(struct tevent_req *req)
+{
+       struct smbd_smb2_read_state *state =
+               tevent_req_data(req,
+               struct smbd_smb2_read_state);
+
+       return cancel_smb2_aio(state->smbreq);
+}
+
 static struct tevent_req *smbd_smb2_read_send(TALLOC_CTX *mem_ctx,
                                              struct tevent_context *ev,
                                              struct smbd_smb2_request *smb2req,
-                                             uint32_t in_smbpid,
-                                             uint64_t in_file_id_volatile,
+                                             struct files_struct *fsp,
                                              uint32_t in_length,
                                              uint64_t in_offset,
                                              uint32_t in_minimum,
@@ -378,8 +431,7 @@ static struct tevent_req *smbd_smb2_read_send(TALLOC_CTX *mem_ctx,
        struct tevent_req *req = NULL;
        struct smbd_smb2_read_state *state = NULL;
        struct smb_request *smbreq = NULL;
-       connection_struct *conn = smb2req->tcon->compat_conn;
-       files_struct *fsp = NULL;
+       connection_struct *conn = smb2req->tcon->compat;
        ssize_t nread = -1;
        struct lock_struct lock;
        int saved_errno;
@@ -396,34 +448,21 @@ static struct tevent_req *smbd_smb2_read_send(TALLOC_CTX *mem_ctx,
        state->out_data = data_blob_null;
        state->out_remaining = 0;
 
-       DEBUG(10,("smbd_smb2_read: file_id[0x%016llX]\n",
-                 (unsigned long long)in_file_id_volatile));
+       DEBUG(10,("smbd_smb2_read: %s - %s\n",
+                 fsp_str_dbg(fsp), fsp_fnum_dbg(fsp)));
 
        smbreq = smbd_smb2_fake_smb_request(smb2req);
        if (tevent_req_nomem(smbreq, req)) {
                return tevent_req_post(req, ev);
        }
+       state->smbreq = smbreq;
 
-       fsp = file_fsp(smbreq, (uint16_t)in_file_id_volatile);
-       if (fsp == NULL) {
-               tevent_req_nterror(req, NT_STATUS_FILE_CLOSED);
-               return tevent_req_post(req, ev);
-       }
-       if (conn != fsp->conn) {
-               tevent_req_nterror(req, NT_STATUS_FILE_CLOSED);
-               return tevent_req_post(req, ev);
-       }
-       if (smb2req->session->vuid != fsp->vuid) {
-               tevent_req_nterror(req, NT_STATUS_FILE_CLOSED);
-               return tevent_req_post(req, ev);
-       }
        if (fsp->is_directory) {
                tevent_req_nterror(req, NT_STATUS_INVALID_DEVICE_REQUEST);
                return tevent_req_post(req, ev);
        }
 
        state->fsp = fsp;
-       state->in_file_id_volatile = in_file_id_volatile;
 
        if (IS_IPC(smbreq->conn)) {
                struct tevent_req *subreq = NULL;
@@ -438,7 +477,7 @@ static struct tevent_req *smbd_smb2_read_send(TALLOC_CTX *mem_ctx,
                        return tevent_req_post(req, ev);
                }
 
-               subreq = np_read_send(state, server_event_context(),
+               subreq = np_read_send(state, ev,
                                      fsp->fake_file_handle,
                                      state->out_data.data,
                                      state->out_data.length);
@@ -461,26 +500,28 @@ static struct tevent_req *smbd_smb2_read_send(TALLOC_CTX *mem_ctx,
                                fsp,
                                state,
                                &state->out_data,
-                               (SMB_OFF_T)in_offset,
+                               (off_t)in_offset,
                                (size_t)in_length);
 
        if (NT_STATUS_IS_OK(status)) {
                /*
-                * Doing an async read.
+                * Doing an async read, allow this
+                * request to be canceled
                 */
+               tevent_req_set_cancel_fn(req, smbd_smb2_read_cancel);
                return req;
        }
 
        if (!NT_STATUS_EQUAL(status, NT_STATUS_RETRY)) {
                /* Real error in setting up aio. Fail. */
-               tevent_req_nterror(req, NT_STATUS_FILE_CLOSED);
+               tevent_req_nterror(req, status);
                return tevent_req_post(req, ev);
        }
 
        /* Fallback to synchronous. */
 
        init_strict_lock_struct(fsp,
-                               in_file_id_volatile,
+                               fsp->op->global->open_persistent_id,
                                in_offset,
                                in_length,
                                READ_LOCK,
@@ -520,10 +561,10 @@ static struct tevent_req *smbd_smb2_read_send(TALLOC_CTX *mem_ctx,
 
        SMB_VFS_STRICT_UNLOCK(conn, fsp, &lock);
 
-       DEBUG(10,("smbd_smb2_read: file %s handle [0x%016llX] offset=%llu "
+       DEBUG(10,("smbd_smb2_read: file %s, %s, offset=%llu "
                "len=%llu returned %lld\n",
                fsp_str_dbg(fsp),
-               (unsigned long long)in_file_id_volatile,
+               fsp_fnum_dbg(fsp),
                (unsigned long long)in_offset,
                (unsigned long long)in_length,
                (long long)nread));
@@ -551,6 +592,8 @@ static void smbd_smb2_read_pipe_done(struct tevent_req *subreq)
        status = np_read_recv(subreq, &nread, &is_data_outstanding);
        TALLOC_FREE(subreq);
        if (!NT_STATUS_IS_OK(status)) {
+               NTSTATUS old = status;
+               status = nt_status_np_pipe(old);
                tevent_req_nterror(req, status);
                return;
        }
@@ -589,6 +632,15 @@ static NTSTATUS smbd_smb2_read_recv(struct tevent_req *req,
        talloc_steal(mem_ctx, out_data->data);
        *out_remaining = state->out_remaining;
 
-       tevent_req_received(req);
+       if (state->out_headers.length > 0) {
+               talloc_steal(mem_ctx, state);
+               talloc_set_destructor(state, smb2_smb2_read_state_deny_destructor);
+               tevent_req_received(req);
+               state->smb2req->queue_entry.sendfile_header = &state->out_headers;
+               talloc_set_destructor(state, smb2_sendfile_send_data);
+       } else {
+               tevent_req_received(req);
+       }
+
        return NT_STATUS_OK;
 }