diff mbox series

cil_conditional_statements.md: fix expr definition

Message ID 20210324151105.3765449-1-dominick.grift@defensec.nl (mailing list archive)
State Accepted
Headers show
Series cil_conditional_statements.md: fix expr definition | expand

Commit Message

Dominick Grift March 24, 2021, 3:11 p.m. UTC
expr "(expr (tunable_id tunable_id))" does not work but "(expr
tunable_id tunable_id)" does work

for example, this works

(tunable test1)
(tunable test2)
(tunableif (or test1 test2)
	   (true
	    (allow a b (c (d)))))

but this does not work:

(tunable test1)
(tunable test2)
(tunableif (or (test1 test2))
	   (true
	    (allow a b (c (d)))))

Signed-off-by: Dominick Grift <dominick.grift@defensec.nl>
---
 secilc/docs/cil_conditional_statements.md | 24 +++++++++++------------
 1 file changed, 12 insertions(+), 12 deletions(-)

Comments

James Carter March 29, 2021, 3:13 p.m. UTC | #1
On Wed, Mar 24, 2021 at 11:13 AM Dominick Grift
<dominick.grift@defensec.nl> wrote:
>
> expr "(expr (tunable_id tunable_id))" does not work but "(expr
> tunable_id tunable_id)" does work
>
> for example, this works
>
> (tunable test1)
> (tunable test2)
> (tunableif (or test1 test2)
>            (true
>             (allow a b (c (d)))))
>
> but this does not work:
>
> (tunable test1)
> (tunable test2)
> (tunableif (or (test1 test2))
>            (true
>             (allow a b (c (d)))))
>
> Signed-off-by: Dominick Grift <dominick.grift@defensec.nl>

Acked-by: James Carter <jwcart2@gmail.com>

> ---
>  secilc/docs/cil_conditional_statements.md | 24 +++++++++++------------
>  1 file changed, 12 insertions(+), 12 deletions(-)
>
> diff --git a/secilc/docs/cil_conditional_statements.md b/secilc/docs/cil_conditional_statements.md
> index a55a9b6c..55f0bfd6 100644
> --- a/secilc/docs/cil_conditional_statements.md
> +++ b/secilc/docs/cil_conditional_statements.md
> @@ -78,12 +78,12 @@ Contains the run time conditional statements that are instantiated in the binary
>  <tr class="odd">
>  <td align="left"><p><code>expr</code></p></td>
>  <td align="left"><p>Zero or more <code>expr</code>'s, the valid operators and syntax are:</p>
> -<p><code>    (and (boolean_id boolean_id))</code></p>
> -<p><code>    (or  (boolean_id boolean_id))</code></p>
> -<p><code>    (xor (boolean_id boolean_id))</code></p>
> -<p><code>    (eq  (boolean_id boolean_id))</code></p>
> -<p><code>    (neq (boolean_id boolean_id))</code></p>
> -<p><code>    (not (boolean_id))</code></p></td>
> +<p><code>    (and boolean_id boolean_id)</code></p>
> +<p><code>    (or  boolean_id boolean_id)</code></p>
> +<p><code>    (xor boolean_id boolean_id)</code></p>
> +<p><code>    (eq  boolean_id boolean_id)</code></p>
> +<p><code>    (neq boolean_id boolean_id)</code></p>
> +<p><code>    (not boolean_id)</code></p></td>
>  </tr>
>  <tr class="even">
>  <td align="left"><p><code>true</code></p></td>
> @@ -196,12 +196,12 @@ Compile time conditional statement that may or may not add CIL statements to be
>  <tr class="odd">
>  <td align="left"><p><code>expr</code></p></td>
>  <td align="left"><p>Zero or more <code>expr</code>'s, the valid operators and syntax are:</p>
> -<p><code>    (and (tunable_id tunable_id))</code></p>
> -<p><code>    (or  (tunable_id tunable_id))</code></p>
> -<p><code>    (xor (tunable_id tunable_id))</code></p>
> -<p><code>    (eq  (tunable_id tunable_id))</code></p>
> -<p><code>    (neq (tunable_id tunable_id))</code></p>
> -<p><code>    (not (tunable_id))</code></p></td>
> +<p><code>    (and tunable_id tunable_id)</code></p>
> +<p><code>    (or  tunable_id tunable_id)</code></p>
> +<p><code>    (xor tunable_id tunable_id)</code></p>
> +<p><code>    (eq  tunable_id tunable_id)</code></p>
> +<p><code>    (neq tunable_id tunable_id)</code></p>
> +<p><code>    (not tunable_id)</code></p></td>
>  </tr>
>  <tr class="even">
>  <td align="left"><p><code>true</code></p></td>
> --
> 2.30.1
>
James Carter April 8, 2021, 5:36 p.m. UTC | #2
On Mon, Mar 29, 2021 at 11:13 AM James Carter <jwcart2@gmail.com> wrote:
>
> On Wed, Mar 24, 2021 at 11:13 AM Dominick Grift
> <dominick.grift@defensec.nl> wrote:
> >
> > expr "(expr (tunable_id tunable_id))" does not work but "(expr
> > tunable_id tunable_id)" does work
> >
> > for example, this works
> >
> > (tunable test1)
> > (tunable test2)
> > (tunableif (or test1 test2)
> >            (true
> >             (allow a b (c (d)))))
> >
> > but this does not work:
> >
> > (tunable test1)
> > (tunable test2)
> > (tunableif (or (test1 test2))
> >            (true
> >             (allow a b (c (d)))))
> >
> > Signed-off-by: Dominick Grift <dominick.grift@defensec.nl>
>
> Acked-by: James Carter <jwcart2@gmail.com>
>

Applied. Thanks,
Jim

> > ---
> >  secilc/docs/cil_conditional_statements.md | 24 +++++++++++------------
> >  1 file changed, 12 insertions(+), 12 deletions(-)
> >
> > diff --git a/secilc/docs/cil_conditional_statements.md b/secilc/docs/cil_conditional_statements.md
> > index a55a9b6c..55f0bfd6 100644
> > --- a/secilc/docs/cil_conditional_statements.md
> > +++ b/secilc/docs/cil_conditional_statements.md
> > @@ -78,12 +78,12 @@ Contains the run time conditional statements that are instantiated in the binary
> >  <tr class="odd">
> >  <td align="left"><p><code>expr</code></p></td>
> >  <td align="left"><p>Zero or more <code>expr</code>'s, the valid operators and syntax are:</p>
> > -<p><code>    (and (boolean_id boolean_id))</code></p>
> > -<p><code>    (or  (boolean_id boolean_id))</code></p>
> > -<p><code>    (xor (boolean_id boolean_id))</code></p>
> > -<p><code>    (eq  (boolean_id boolean_id))</code></p>
> > -<p><code>    (neq (boolean_id boolean_id))</code></p>
> > -<p><code>    (not (boolean_id))</code></p></td>
> > +<p><code>    (and boolean_id boolean_id)</code></p>
> > +<p><code>    (or  boolean_id boolean_id)</code></p>
> > +<p><code>    (xor boolean_id boolean_id)</code></p>
> > +<p><code>    (eq  boolean_id boolean_id)</code></p>
> > +<p><code>    (neq boolean_id boolean_id)</code></p>
> > +<p><code>    (not boolean_id)</code></p></td>
> >  </tr>
> >  <tr class="even">
> >  <td align="left"><p><code>true</code></p></td>
> > @@ -196,12 +196,12 @@ Compile time conditional statement that may or may not add CIL statements to be
> >  <tr class="odd">
> >  <td align="left"><p><code>expr</code></p></td>
> >  <td align="left"><p>Zero or more <code>expr</code>'s, the valid operators and syntax are:</p>
> > -<p><code>    (and (tunable_id tunable_id))</code></p>
> > -<p><code>    (or  (tunable_id tunable_id))</code></p>
> > -<p><code>    (xor (tunable_id tunable_id))</code></p>
> > -<p><code>    (eq  (tunable_id tunable_id))</code></p>
> > -<p><code>    (neq (tunable_id tunable_id))</code></p>
> > -<p><code>    (not (tunable_id))</code></p></td>
> > +<p><code>    (and tunable_id tunable_id)</code></p>
> > +<p><code>    (or  tunable_id tunable_id)</code></p>
> > +<p><code>    (xor tunable_id tunable_id)</code></p>
> > +<p><code>    (eq  tunable_id tunable_id)</code></p>
> > +<p><code>    (neq tunable_id tunable_id)</code></p>
> > +<p><code>    (not tunable_id)</code></p></td>
> >  </tr>
> >  <tr class="even">
> >  <td align="left"><p><code>true</code></p></td>
> > --
> > 2.30.1
> >
diff mbox series

Patch

diff --git a/secilc/docs/cil_conditional_statements.md b/secilc/docs/cil_conditional_statements.md
index a55a9b6c..55f0bfd6 100644
--- a/secilc/docs/cil_conditional_statements.md
+++ b/secilc/docs/cil_conditional_statements.md
@@ -78,12 +78,12 @@  Contains the run time conditional statements that are instantiated in the binary
 <tr class="odd">
 <td align="left"><p><code>expr</code></p></td>
 <td align="left"><p>Zero or more <code>expr</code>'s, the valid operators and syntax are:</p>
-<p><code>    (and (boolean_id boolean_id))</code></p>
-<p><code>    (or  (boolean_id boolean_id))</code></p>
-<p><code>    (xor (boolean_id boolean_id))</code></p>
-<p><code>    (eq  (boolean_id boolean_id))</code></p>
-<p><code>    (neq (boolean_id boolean_id))</code></p>
-<p><code>    (not (boolean_id))</code></p></td>
+<p><code>    (and boolean_id boolean_id)</code></p>
+<p><code>    (or  boolean_id boolean_id)</code></p>
+<p><code>    (xor boolean_id boolean_id)</code></p>
+<p><code>    (eq  boolean_id boolean_id)</code></p>
+<p><code>    (neq boolean_id boolean_id)</code></p>
+<p><code>    (not boolean_id)</code></p></td>
 </tr>
 <tr class="even">
 <td align="left"><p><code>true</code></p></td>
@@ -196,12 +196,12 @@  Compile time conditional statement that may or may not add CIL statements to be
 <tr class="odd">
 <td align="left"><p><code>expr</code></p></td>
 <td align="left"><p>Zero or more <code>expr</code>'s, the valid operators and syntax are:</p>
-<p><code>    (and (tunable_id tunable_id))</code></p>
-<p><code>    (or  (tunable_id tunable_id))</code></p>
-<p><code>    (xor (tunable_id tunable_id))</code></p>
-<p><code>    (eq  (tunable_id tunable_id))</code></p>
-<p><code>    (neq (tunable_id tunable_id))</code></p>
-<p><code>    (not (tunable_id))</code></p></td>
+<p><code>    (and tunable_id tunable_id)</code></p>
+<p><code>    (or  tunable_id tunable_id)</code></p>
+<p><code>    (xor tunable_id tunable_id)</code></p>
+<p><code>    (eq  tunable_id tunable_id)</code></p>
+<p><code>    (neq tunable_id tunable_id)</code></p>
+<p><code>    (not tunable_id)</code></p></td>
 </tr>
 <tr class="even">
 <td align="left"><p><code>true</code></p></td>