On Fri, 3 Jun 2011 12:39:41 -0500 Steve French <[email protected]> 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 <[email protected]> wrote: > > On Fri, 03 Jun 2011 14:19:01 +0530 > > Suresh Jayaraman <[email protected]> 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 <[email protected]> > >> --- > >> 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 <[email protected]> > > > > > -- Jeff Layton <[email protected]> -- 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
