mbox series

[0/6] More secilc-fuzzer problems fixed

Message ID 20210513185254.559588-1-jwcart2@gmail.com (mailing list archive)
Headers show
Series More secilc-fuzzer problems fixed | expand

Message

James Carter May 13, 2021, 6:52 p.m. UTC 
Patches 1-4 fix bug found directly by the secilc-fuzzer.
Patch 6 is the same class of bug as that fixed with patch 4.
Patch 5 fixes problems found while investigating patch 4 and 6.

James Carter (6):
  libsepol/cil: Handle disabled optional blocks in earlier passes
  libsepol/cil: Destroy the permission nodes when exiting with an error
  libsepol/cil: Limit the number of open parenthesis allowed
  libsepol/cil: Resolve anonymous class permission sets only once
  libsepol/cil: Pointers to datums should be set to NULL when resetting
  libsepol/cil: Resolve anonymous levels only once

 libsepol/cil/src/cil_build_ast.c   |  2 +
 libsepol/cil/src/cil_parser.c      |  7 +++-
 libsepol/cil/src/cil_reset_ast.c   | 56 +++++++++++++++++++++++++--
 libsepol/cil/src/cil_resolve_ast.c | 62 +++++++++++++++++-------------
 4 files changed, 97 insertions(+), 30 deletions(-)

Comments

James Carter June 3, 2021, 5:05 p.m. UTC | #1 
On Thu, May 13, 2021 at 2:53 PM James Carter <jwcart2@gmail.com> wrote:
>
> Patches 1-4 fix bug found directly by the secilc-fuzzer.
> Patch 6 is the same class of bug as that fixed with patch 4.
> Patch 5 fixes problems found while investigating patch 4 and 6.
>
> James Carter (6):
>   libsepol/cil: Handle disabled optional blocks in earlier passes
>   libsepol/cil: Destroy the permission nodes when exiting with an error
>   libsepol/cil: Limit the number of open parenthesis allowed
>   libsepol/cil: Resolve anonymous class permission sets only once
>   libsepol/cil: Pointers to datums should be set to NULL when resetting
>   libsepol/cil: Resolve anonymous levels only once
>
>  libsepol/cil/src/cil_build_ast.c   |  2 +
>  libsepol/cil/src/cil_parser.c      |  7 +++-
>  libsepol/cil/src/cil_reset_ast.c   | 56 +++++++++++++++++++++++++--
>  libsepol/cil/src/cil_resolve_ast.c | 62 +++++++++++++++++-------------
>  4 files changed, 97 insertions(+), 30 deletions(-)
>
> --
> 2.26.3
>

There haven't been any comments on this series, and it has been three
weeks, so I plan on merging it tomorrow.
Jim
James Carter June 4, 2021, 4:58 p.m. UTC | #2 
On Thu, Jun 3, 2021 at 1:05 PM James Carter <jwcart2@gmail.com> wrote:
>
> On Thu, May 13, 2021 at 2:53 PM James Carter <jwcart2@gmail.com> wrote:
> >
> > Patches 1-4 fix bug found directly by the secilc-fuzzer.
> > Patch 6 is the same class of bug as that fixed with patch 4.
> > Patch 5 fixes problems found while investigating patch 4 and 6.
> >
> > James Carter (6):
> >   libsepol/cil: Handle disabled optional blocks in earlier passes
> >   libsepol/cil: Destroy the permission nodes when exiting with an error
> >   libsepol/cil: Limit the number of open parenthesis allowed
> >   libsepol/cil: Resolve anonymous class permission sets only once
> >   libsepol/cil: Pointers to datums should be set to NULL when resetting
> >   libsepol/cil: Resolve anonymous levels only once
> >
> >  libsepol/cil/src/cil_build_ast.c   |  2 +
> >  libsepol/cil/src/cil_parser.c      |  7 +++-
> >  libsepol/cil/src/cil_reset_ast.c   | 56 +++++++++++++++++++++++++--
> >  libsepol/cil/src/cil_resolve_ast.c | 62 +++++++++++++++++-------------
> >  4 files changed, 97 insertions(+), 30 deletions(-)
> >
> > --
> > 2.26.3
> >
>
> There haven't been any comments on this series, and it has been three
> weeks, so I plan on merging it tomorrow.
> Jim

This series has been merged.
Jim