[PATCH -next] apparmor: Remove unused parameter L1 in macro next_comb

John Johansen john.johansen at canonical.com
Sat Nov 9 20:39:27 UTC 2024


On 9/2/24 00:39, Jinjie Ruan wrote:
> In the macro definition of next_comb(), a parameter L1 is accepted,
> but it is not used. Hence, it should be removed.
> 
> Signed-off-by: Jinjie Ruan <ruanjinjie at huawei.com>

Acked-by: John Johansen <john.johansen at canonical.com>

I have pulled this into my tree

> ---
>   security/apparmor/include/label.h | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/security/apparmor/include/label.h b/security/apparmor/include/label.h
> index 2a72e6b17d68..2635eef0ae2b 100644
> --- a/security/apparmor/include/label.h
> +++ b/security/apparmor/include/label.h
> @@ -160,7 +160,7 @@ int aa_label_next_confined(struct aa_label *l, int i);
>   #define label_for_each_cont(I, L, P)					\
>   	for (++((I).i); ((P) = (L)->vec[(I).i]); ++((I).i))
>   
> -#define next_comb(I, L1, L2)						\
> +#define next_comb(I, L2)						\
>   do {									\
>   	(I).j++;							\
>   	if ((I).j >= (L2)->size) {					\
> @@ -174,7 +174,7 @@ do {									\
>   #define label_for_each_comb(I, L1, L2, P1, P2)				\
>   for ((I).i = (I).j = 0;							\
>   	((P1) = (L1)->vec[(I).i]) && ((P2) = (L2)->vec[(I).j]);		\
> -	(I) = next_comb(I, L1, L2))
> +	(I) = next_comb(I, L2))
>   
>   #define fn_for_each_comb(L1, L2, P1, P2, FN)				\
>   ({									\




More information about the Linux-security-module-archive mailing list