Re: [PATCH v2 4/6] libsepol/cil: comment out unused function __cil_verify_rule

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

 



On Wed, Apr 6, 2022 at 12:15 PM James Carter <jwcart2@xxxxxxxxx> wrote:
>
> On Wed, Apr 6, 2022 at 9:28 AM Christian Göttsche
> <cgzones@xxxxxxxxxxxxxx> wrote:
> >
> > The function __cil_verify_rule() is currently not used as all call sites
> > are commented out.  Keep the function for future references.
> >
> > Signed-off-by: Christian Göttsche <cgzones@xxxxxxxxxxxxxx>
>
> Patches 1-3 have already been merged.
>
> For patches 4-6:
> Acked-by: James Carter <jwcart2@xxxxxxxxx>
>

Now that the libsepol and libsemanage tests work for
"-Wmissing-prototypes", patches 4-6 have been merged.
Thanks,
Jim

> > ---
> >  libsepol/cil/src/cil_verify.c | 4 +++-
> >  1 file changed, 3 insertions(+), 1 deletion(-)
> >
> > diff --git a/libsepol/cil/src/cil_verify.c b/libsepol/cil/src/cil_verify.c
> > index aecbdc19..4640dc59 100644
> > --- a/libsepol/cil/src/cil_verify.c
> > +++ b/libsepol/cil/src/cil_verify.c
> > @@ -970,7 +970,8 @@ exit:
> >         return rc;
> >  }
> >
> > -int __cil_verify_rule(struct cil_tree_node *node, struct cil_complex_symtab *symtab)
> > +/*
> > +static int __cil_verify_rule(struct cil_tree_node *node, struct cil_complex_symtab *symtab)
> >  {
> >
> >         int rc = SEPOL_ERR;
> > @@ -1016,6 +1017,7 @@ exit:
> >         cil_tree_log(node, CIL_ERR, "Invalid rule");
> >         return rc;
> >  }
> > +*/
> >
> >  static int __cil_verify_booleanif_helper(struct cil_tree_node *node, __attribute__((unused)) uint32_t *finished, __attribute__((unused)) void *extra_args)
> >  {
> > --
> > 2.35.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