Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 31b59d11 authored by Nicolas Iooss's avatar Nicolas Iooss Committed by Gerrit - the friendly Code Review server
Browse files

apparmor: use SHASH_DESC_ON_STACK 



When building the kernel with clang, the compiler fails to build
security/apparmor/crypto.c with the following error:

    security/apparmor/crypto.c:36:8: error: fields must have a constant
    size: 'variable length array in structure' extension will never be
    supported
                    char ctx[crypto_shash_descsize(apparmor_tfm)];
                         ^

Since commit a0a77af1 ("crypto: LLVMLinux: Add macro to remove use
of VLAIS in crypto code"), include/crypto/hash.h defines
SHASH_DESC_ON_STACK to work around this issue. Use it in aa_calc_hash()
and aa_calc_profile_hash().

Change-Id: I7668f8c6440660e7b68afd8aefd066e496f3beb4
Signed-off-by: default avatarNicolas Iooss <nicolas.iooss_linux@m4x.org>
Signed-off-by: default avatarJohn Johansen <john.johansen@canonical.com>
Signed-off-by: default avatarJames Morris <james.l.morris@oracle.com>
Git-commit: 9814448da7a84dd50b69e4ada2d7d1c042493daf
Git-repo: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git


Signed-off-by: default avatarShiraz Hashim <shashim@codeaurora.org>
parent dbb19355

security/apparmor/crypto.c

+7 −10
Original line number Diff line number Diff line
@@ -32,10 +32,7 @@ unsigned int aa_hash_size(void) 
int aa_calc_profile_hash(struct aa_profile *profile, u32 version, void *start,

			 size_t len)

{

	struct {

		struct shash_desc shash;

		char ctx[crypto_shash_descsize(apparmor_tfm)];

	} desc;

	SHASH_DESC_ON_STACK(desc, apparmor_tfm);

	int error = -ENOMEM;

	u32 le32_version = cpu_to_le32(version);
@@ -49,19 +46,19 @@ int aa_calc_profile_hash(struct aa_profile *profile, u32 version, void *start, 
	if (!profile->hash)

		goto fail;



	desc.shash.tfm = apparmor_tfm;

	desc.shash.flags = 0;

	desc->tfm = apparmor_tfm;

	desc->flags = 0;



	error = crypto_shash_init(&desc.shash);

	error = crypto_shash_init(desc);

	if (error)

		goto fail;

	error = crypto_shash_update(&desc.shash, (u8 *) &le32_version, 4);

	error = crypto_shash_update(desc, (u8 *) &le32_version, 4);

	if (error)

		goto fail;

	error = crypto_shash_update(&desc.shash, (u8 *) start, len);

	error = crypto_shash_update(desc, (u8 *) start, len);

	if (error)

		goto fail;

	error = crypto_shash_final(&desc.shash, profile->hash);

	error = crypto_shash_final(desc, profile->hash);

	if (error)

		goto fail;