2007-01-27 15:05:32

by Michal Piotrowski

[permalink] [raw]
Subject: Re: mm snapshot broken-out-2007-01-26-00-36.tar.gz uploaded

[email protected] napisał(a):
> The mm snapshot broken-out-2007-01-26-00-36.tar.gz has been uploaded to
>
> ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/mm/broken-out-2007-01-26-00-36.tar.gz
>

fs/ecryptfs/crypto.c: In function ‘ecryptfs_init_crypt_ctx’:
fs/ecryptfs/crypto.c:831: error: ‘CRYPTO_TFM_MODE_CBC’ undeclared (first use in this function)
fs/ecryptfs/crypto.c:831: error: (Each undeclared identifier is reported only once
fs/ecryptfs/crypto.c:831: error: for each function it appears in.)
make[2]: *** [fs/ecryptfs/crypto.o] Błąd 1
make[1]: *** [fs/ecryptfs] Błąd 2
make: *** [fs] Błąd 2

git-cryptodev.patch removes CRYPTO_TFM_MODE_CBC but it is still defined as ECRYPTFS_DEFAULT_CHAINING_MODE
in fs/ecryptfs/ecryptfs_kernel.h

What should be a new ECRYPTFS_DEFAULT_CHAINING_MODE ?

Regards,
Michal

--
Michal K. K. Piotrowski
LTG - Linux Testers Group
(http://www.stardust.webpages.pl/ltg/)


2007-01-27 20:38:46

by Herbert Xu

[permalink] [raw]
Subject: Re: mm snapshot broken-out-2007-01-26-00-36.tar.gz uploaded

On Sat, Jan 27, 2007 at 04:05:19PM +0100, Michal Piotrowski wrote:
>
> git-cryptodev.patch removes CRYPTO_TFM_MODE_CBC but it is still defined as ECRYPTFS_DEFAULT_CHAINING_MODE
> in fs/ecryptfs/ecryptfs_kernel.h
>
> What should be a new ECRYPTFS_DEFAULT_CHAINING_MODE ?

It should simply be removed.

Cheers,
--
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <[email protected]>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
--
On Sat, Jan 27, 2007 at 09:49:21AM +1100, Herbert Xu wrote:
>
> That macro has been obsolete for ages. I wonder why ecryptfs is still
> using it. Oh well I'll restore it for now.

What I ended up doing is adding the following patch to cryptodev-2.6.
So if you pull again it should be there now.

Cheers,
--
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <[email protected]>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
--
diff --git a/fs/ecryptfs/crypto.c b/fs/ecryptfs/crypto.c
index 7196f50..a86a55c 100644
--- a/fs/ecryptfs/crypto.c
+++ b/fs/ecryptfs/crypto.c
@@ -828,9 +828,7 @@ int ecryptfs_init_crypt_ctx(struct ecryptfs_crypt_stat *crypt_stat)
mutex_unlock(&crypt_stat->cs_tfm_mutex);
goto out;
}
- crypto_blkcipher_set_flags(crypt_stat->tfm,
- (ECRYPTFS_DEFAULT_CHAINING_MODE
- | CRYPTO_TFM_REQ_WEAK_KEY));
+ crypto_blkcipher_set_flags(crypt_stat->tfm, CRYPTO_TFM_REQ_WEAK_KEY);
mutex_unlock(&crypt_stat->cs_tfm_mutex);
rc = 0;
out:
diff --git a/fs/ecryptfs/ecryptfs_kernel.h b/fs/ecryptfs/ecryptfs_kernel.h
index afb64bd..0f89710 100644
--- a/fs/ecryptfs/ecryptfs_kernel.h
+++ b/fs/ecryptfs/ecryptfs_kernel.h
@@ -176,7 +176,6 @@ ecryptfs_get_key_payload_data(struct key *key)
#define ECRYPTFS_FILE_SIZE_BYTES 8
#define ECRYPTFS_DEFAULT_CIPHER "aes"
#define ECRYPTFS_DEFAULT_KEY_BYTES 16
-#define ECRYPTFS_DEFAULT_CHAINING_MODE CRYPTO_TFM_MODE_CBC
#define ECRYPTFS_DEFAULT_HASH "md5"
#define ECRYPTFS_TAG_3_PACKET_TYPE 0x8C
#define ECRYPTFS_TAG_11_PACKET_TYPE 0xED