On Tue, Nov 20, 2018 at 10:47 PM Paul Moore <paul@xxxxxxxxxxxxxx> wrote: > On Tue, Nov 13, 2018 at 8:53 AM Ondrej Mosnacek <omosnace@xxxxxxxxxx> wrote: > > This is a purely cosmetic change that encapsulates the three-step sidtab > > conversion logic (shutdown -> clone -> map) into a single function > > defined in sidtab.c (as opposed to services.c). > > > > Signed-off-by: Ondrej Mosnacek <omosnace@xxxxxxxxxx> > > --- > > security/selinux/ss/services.c | 22 +-------------- > > security/selinux/ss/sidtab.c | 50 ++++++++++++++++++++++++---------- > > security/selinux/ss/sidtab.h | 11 ++++---- > > 3 files changed, 42 insertions(+), 41 deletions(-) > > Merged into selinux/next with some whitespace fixes (inherited from > code you cut n' pasted). Please remember to run your patches through > scripts/checkpatch.pl before submission. Damn, I still haven't set up that commit hook... Done now, seems to work fine. Sorry for not having done that sooner and thanks for nagging me about it :) -- Ondrej Mosnacek <omosnace at redhat dot com> Associate Software Engineer, Security Technologies Red Hat, Inc.
