[PATCH 3/7] Flask: replace uses of __u32

Jan Beulich posted 7 patches 2 months, 2 weeks ago
[PATCH 3/7] Flask: replace uses of __u32
Posted by Jan Beulich 2 months, 2 weeks ago
... by uint32_t.

Signed-off-by: Jan Beulich <jbeulich@suse.com>
---
Uses of bool_val look suspicious; I looked at them because by its name
I would have hoped I can switch it to bool instead.

--- a/xen/xsm/flask/ss/conditional.h
+++ b/xen/xsm/flask/ss/conditional.h
@@ -29,8 +29,8 @@ struct cond_expr {
 #define COND_EQ        6 /* bool == bool */
 #define COND_NEQ    7 /* bool != bool */
 #define COND_LAST    COND_NEQ
-    __u32 expr_type;
-    __u32 bool_val;
+    uint32_t expr_type;
+    uint32_t bool_val;
     struct cond_expr *next;
 };
 
--- a/xen/xsm/flask/ss/policydb.h
+++ b/xen/xsm/flask/ss/policydb.h
@@ -131,7 +131,7 @@ struct range_trans {
 
 /* Boolean data type */
 struct cond_bool_datum {
-    __u32 value;        /* internal type value */
+    uint32_t value;        /* internal type value */
     int state;
 };
Re: [PATCH 3/7] Flask: replace uses of __u32
Posted by Daniel P. Smith 2 months, 1 week ago
On 10/9/24 05:23, Jan Beulich wrote:
> ... by uint32_t.
> 
> Signed-off-by: Jan Beulich <jbeulich@suse.com>
> ---
> Uses of bool_val look suspicious; I looked at them because by its name
> I would have hoped I can switch it to bool instead.
> 
> --- a/xen/xsm/flask/ss/conditional.h
> +++ b/xen/xsm/flask/ss/conditional.h
> @@ -29,8 +29,8 @@ struct cond_expr {
>   #define COND_EQ        6 /* bool == bool */
>   #define COND_NEQ    7 /* bool != bool */
>   #define COND_LAST    COND_NEQ
> -    __u32 expr_type;
> -    __u32 bool_val;
> +    uint32_t expr_type;
> +    uint32_t bool_val;
>       struct cond_expr *next;
>   };
>   
> --- a/xen/xsm/flask/ss/policydb.h
> +++ b/xen/xsm/flask/ss/policydb.h
> @@ -131,7 +131,7 @@ struct range_trans {
>   
>   /* Boolean data type */
>   struct cond_bool_datum {
> -    __u32 value;        /* internal type value */
> +    uint32_t value;        /* internal type value */
>       int state;
>   };
>   

Acked-by: Daniel P. Smith <dpsmith@apertussolutions.com>