[PATCH -next] cifs: remove set but not used variable 'smb_buf'
Steve French
smfrench at gmail.com
Fri Dec 7 07:27:42 UTC 2018
merged into cifs-2.6.git for-next
On Fri, Dec 7, 2018 at 12:57 AM YueHaibing <yuehaibing at huawei.com> wrote:
>
> Fixes gcc '-Wunused-but-set-variable' warning:
>
> fs/cifs/sess.c: In function '_sess_auth_rawntlmssp_assemble_req':
> fs/cifs/sess.c:1157:18: warning:
> variable 'smb_buf' set but not used [-Wunused-but-set-variable]
>
> It never used since commit cc87c47d9d7a ("cifs: Separate rawntlmssp auth
> from CIFS_SessSetup()")
>
> Signed-off-by: YueHaibing <yuehaibing at huawei.com>
> ---
> fs/cifs/sess.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/fs/cifs/sess.c b/fs/cifs/sess.c
> index 7fa447a..dcd49ad 100644
> --- a/fs/cifs/sess.c
> +++ b/fs/cifs/sess.c
> @@ -1154,14 +1154,12 @@ struct sess_data {
> static int
> _sess_auth_rawntlmssp_assemble_req(struct sess_data *sess_data)
> {
> - struct smb_hdr *smb_buf;
> SESSION_SETUP_ANDX *pSMB;
> struct cifs_ses *ses = sess_data->ses;
> __u32 capabilities;
> char *bcc_ptr;
>
> pSMB = (SESSION_SETUP_ANDX *)sess_data->iov[0].iov_base;
> - smb_buf = (struct smb_hdr *)pSMB;
>
> capabilities = cifs_ssetup_hdr(ses, pSMB);
> if ((pSMB->req.hdr.Flags2 & SMBFLG2_UNICODE) == 0) {
>
>
>
--
Thanks,
Steve
More information about the samba-technical
mailing list