2017-03-30 10:37:40

by Tobias Regnery

[permalink] [raw]
Subject: [PATCH] CIFS: Fix build failure with smb2

I saw the following build error during a randconfig build:

fs/cifs/smb2ops.c: In function 'smb2_new_lease_key':
fs/cifs/smb2ops.c:1104:2: error: implicit declaration of function 'generate_random_uuid' [-Werror=implicit-function-declaration]

Explicit include the right header to fix this issue.

Signed-off-by: Tobias Regnery <[email protected]>
---
This is against next-20170330

fs/cifs/smb2ops.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/fs/cifs/smb2ops.c b/fs/cifs/smb2ops.c
index c6d14cc44510..c1b70559496c 100644
--- a/fs/cifs/smb2ops.c
+++ b/fs/cifs/smb2ops.c
@@ -21,6 +21,7 @@
#include <linux/vfs.h>
#include <linux/falloc.h>
#include <linux/scatterlist.h>
+#include <linux/uuid.h>
#include <crypto/aead.h>
#include "cifsglob.h"
#include "smb2pdu.h"
--
2.11.0


2017-03-30 11:47:54

by Aurélien Aptel

[permalink] [raw]
Subject: Re: [PATCH] CIFS: Fix build failure with smb2

Tobias Regnery <[email protected]> writes:
> Explicit include the right header to fix this issue.

uuid.h indeed declares generate_random_uuid(). Looks good to me.

Reviewed-by: Aurelien Aptel <[email protected]>

--
Aurélien Aptel / SUSE Labs Samba Team
GPG: 1839 CB5F 9F5B FB9B AA97 8C99 03C8 A49B 521B D5D3
SUSE Linux GmbH, Maxfeldstraße 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Jane Smithard, Graham Norton, HRB 21284 (AG Nürnberg)

2017-04-04 07:24:13

by Steve French

[permalink] [raw]
Subject: Re: [PATCH] CIFS: Fix build failure with smb2

merged into cifs-2.6.git

thx

On Thu, Mar 30, 2017 at 6:47 AM, Aurélien Aptel
<[email protected]> wrote:
> Tobias Regnery <[email protected]> writes:
>> Explicit include the right header to fix this issue.
>
> uuid.h indeed declares generate_random_uuid(). Looks good to me.
>
> Reviewed-by: Aurelien Aptel <[email protected]>
>
> --
> Aurélien Aptel / SUSE Labs Samba Team
> GPG: 1839 CB5F 9F5B FB9B AA97 8C99 03C8 A49B 521B D5D3
> SUSE Linux GmbH, Maxfeldstraße 5, 90409 Nürnberg, Germany
> GF: Felix Imendörffer, Jane Smithard, Graham Norton, HRB 21284 (AG Nürnberg)
> --
> To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html



--
Thanks,

Steve