Messages in this thread | | | Date | Wed, 09 Aug 2023 19:07:42 -0400 | From | Paul Moore <> | Subject | Re: [PATCH v3 3/7] selinux: update type for number of class permissions in services code |
| |
On Aug 7, 2023 =?UTF-8?q?Christian=20G=C3=B6ttsche?= <cgzones@googlemail.com> wrote: > > Security classes have only up to 32 permissions, hence using an u16 is > sufficient (while improving padding in struct selinux_mapping). > > Signed-off-by: Christian Göttsche <cgzones@googlemail.com> > --- > v3: > - drop type change of arithmetic variable; it might effect performance > as suggested by David. > - split bogus and corrected cast into separate patch > v2: > update commit description: > - mention struct selinux_mapping in the padding argument > (currently between the first and second member there are 2 bytes > padding) > - mention overflow in the cast argument and the result of setting > no bits due to it > --- > security/selinux/ss/services.c | 2 +- > security/selinux/ss/services.h | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-)
Merged into selinux/next, thanks.
-- paul-moore.com
| |