[PATCH] cifs: make CIFS depend on CRYPTO_ECB

[PATCH] cifs: make CIFS depend on CRYPTO_ECB

[Suresh Jayaraman]

When CONFIG_CRYPTO_ECB is not set, trying to mount a CIFS share with NTLM
security resulted in mount failure with the following error:
   "CIFS VFS: could not allocate des crypto API"

Seems like a leftover from commit 43988d7.

Signed-off-by: Suresh Jayaraman <sjayaraman-l3A5Bk7waGM@public.gmane.org>
---
 fs/cifs/Kconfig |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/fs/cifs/Kconfig b/fs/cifs/Kconfig
index 1cd4c3a..66c68ab 100644
--- a/fs/cifs/Kconfig
+++ b/fs/cifs/Kconfig
@@ -7,6 +7,7 @@ config CIFS
 	select CRYPTO_MD5
 	select CRYPTO_HMAC
 	select CRYPTO_ARC4
+	select CRYPTO_ECB
 	select CRYPTO_DES
 	help
 	  This is the client VFS module for the Common Internet File System
-- 
1.7.1

Re: [PATCH] cifs: make CIFS depend on CRYPTO_ECB

[Jeff Layton]

On Fri, 03 Jun 2011 14:19:01 +0530
Suresh Jayaraman <sjayaraman-l3A5Bk7waGM@public.gmane.org> wrote:

> When CONFIG_CRYPTO_ECB is not set, trying to mount a CIFS share with NTLM
> security resulted in mount failure with the following error:
>    "CIFS VFS: could not allocate des crypto API"
> 
> Seems like a leftover from commit 43988d7.
> 
> Signed-off-by: Suresh Jayaraman <sjayaraman-l3A5Bk7waGM@public.gmane.org>
> ---
>  fs/cifs/Kconfig |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/fs/cifs/Kconfig b/fs/cifs/Kconfig
> index 1cd4c3a..66c68ab 100644
> --- a/fs/cifs/Kconfig
> +++ b/fs/cifs/Kconfig
> @@ -7,6 +7,7 @@ config CIFS
>  	select CRYPTO_MD5
>  	select CRYPTO_HMAC
>  	select CRYPTO_ARC4
> +	select CRYPTO_ECB
>  	select CRYPTO_DES
>  	help
>  	  This is the client VFS module for the Common Internet File System

Reviewed-by: Jeff Layton <jlayton-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>

Re: [PATCH] cifs: make CIFS depend on CRYPTO_ECB

[Steve French]

Merged.  This is a strange dependency - it looks like thee real
problem is that the two crypto libs we depend on that depend on this
don't select ECB but it is harmless to add the select.

On Fri, Jun 3, 2011 at 8:10 AM, Jeff Layton <jlayton-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> wrote:
> On Fri, 03 Jun 2011 14:19:01 +0530
> Suresh Jayaraman <sjayaraman-l3A5Bk7waGM@public.gmane.org> wrote:
>
>> When CONFIG_CRYPTO_ECB is not set, trying to mount a CIFS share with NTLM
>> security resulted in mount failure with the following error:
>>    "CIFS VFS: could not allocate des crypto API"
>>
>> Seems like a leftover from commit 43988d7.
>>
>> Signed-off-by: Suresh Jayaraman <sjayaraman-l3A5Bk7waGM@public.gmane.org>
>> ---
>>  fs/cifs/Kconfig |    1 +
>>  1 files changed, 1 insertions(+), 0 deletions(-)
>>
>> diff --git a/fs/cifs/Kconfig b/fs/cifs/Kconfig
>> index 1cd4c3a..66c68ab 100644
>> --- a/fs/cifs/Kconfig
>> +++ b/fs/cifs/Kconfig
>> @@ -7,6 +7,7 @@ config CIFS
>>       select CRYPTO_MD5
>>       select CRYPTO_HMAC
>>       select CRYPTO_ARC4
>> +     select CRYPTO_ECB
>>       select CRYPTO_DES
>>       help
>>         This is the client VFS module for the Common Internet File System
>
> Reviewed-by: Jeff Layton <jlayton-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
>



-- 
Thanks,

Steve

Re: [PATCH] cifs: make CIFS depend on CRYPTO_ECB

[Jeff Layton]

On Fri, 3 Jun 2011 12:39:41 -0500
Steve French <smfrench-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:

> Merged.  This is a strange dependency - it looks like thee real
> problem is that the two crypto libs we depend on that depend on this
> don't select ECB but it is harmless to add the select.
> 

Hmm...then maybe the patch should this dependency there so that other
kernel subsystems don't get bitten by the same problem?

> On Fri, Jun 3, 2011 at 8:10 AM, Jeff Layton <jlayton-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> wrote:
> > On Fri, 03 Jun 2011 14:19:01 +0530
> > Suresh Jayaraman <sjayaraman-l3A5Bk7waGM@public.gmane.org> wrote:
> >
> >> When CONFIG_CRYPTO_ECB is not set, trying to mount a CIFS share with NTLM
> >> security resulted in mount failure with the following error:
> >>    "CIFS VFS: could not allocate des crypto API"
> >>
> >> Seems like a leftover from commit 43988d7.
> >>
> >> Signed-off-by: Suresh Jayaraman <sjayaraman-l3A5Bk7waGM@public.gmane.org>
> >> ---
> >>  fs/cifs/Kconfig |    1 +
> >>  1 files changed, 1 insertions(+), 0 deletions(-)
> >>
> >> diff --git a/fs/cifs/Kconfig b/fs/cifs/Kconfig
> >> index 1cd4c3a..66c68ab 100644
> >> --- a/fs/cifs/Kconfig
> >> +++ b/fs/cifs/Kconfig
> >> @@ -7,6 +7,7 @@ config CIFS
> >>       select CRYPTO_MD5
> >>       select CRYPTO_HMAC
> >>       select CRYPTO_ARC4
> >> +     select CRYPTO_ECB
> >>       select CRYPTO_DES
> >>       help
> >>         This is the client VFS module for the Common Internet File System
> >
> > Reviewed-by: Jeff Layton <jlayton-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
> >
> 
> 
> 


-- 
Jeff Layton <jlayton-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>

Re: [PATCH] cifs: make CIFS depend on CRYPTO_ECB

[Suresh Jayaraman]

On 06/03/2011 11:32 PM, Jeff Layton wrote:
> On Fri, 3 Jun 2011 12:39:41 -0500
> Steve French <smfrench-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
> 
>> Merged.  This is a strange dependency - it looks like thee real
>> problem is that the two crypto libs we depend on that depend on this
>> don't select ECB but it is harmless to add the select.
>>
> 
> Hmm...then maybe the patch should this dependency there so that other
> kernel subsystems don't get bitten by the same problem?

I looked at a few in-tree examples:

>From drivers/net/Kconfig

config PPP_MPPE
        tristate "PPP MPPE compression (encryption) (EXPERIMENTAL)"
        depends on PPP && EXPERIMENTAL
        select CRYPTO
        select CRYPTO_SHA1
        select CRYPTO_ARC4
        select CRYPTO_ECB
(we seem to have both ARC4 and ECB explicitly)

>From net/sunrpc/Kconfig:

config RPCSEC_GSS_KRB5
        tristate "Secure RPC: Kerberos V mechanism"
        depends on SUNRPC && CRYPTO
        depends on CRYPTO_MD5 && CRYPTO_DES && CRYPTO_CBC && CRYPTO_CTS
        depends on CRYPTO_ECB && CRYPTO_HMAC && CRYPTO_SHA1 && CRYPTO_AES
        depends on CRYPTO_ARC4

>From drivers/crypto/Kconfig:

config CRYPTO_DEV_PICOXCELL
        tristate "Support for picoXcell IPSEC and Layer2 crypto engines"
        depends on ARCH_PICOXCELL
        select CRYPTO_AES
        select CRYPTO_AUTHENC
        select CRYPTO_ALGAPI
        select CRYPTO_DES
        select CRYPTO_CBC
        select CRYPTO_ECB
        select CRYPTO_SEQIV

and others in 
 drivers/net/wireless/hostap/Kconfig
 drivers/net/wireless/ipw2x00/Kconfig 

Looks like none of them will be affected as they use explicitly. However,
it is not clear to me why they the crypto lib itself is not marking the
dependency..

>> On Fri, Jun 3, 2011 at 8:10 AM, Jeff Layton <jlayton-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> wrote:
>>> On Fri, 03 Jun 2011 14:19:01 +0530
>>> Suresh Jayaraman <sjayaraman-l3A5Bk7waGM@public.gmane.org> wrote:
>>>
>>>> When CONFIG_CRYPTO_ECB is not set, trying to mount a CIFS share with NTLM
>>>> security resulted in mount failure with the following error:
>>>>    "CIFS VFS: could not allocate des crypto API"
>>>>
>>>> Seems like a leftover from commit 43988d7.
>>>>
>>>> Signed-off-by: Suresh Jayaraman <sjayaraman-l3A5Bk7waGM@public.gmane.org>
>>>> ---
>>>>  fs/cifs/Kconfig |    1 +
>>>>  1 files changed, 1 insertions(+), 0 deletions(-)
>>>>
>>>> diff --git a/fs/cifs/Kconfig b/fs/cifs/Kconfig
>>>> index 1cd4c3a..66c68ab 100644
>>>> --- a/fs/cifs/Kconfig
>>>> +++ b/fs/cifs/Kconfig
>>>> @@ -7,6 +7,7 @@ config CIFS
>>>>       select CRYPTO_MD5
>>>>       select CRYPTO_HMAC
>>>>       select CRYPTO_ARC4
>>>> +     select CRYPTO_ECB
>>>>       select CRYPTO_DES
>>>>       help
>>>>         This is the client VFS module for the Common Internet File System
>>>
>>> Reviewed-by: Jeff Layton <jlayton-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
>>>
>>
>>
>>
> 
> 


-- 
Suresh Jayaraman