Re: [PATCH RFC 8/20] selinux: services: avoid implicit conversions

From: Paul Moore
Date: Tue Jul 18 2023 - 18:01:48 EST


On Jul 6, 2023 =?UTF-8?q?Christian=20G=C3=B6ttsche?= <cgzones@xxxxxxxxxxxxxx> wrote:
>
> Use u32 as the output parameter type in security_get_classes() and
> security_get_permissions(), based on the type of the symtab nprim
> member.
>
> Declare the read-only class string parameter of
> security_get_permissions() const.
>
> Avoid several implicit conversions by using the identical type for the
> destination.
>
> Signed-off-by: Christian Göttsche <cgzones@xxxxxxxxxxxxxx>
> ---
> security/selinux/include/security.h | 4 ++--
> security/selinux/selinuxfs.c | 7 ++++---
> security/selinux/ss/services.c | 22 +++++++++-------------
> 3 files changed, 15 insertions(+), 18 deletions(-)

More loop iterators ...

--
paul-moore.com