Compiler errors on ubuntu 14.04 (svhdx changes)
Jeremy Allison
jra at samba.org
Fri Aug 28 15:53:45 UTC 2015
On Fri, Aug 28, 2015 at 02:19:24PM +0200, Stefan Metzmacher wrote:
> Hi,
>
> can someone please push this fix?
>
> It's required when using --pick-developer on ubuntu 14.04.
Should we be running autobuilds with --picky-developer
too ?
> Am 28.08.2015 um 03:20 schrieb Jeremy Allison:
> > The branch, master has been updated
> > via e6c234d Move the error handling for svhdx to vfswrap_create to give VFS module writers a chance to handle RSVD opens if they want to.
> > from d9166eb lib/crypto: make it possible to use only parts of aes.[ch]
> >
> > https://git.samba.org/?p=samba.git;a=shortlog;h=master
> >
> >
> > - Log -----------------------------------------------------------------
> > commit e6c234d31ad22120d0890b561dac9b456f8f6530
> > Author: Richard Sharpe <rsharpe at samba.org>
> > Date: Tue Jul 28 19:08:02 2015 -0700
> >
> > Move the error handling for svhdx to vfswrap_create to give VFS module writers a chance to handle RSVD opens if they want to.
> >
> > Also handle a review comment by Metze.
> >
> > Signed-off-by: Richard Sharpe <rsharpe at samba.org>
> > Reviewed-by: Ira Cooper <ira at samba.org>
> > Reviewed-by: Jeremy Allison <jra at samba.org>
> >
> > Autobuild-User(master): Jeremy Allison <jra at samba.org>
> > Autobuild-Date(master): Fri Aug 28 03:19:36 CEST 2015 on sn-devel-104
> >
> > -----------------------------------------------------------------------
> >
> > Summary of changes:
> > source3/modules/vfs_default.c | 16 ++++++++++++++++
> > source3/smbd/smb2_create.c | 7 -------
> > 2 files changed, 16 insertions(+), 7 deletions(-)
> >
> >
> > Changeset truncated at 500 lines:
> >
> > diff --git a/source3/modules/vfs_default.c b/source3/modules/vfs_default.c
> > index 460837c..9ea630a 100644
> > --- a/source3/modules/vfs_default.c
> > +++ b/source3/modules/vfs_default.c
> > @@ -563,6 +563,22 @@ static NTSTATUS vfswrap_create_file(vfs_handle_struct *handle,
> > const struct smb2_create_blobs *in_context_blobs,
> > struct smb2_create_blobs *out_context_blobs)
> > {
> > + struct smb2_create_blob *svhdx = NULL;
> > +
> > + /*
> > + * It might be empty ... and smb2_create_blob_find does not handle that
> > + */
> > + if (in_context_blobs) {
> > + svhdx = smb2_create_blob_find(in_context_blobs,
> > + SVHDX_OPEN_DEVICE_CONTEXT);
> > + }
> > +
> > + if (svhdx != NULL) {
> > + /* SharedVHD is not yet supported */
> > + DEBUG(10, ("Shared VHD not yet supported, INVALID_DEVICE_REQUEST\n"));
> > + return NT_STATUS_INVALID_DEVICE_REQUEST;
> > + }
> > +
> > return create_file_default(handle->conn, req, root_dir_fid, smb_fname,
> > access_mask, share_access,
> > create_disposition, create_options,
> > diff --git a/source3/smbd/smb2_create.c b/source3/smbd/smb2_create.c
> > index 880ceee..9f14544 100644
> > --- a/source3/smbd/smb2_create.c
> > +++ b/source3/smbd/smb2_create.c
> > @@ -910,13 +910,6 @@ static struct tevent_req *smbd_smb2_create_send(TALLOC_CTX *mem_ctx,
> > }
> > }
> >
> > - if (svhdx != NULL) {
> > - /* SharedVHD is not yet supported */
> > - tevent_req_nterror(
> > - req, NT_STATUS_INVALID_DEVICE_REQUEST);
> > - return tevent_req_post(req, ev);
> > - }
> > -
> > /* these are ignored for SMB2 */
> > in_create_options &= ~(0x10);/* NTCREATEX_OPTIONS_SYNC_ALERT */
> > in_create_options &= ~(0x20);/* NTCREATEX_OPTIONS_ASYNC_ALERT */
> >
> >
> From 56edd939fa9299a6e6b62cff2890daed9455d89e Mon Sep 17 00:00:00 2001
> From: Stefan Metzmacher <metze at samba.org>
> Date: Fri, 28 Aug 2015 14:16:14 +0200
> Subject: [PATCH] s3:smb2_create: #if 0 unused variable
> MIME-Version: 1.0
> Content-Type: text/plain; charset=UTF-8
> Content-Transfer-Encoding: 8bit
>
> This fixes the build on ubuntu 14.04, which failed like this:
>
> [2852/3952] Compiling source3/smbd/smb2_create.c
> ../source3/smbd/smb2_create.c: In function ‘smbd_smb2_create_send’:
> ../source3/smbd/smb2_create.c:678:28: error: variable ‘svhdx’ set but not used [-Werror=unused-but-set-variable]
> struct smb2_create_blob *svhdx = NULL;
>
> Signed-off-by: Stefan Metzmacher <metze at samba.org>
> ---
> source3/smbd/smb2_create.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/source3/smbd/smb2_create.c b/source3/smbd/smb2_create.c
> index 9f14544..e151e96 100644
> --- a/source3/smbd/smb2_create.c
> +++ b/source3/smbd/smb2_create.c
> @@ -675,7 +675,9 @@ static struct tevent_req *smbd_smb2_create_send(TALLOC_CTX *mem_ctx,
> struct smb2_lease lease;
> struct smb2_lease *lease_ptr = NULL;
> ssize_t lease_len = -1;
> +#if 0
> struct smb2_create_blob *svhdx = NULL;
> +#endif
>
> exta = smb2_create_blob_find(&in_context_blobs,
> SMB2_CREATE_TAG_EXTA);
> @@ -689,6 +691,7 @@ static struct tevent_req *smbd_smb2_create_send(TALLOC_CTX *mem_ctx,
> SMB2_CREATE_TAG_TWRP);
> qfid = smb2_create_blob_find(&in_context_blobs,
> SMB2_CREATE_TAG_QFID);
> +#if 0
> if (smb2req->xconn->protocol >= PROTOCOL_SMB3_02) {
> /*
> * This was introduced with SMB3_02
> @@ -696,6 +699,7 @@ static struct tevent_req *smbd_smb2_create_send(TALLOC_CTX *mem_ctx,
> svhdx = smb2_create_blob_find(&in_context_blobs,
> SVHDX_OPEN_DEVICE_CONTEXT);
> }
> +#endif
>
> fname = talloc_strdup(state, in_name);
> if (tevent_req_nomem(fname, req)) {
> --
> 1.9.1
>
More information about the samba-technical
mailing list