SMB3: GUIDs should be constructed as random but valid uuids
GUIDs although random, and 16 bytes, need to be generated as proper uuids. Signed-off-by: Steve French <steve.french@primarydata.com> Reviewed-by: Aurelien Aptel <aaptel@suse.com> Reported-by: David Goebels <davidgoe@microsoft.com> CC: Stable <stable@vger.kernel.org>
This commit is contained in:
parent
c2afb8147e
commit
fa70b87cc6
|
@ -271,7 +271,7 @@ cifs_alloc_inode(struct super_block *sb)
|
|||
cifs_inode->createtime = 0;
|
||||
cifs_inode->epoch = 0;
|
||||
#ifdef CONFIG_CIFS_SMB2
|
||||
get_random_bytes(cifs_inode->lease_key, SMB2_LEASE_KEY_SIZE);
|
||||
generate_random_uuid(cifs_inode->lease_key);
|
||||
#endif
|
||||
/*
|
||||
* Can not set i_flags here - they get immediately overwritten to zero
|
||||
|
|
|
@ -2163,7 +2163,7 @@ cifs_get_tcp_session(struct smb_vol *volume_info)
|
|||
memcpy(&tcp_ses->dstaddr, &volume_info->dstaddr,
|
||||
sizeof(tcp_ses->dstaddr));
|
||||
#ifdef CONFIG_CIFS_SMB2
|
||||
get_random_bytes(tcp_ses->client_guid, SMB2_CLIENT_GUID_SIZE);
|
||||
generate_random_uuid(tcp_ses->client_guid);
|
||||
#endif
|
||||
/*
|
||||
* at this point we are the only ones with the pointer
|
||||
|
|
|
@ -1041,7 +1041,7 @@ smb2_set_lease_key(struct inode *inode, struct cifs_fid *fid)
|
|||
static void
|
||||
smb2_new_lease_key(struct cifs_fid *fid)
|
||||
{
|
||||
get_random_bytes(fid->lease_key, SMB2_LEASE_KEY_SIZE);
|
||||
generate_random_uuid(fid->lease_key);
|
||||
}
|
||||
|
||||
#define SMB2_SYMLINK_STRUCT_SIZE \
|
||||
|
|
|
@ -1183,7 +1183,7 @@ create_durable_v2_buf(struct cifs_fid *pfid)
|
|||
|
||||
buf->dcontext.Timeout = 0; /* Should this be configurable by workload */
|
||||
buf->dcontext.Flags = cpu_to_le32(SMB2_DHANDLE_FLAG_PERSISTENT);
|
||||
get_random_bytes(buf->dcontext.CreateGuid, 16);
|
||||
generate_random_uuid(buf->dcontext.CreateGuid);
|
||||
memcpy(pfid->create_guid, buf->dcontext.CreateGuid, 16);
|
||||
|
||||
/* SMB2_CREATE_DURABLE_HANDLE_REQUEST is "DH2Q" */
|
||||
|
|
Loading…
Reference in New Issue