Commit 3339d808 authored Oct 21, 2022 by Xiu Jianfeng Committed by Greg Kroah-Hartman Jan 18, 2023

apparmor: Use pointer to struct aa_label for lbs_cred



[ Upstream commit 37923d4321b1e38170086da2c117f78f2b0f49c6 ]

According to the implementations of cred_label() and set_cred_label(),
we should use pointer to struct aa_label for lbs_cred instead of struct
aa_task_ctx, this patch fixes it.

Fixes: bbd3662a ("Infrastructure management of the cred security blob")
Signed-off-by: Xiu Jianfeng <xiujianfeng@huawei.com>
Signed-off-by: John Johansen <john.johansen@canonical.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>

parent f7368ac8

security/apparmor/lsm.c

+2 −2

Original line number	Diff line number	Diff line
		@@ -1148,10 +1148,10 @@ static int apparmor_inet_conn_request(struct sock sk, struct sk_buff skb,
		#endif

		/*
		* The cred blob is a pointer to, not an instance of, an aa_task_ctx.
		* The cred blob is a pointer to, not an instance of, an aa_label.
		*/
		struct lsm_blob_sizes apparmor_blob_sizes __lsm_ro_after_init = {
		.lbs_cred = sizeof(struct aa_task_ctx *),
		.lbs_cred = sizeof(struct aa_label *),
		.lbs_file = sizeof(struct aa_file_ctx),
		.lbs_task = sizeof(struct aa_task_ctx),
		};