Re: [PATCH 1/2] selinux: constify source policy in cond_policydb_dup()

From: Paul Moore
Date: Tue Apr 30 2024 - 18:55:18 EST


On Apr 5, 2024 =?UTF-8?q?Christian=20G=C3=B6ttsche?= <cgoettsche@xxxxxxxxxxxxx> wrote:
>
> cond_policydb_dup() duplicates conditional parts of an existing policy.
> Declare the source policy const, since it should not be modified.
>
> Signed-off-by: Christian Göttsche <cgzones@xxxxxxxxxxxxxx>
> ---
> security/selinux/ss/conditional.c | 12 ++++++------
> security/selinux/ss/conditional.h | 2 +-
> security/selinux/ss/hashtab.c | 9 +++++----
> security/selinux/ss/hashtab.h | 4 ++--
> 4 files changed, 14 insertions(+), 13 deletions(-)

I had to do some line length fixups, but otherwise this looked good
to me, merged into selinux/dev. Thanks!

--
paul-moore.com