Re: [PATCH v7 09/17] KEYS: Rename get_builtin_and_secondary_restriction

From: Mimi Zohar
Date: Thu Nov 18 2021 - 19:07:05 EST


Hi Eric,

On Mon, 2021-11-15 at 19:15 -0500, Eric Snowberg wrote:
> To improve clarity, rename get_builtin_and_secondary_restriction to
> get_secondary_restriction.

The existing name clarity is fine. Perhaps instead prefix the above
sentence with "In preparation for returning either the existing
restrict_link_by_builtin_and_secondary_trusted or the new restriction
that includes the trusted builtin, secondary and machine keys, ..."
>
> Suggested-by: Mimi Zohar <zohar@xxxxxxxxxxxxx>
> Signed-off-by: Eric Snowberg <eric.snowberg@xxxxxxxxxx>

Otherwise,
Reviewed-by: Mimi Zohar <zohar@xxxxxxxxxxxxx>