Re: [RFC PATCH v2 14/27] libselinux: parameter simplifications

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Oct 5, 2023 at 11:39 AM James Carter <jwcart2@xxxxxxxxx> wrote:
>
> On Mon, Aug 14, 2023 at 9:41 AM Christian Göttsche
> <cgzones@xxxxxxxxxxxxxx> wrote:
> >
> > Use bool where applicable.
> >
> > Signed-off-by: Christian Göttsche <cgzones@xxxxxxxxxxxxxx>
>
> Acked-by: James Carter <jwcart2@xxxxxxxxx>
>

Merged.
Thanks,
Jim

> > ---
> >  libselinux/src/label.c | 16 ++++++++--------
> >  1 file changed, 8 insertions(+), 8 deletions(-)
> >
> > diff --git a/libselinux/src/label.c b/libselinux/src/label.c
> > index c0f586a2..9a972f79 100644
> > --- a/libselinux/src/label.c
> > +++ b/libselinux/src/label.c
> > @@ -137,9 +137,9 @@ out:
> >  }
> >
> >  /* Public API helpers */
> > -static int selabel_fini(struct selabel_handle *rec,
> > +static int selabel_fini(const struct selabel_handle *rec,
> >                             struct selabel_lookup_rec *lr,
> > -                           int translating)
> > +                           bool translating)
> >  {
> >         if (compat_validate(rec, lr, rec->spec_file, lr->lineno))
> >                 return -1;
> > @@ -152,7 +152,7 @@ static int selabel_fini(struct selabel_handle *rec,
> >  }
> >
> >  static struct selabel_lookup_rec *
> > -selabel_lookup_common(struct selabel_handle *rec, int translating,
> > +selabel_lookup_common(struct selabel_handle *rec, bool translating,
> >                       const char *key, int type)
> >  {
> >         struct selabel_lookup_rec *lr;
> > @@ -173,7 +173,7 @@ selabel_lookup_common(struct selabel_handle *rec, int translating,
> >  }
> >
> >  static struct selabel_lookup_rec *
> > -selabel_lookup_bm_common(struct selabel_handle *rec, int translating,
> > +selabel_lookup_bm_common(struct selabel_handle *rec, bool translating,
> >                       const char *key, int type, const char **aliases)
> >  {
> >         struct selabel_lookup_rec *lr;
> > @@ -239,7 +239,7 @@ int selabel_lookup(struct selabel_handle *rec, char **con,
> >  {
> >         struct selabel_lookup_rec *lr;
> >
> > -       lr = selabel_lookup_common(rec, 1, key, type);
> > +       lr = selabel_lookup_common(rec, true, key, type);
> >         if (!lr)
> >                 return -1;
> >
> > @@ -252,7 +252,7 @@ int selabel_lookup_raw(struct selabel_handle *rec, char **con,
> >  {
> >         struct selabel_lookup_rec *lr;
> >
> > -       lr = selabel_lookup_common(rec, 0, key, type);
> > +       lr = selabel_lookup_common(rec, false, key, type);
> >         if (!lr)
> >                 return -1;
> >
> > @@ -307,7 +307,7 @@ int selabel_lookup_best_match(struct selabel_handle *rec, char **con,
> >                 return -1;
> >         }
> >
> > -       lr = selabel_lookup_bm_common(rec, 1, key, type, aliases);
> > +       lr = selabel_lookup_bm_common(rec, true, key, type, aliases);
> >         if (!lr)
> >                 return -1;
> >
> > @@ -325,7 +325,7 @@ int selabel_lookup_best_match_raw(struct selabel_handle *rec, char **con,
> >                 return -1;
> >         }
> >
> > -       lr = selabel_lookup_bm_common(rec, 0, key, type, aliases);
> > +       lr = selabel_lookup_bm_common(rec, false, key, type, aliases);
> >         if (!lr)
> >                 return -1;
> >
> > --
> > 2.40.1
> >




[Index of Archives]     [Selinux Refpolicy]     [Linux SGX]     [Fedora Users]     [Fedora Desktop]     [Yosemite Photos]     [Yosemite Camping]     [Yosemite Campsites]     [KDE Users]     [Gnome Users]

  Powered by Linux