[Pkg-samba-maint] [Git][samba-team/samba][stretch-ci] 3 commits: s3:libsmb: add cli_smb2_delete_on_close*()
Mathieu Parent
gitlab at salsa.debian.org
Sun Jan 20 13:15:48 GMT 2019
Mathieu Parent pushed to branch stretch-ci at Debian Samba Team / samba
Commits:
a4ecc2dd by Stefan Metzmacher at 2019-01-20T11:19:08Z
s3:libsmb: add cli_smb2_delete_on_close*()
Signed-off-by: Stefan Metzmacher <metze at samba.org>
Reviewed-by: Jeremy Allison <jra at samba.org>
(cherry picked from commit 8d4005b07b08d5673b75d5d79f9b3d6936596fae)
- - - - -
704c38f1 by Mathieu Parent at 2019-01-20T11:19:08Z
Add patch for previous commit
s3:libsmb: add cli_smb2_delete_on_close*()
Signed-off-by: Stefan Metzmacher <metze at samba.org>
Reviewed-by: Jeremy Allison <jra at samba.org>
(cherry picked from commit 8d4005b07b08d5673b75d5d79f9b3d6936596fae)
- - - - -
b634b1ba by Mathieu Parent at 2019-01-20T13:10:28Z
Add debian/gitlab-ci.yml
- - - - -
4 changed files:
- + debian/gitlab-ci.yml
- debian/patches/fix-rmdir.patch
- source3/libsmb/cli_smb2_fnum.c
- source3/libsmb/cli_smb2_fnum.h
Changes:
=====================================
debian/gitlab-ci.yml
=====================================
@@ -0,0 +1,20 @@
+#include: https://salsa.debian.org/salsa-ci-team/pipeline/raw/master/salsa-ci.yml
+include: https://salsa.debian.org/sathieu/pipeline/raw/master/salsa-ci.yml
+
+build:
+ extends: .build-stretch
+
+reprotest:
+ extends: .test-reprotest
+ # Until https://bugs.debian.org/912340 is fixed
+ allow_failure: true
+
+lintian:
+ extends: .test-lintian
+
+autopkgtest:
+ extends: .test-autopkgtest-stretch
+ image: registry.salsa.debian.org/sathieu/images/autopkgtest:stretch
+
+piuparts:
+ extends: .test-piuparts
=====================================
debian/patches/fix-rmdir.patch
=====================================
@@ -1,3 +1,170 @@
+From: Stefan Metzmacher <metze at samba.org>
+Date: Tue, 20 Jun 2017 08:35:13 +0200
+Subject: s3:libsmb: add cli_smb2_delete_on_close*()
+
+Signed-off-by: Stefan Metzmacher <metze at samba.org>
+Reviewed-by: Jeremy Allison <jra at samba.org>
+(cherry picked from commit 8d4005b07b08d5673b75d5d79f9b3d6936596fae)
+
+diff --git a/source3/libsmb/cli_smb2_fnum.c b/source3/libsmb/cli_smb2_fnum.c
+index 43f116b681d..674c52e4405 100644
+--- a/source3/libsmb/cli_smb2_fnum.c
++++ b/source3/libsmb/cli_smb2_fnum.c
+@@ -484,6 +484,133 @@ NTSTATUS cli_smb2_close_fnum(struct cli_state *cli, uint16_t fnum)
+ return status;
+ }
+
++struct cli_smb2_delete_on_close_state {
++ struct cli_state *cli;
++ uint16_t fnum;
++ struct smb2_hnd *ph;
++ uint8_t data[1];
++ DATA_BLOB inbuf;
++};
++
++static void cli_smb2_delete_on_close_done(struct tevent_req *subreq);
++
++struct tevent_req *cli_smb2_delete_on_close_send(TALLOC_CTX *mem_ctx,
++ struct tevent_context *ev,
++ struct cli_state *cli,
++ uint16_t fnum,
++ bool flag)
++{
++ struct tevent_req *req = NULL;
++ struct cli_smb2_delete_on_close_state *state = NULL;
++ struct tevent_req *subreq = NULL;
++ uint8_t in_info_type;
++ uint8_t in_file_info_class;
++ NTSTATUS status;
++
++ req = tevent_req_create(mem_ctx, &state,
++ struct cli_smb2_delete_on_close_state);
++ if (req == NULL) {
++ return NULL;
++ }
++ state->cli = cli;
++ state->fnum = fnum;
++
++ if (smbXcli_conn_protocol(cli->conn) < PROTOCOL_SMB2_02) {
++ tevent_req_nterror(req, NT_STATUS_INVALID_PARAMETER);
++ return tevent_req_post(req, ev);
++ }
++
++ status = map_fnum_to_smb2_handle(cli, fnum, &state->ph);
++ if (tevent_req_nterror(req, status)) {
++ return tevent_req_post(req, ev);
++ }
++
++ /*
++ * setinfo on the handle with info_type SMB2_SETINFO_FILE (1),
++ * level 13 (SMB_FILE_DISPOSITION_INFORMATION - 1000).
++ */
++ in_info_type = 1;
++ in_file_info_class = SMB_FILE_DISPOSITION_INFORMATION - 1000;
++ /* Setup data array. */
++ SCVAL(&state->data[0], 0, flag ? 1 : 0);
++ state->inbuf.data = &state->data[0];
++ state->inbuf.length = 1;
++
++ subreq = smb2cli_set_info_send(state, ev,
++ cli->conn,
++ cli->timeout,
++ cli->smb2.session,
++ cli->smb2.tcon,
++ in_info_type,
++ in_file_info_class,
++ &state->inbuf, /* in_input_buffer */
++ 0, /* in_additional_info */
++ state->ph->fid_persistent,
++ state->ph->fid_volatile);
++ if (tevent_req_nomem(subreq, req)) {
++ return tevent_req_post(req, ev);
++ }
++ tevent_req_set_callback(subreq,
++ cli_smb2_delete_on_close_done,
++ req);
++ return req;
++}
++
++static void cli_smb2_delete_on_close_done(struct tevent_req *subreq)
++{
++ NTSTATUS status = smb2cli_set_info_recv(subreq);
++ tevent_req_simple_finish_ntstatus(subreq, status);
++}
++
++NTSTATUS cli_smb2_delete_on_close_recv(struct tevent_req *req)
++{
++ struct cli_smb2_delete_on_close_state *state =
++ tevent_req_data(req,
++ struct cli_smb2_delete_on_close_state);
++ NTSTATUS status;
++
++ if (tevent_req_is_nterror(req, &status)) {
++ state->cli->raw_status = status;
++ tevent_req_received(req);
++ return status;
++ }
++
++ state->cli->raw_status = NT_STATUS_OK;
++ tevent_req_received(req);
++ return NT_STATUS_OK;
++}
++
++NTSTATUS cli_smb2_delete_on_close(struct cli_state *cli, uint16_t fnum, bool flag)
++{
++ TALLOC_CTX *frame = talloc_stackframe();
++ struct tevent_context *ev;
++ struct tevent_req *req;
++ NTSTATUS status = NT_STATUS_NO_MEMORY;
++
++ if (smbXcli_conn_has_async_calls(cli->conn)) {
++ /*
++ * Can't use sync call while an async call is in flight
++ */
++ status = NT_STATUS_INVALID_PARAMETER;
++ goto fail;
++ }
++ ev = samba_tevent_context_init(frame);
++ if (ev == NULL) {
++ goto fail;
++ }
++ req = cli_smb2_delete_on_close_send(frame, ev, cli, fnum, flag);
++ if (req == NULL) {
++ goto fail;
++ }
++ if (!tevent_req_poll_ntstatus(req, ev, &status)) {
++ goto fail;
++ }
++ status = cli_smb2_delete_on_close_recv(req);
++ fail:
++ TALLOC_FREE(frame);
++ return status;
++}
++
+ /***************************************************************
+ Small wrapper that allows SMB2 to create a directory
+ Synchronous only.
+diff --git a/source3/libsmb/cli_smb2_fnum.h b/source3/libsmb/cli_smb2_fnum.h
+index 435a5c28db3..b7d64f1da4d 100644
+--- a/source3/libsmb/cli_smb2_fnum.h
++++ b/source3/libsmb/cli_smb2_fnum.h
+@@ -54,6 +54,13 @@ struct tevent_req *cli_smb2_close_fnum_send(TALLOC_CTX *mem_ctx,
+ uint16_t fnum);
+ NTSTATUS cli_smb2_close_fnum_recv(struct tevent_req *req);
+ NTSTATUS cli_smb2_close_fnum(struct cli_state *cli, uint16_t fnum);
++struct tevent_req *cli_smb2_delete_on_close_send(TALLOC_CTX *mem_ctx,
++ struct tevent_context *ev,
++ struct cli_state *cli,
++ uint16_t fnum,
++ bool flag);
++NTSTATUS cli_smb2_delete_on_close_recv(struct tevent_req *req);
++NTSTATUS cli_smb2_delete_on_close(struct cli_state *cli, uint16_t fnum, bool flag);
+ NTSTATUS cli_smb2_mkdir(struct cli_state *cli, const char *dirname);
+ NTSTATUS cli_smb2_rmdir(struct cli_state *cli, const char *dirname);
+ NTSTATUS cli_smb2_unlink(struct cli_state *cli,const char *fname);
+--
+2.20.1
+
From: Anoop C S <anoopcs at redhat.com>
Date: Thu, 9 Aug 2018 12:28:41 +0530
Subject: s3/libsmb: Explicitly set delete_on_close token for rmdir
@@ -20,10 +187,10 @@ Bug-Debian: https://bugs.debian.org/915248
Bug-Ubuntu: https://bugs.launchpad.net/ubuntu/+source/samba/+bug/1795772
diff --git a/source3/libsmb/cli_smb2_fnum.c b/source3/libsmb/cli_smb2_fnum.c
-index 237e6bb2b..d4ff8bd28 100644
+index fc99be23d03..43f116b681d 100644
--- a/source3/libsmb/cli_smb2_fnum.c
+++ b/source3/libsmb/cli_smb2_fnum.c
-@@ -682,13 +682,20 @@ NTSTATUS cli_smb2_rmdir(struct cli_state *cli, const char *dname)
+@@ -550,13 +550,20 @@ NTSTATUS cli_smb2_rmdir(struct cli_state *cli, const char *dname)
FILE_ATTRIBUTE_DIRECTORY, /* file attributes */
FILE_SHARE_READ|FILE_SHARE_WRITE|FILE_SHARE_DELETE, /* share_access */
FILE_OPEN, /* create_disposition */
@@ -45,3 +212,6 @@ index 237e6bb2b..d4ff8bd28 100644
return cli_smb2_close_fnum(cli, fnum);
}
+--
+2.20.1
+
=====================================
source3/libsmb/cli_smb2_fnum.c
=====================================
@@ -484,6 +484,133 @@ NTSTATUS cli_smb2_close_fnum(struct cli_state *cli, uint16_t fnum)
return status;
}
+struct cli_smb2_delete_on_close_state {
+ struct cli_state *cli;
+ uint16_t fnum;
+ struct smb2_hnd *ph;
+ uint8_t data[1];
+ DATA_BLOB inbuf;
+};
+
+static void cli_smb2_delete_on_close_done(struct tevent_req *subreq);
+
+struct tevent_req *cli_smb2_delete_on_close_send(TALLOC_CTX *mem_ctx,
+ struct tevent_context *ev,
+ struct cli_state *cli,
+ uint16_t fnum,
+ bool flag)
+{
+ struct tevent_req *req = NULL;
+ struct cli_smb2_delete_on_close_state *state = NULL;
+ struct tevent_req *subreq = NULL;
+ uint8_t in_info_type;
+ uint8_t in_file_info_class;
+ NTSTATUS status;
+
+ req = tevent_req_create(mem_ctx, &state,
+ struct cli_smb2_delete_on_close_state);
+ if (req == NULL) {
+ return NULL;
+ }
+ state->cli = cli;
+ state->fnum = fnum;
+
+ if (smbXcli_conn_protocol(cli->conn) < PROTOCOL_SMB2_02) {
+ tevent_req_nterror(req, NT_STATUS_INVALID_PARAMETER);
+ return tevent_req_post(req, ev);
+ }
+
+ status = map_fnum_to_smb2_handle(cli, fnum, &state->ph);
+ if (tevent_req_nterror(req, status)) {
+ return tevent_req_post(req, ev);
+ }
+
+ /*
+ * setinfo on the handle with info_type SMB2_SETINFO_FILE (1),
+ * level 13 (SMB_FILE_DISPOSITION_INFORMATION - 1000).
+ */
+ in_info_type = 1;
+ in_file_info_class = SMB_FILE_DISPOSITION_INFORMATION - 1000;
+ /* Setup data array. */
+ SCVAL(&state->data[0], 0, flag ? 1 : 0);
+ state->inbuf.data = &state->data[0];
+ state->inbuf.length = 1;
+
+ subreq = smb2cli_set_info_send(state, ev,
+ cli->conn,
+ cli->timeout,
+ cli->smb2.session,
+ cli->smb2.tcon,
+ in_info_type,
+ in_file_info_class,
+ &state->inbuf, /* in_input_buffer */
+ 0, /* in_additional_info */
+ state->ph->fid_persistent,
+ state->ph->fid_volatile);
+ if (tevent_req_nomem(subreq, req)) {
+ return tevent_req_post(req, ev);
+ }
+ tevent_req_set_callback(subreq,
+ cli_smb2_delete_on_close_done,
+ req);
+ return req;
+}
+
+static void cli_smb2_delete_on_close_done(struct tevent_req *subreq)
+{
+ NTSTATUS status = smb2cli_set_info_recv(subreq);
+ tevent_req_simple_finish_ntstatus(subreq, status);
+}
+
+NTSTATUS cli_smb2_delete_on_close_recv(struct tevent_req *req)
+{
+ struct cli_smb2_delete_on_close_state *state =
+ tevent_req_data(req,
+ struct cli_smb2_delete_on_close_state);
+ NTSTATUS status;
+
+ if (tevent_req_is_nterror(req, &status)) {
+ state->cli->raw_status = status;
+ tevent_req_received(req);
+ return status;
+ }
+
+ state->cli->raw_status = NT_STATUS_OK;
+ tevent_req_received(req);
+ return NT_STATUS_OK;
+}
+
+NTSTATUS cli_smb2_delete_on_close(struct cli_state *cli, uint16_t fnum, bool flag)
+{
+ TALLOC_CTX *frame = talloc_stackframe();
+ struct tevent_context *ev;
+ struct tevent_req *req;
+ NTSTATUS status = NT_STATUS_NO_MEMORY;
+
+ if (smbXcli_conn_has_async_calls(cli->conn)) {
+ /*
+ * Can't use sync call while an async call is in flight
+ */
+ status = NT_STATUS_INVALID_PARAMETER;
+ goto fail;
+ }
+ ev = samba_tevent_context_init(frame);
+ if (ev == NULL) {
+ goto fail;
+ }
+ req = cli_smb2_delete_on_close_send(frame, ev, cli, fnum, flag);
+ if (req == NULL) {
+ goto fail;
+ }
+ if (!tevent_req_poll_ntstatus(req, ev, &status)) {
+ goto fail;
+ }
+ status = cli_smb2_delete_on_close_recv(req);
+ fail:
+ TALLOC_FREE(frame);
+ return status;
+}
+
/***************************************************************
Small wrapper that allows SMB2 to create a directory
Synchronous only.
=====================================
source3/libsmb/cli_smb2_fnum.h
=====================================
@@ -54,6 +54,13 @@ struct tevent_req *cli_smb2_close_fnum_send(TALLOC_CTX *mem_ctx,
uint16_t fnum);
NTSTATUS cli_smb2_close_fnum_recv(struct tevent_req *req);
NTSTATUS cli_smb2_close_fnum(struct cli_state *cli, uint16_t fnum);
+struct tevent_req *cli_smb2_delete_on_close_send(TALLOC_CTX *mem_ctx,
+ struct tevent_context *ev,
+ struct cli_state *cli,
+ uint16_t fnum,
+ bool flag);
+NTSTATUS cli_smb2_delete_on_close_recv(struct tevent_req *req);
+NTSTATUS cli_smb2_delete_on_close(struct cli_state *cli, uint16_t fnum, bool flag);
NTSTATUS cli_smb2_mkdir(struct cli_state *cli, const char *dirname);
NTSTATUS cli_smb2_rmdir(struct cli_state *cli, const char *dirname);
NTSTATUS cli_smb2_unlink(struct cli_state *cli,const char *fname);
View it on GitLab: https://salsa.debian.org/samba-team/samba/compare/8bc24c41301689a593e05f26e0454220f23e0ec8...b634b1bacee9d1570b330d9b0fd9f9576472e6cb
--
View it on GitLab: https://salsa.debian.org/samba-team/samba/compare/8bc24c41301689a593e05f26e0454220f23e0ec8...b634b1bacee9d1570b330d9b0fd9f9576472e6cb
You're receiving this email because of your account on salsa.debian.org.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://alioth-lists.debian.net/pipermail/pkg-samba-maint/attachments/20190120/5a6a865c/attachment-0001.html>
More information about the Pkg-samba-maint
mailing list