On Wed, Feb 3, 2021 at 4:00 AM Nicolas Iooss <nicolas.iooss@xxxxxxx> wrote:
>
> It is good practise in C to include the header file that specifies the
> prototype of functions which are defined in the source file. Otherwise,
> the function prototypes which be different, which could cause unexpected
> issues.
>
> Add the include directives to do this.
>
> Signed-off-by: Nicolas Iooss <nicolas.iooss@xxxxxxx>
Acked-by: James Carter <jwcart2@xxxxxxxxx>
> ---
> libsepol/cil/src/cil_find.c | 1 +
> libsepol/cil/src/cil_fqn.c | 1 +
> libsepol/cil/src/cil_mem.c | 1 +
> libsepol/cil/src/cil_parser.c | 1 +
> libsepol/cil/src/cil_policy.c | 1 +
> libsepol/cil/src/cil_reset_ast.c | 1 +
> libsepol/src/kernel_to_cil.c | 1 +
> libsepol/src/kernel_to_conf.c | 1 +
> libsepol/src/services.c | 1 +
> 9 files changed, 9 insertions(+)
>
> diff --git a/libsepol/cil/src/cil_find.c b/libsepol/cil/src/cil_find.c
> index 638b675db826..3898725f18d5 100644
> --- a/libsepol/cil/src/cil_find.c
> +++ b/libsepol/cil/src/cil_find.c
> @@ -30,6 +30,7 @@
> #include <sepol/policydb/ebitmap.h>
>
> #include "cil_internal.h"
> +#include "cil_find.h"
> #include "cil_flavor.h"
> #include "cil_list.h"
> #include "cil_log.h"
> diff --git a/libsepol/cil/src/cil_fqn.c b/libsepol/cil/src/cil_fqn.c
> index 2e76f8737754..097222a83da9 100644
> --- a/libsepol/cil/src/cil_fqn.c
> +++ b/libsepol/cil/src/cil_fqn.c
> @@ -31,6 +31,7 @@
> #include <stdio.h>
> #include <string.h>
>
> +#include "cil_fqn.h"
> #include "cil_internal.h"
> #include "cil_log.h"
> #include "cil_strpool.h"
> diff --git a/libsepol/cil/src/cil_mem.c b/libsepol/cil/src/cil_mem.c
> index f73021b58d50..8e4a1d246f2c 100644
> --- a/libsepol/cil/src/cil_mem.c
> +++ b/libsepol/cil/src/cil_mem.c
> @@ -33,6 +33,7 @@
> #include <string.h>
>
> #include "cil_log.h"
> +#include "cil_mem.h"
>
> void *cil_malloc(size_t size)
> {
> diff --git a/libsepol/cil/src/cil_parser.c b/libsepol/cil/src/cil_parser.c
> index b62043b95806..0038eed6dd1b 100644
> --- a/libsepol/cil/src/cil_parser.c
> +++ b/libsepol/cil/src/cil_parser.c
> @@ -38,6 +38,7 @@
> #include "cil_mem.h"
> #include "cil_tree.h"
> #include "cil_lexer.h"
> +#include "cil_parser.h"
> #include "cil_strpool.h"
> #include "cil_stack.h"
>
> diff --git a/libsepol/cil/src/cil_policy.c b/libsepol/cil/src/cil_policy.c
> index 06d7d74e54c3..74edb34575ea 100644
> --- a/libsepol/cil/src/cil_policy.c
> +++ b/libsepol/cil/src/cil_policy.c
> @@ -41,6 +41,7 @@
> #include "cil_flavor.h"
> #include "cil_find.h"
> #include "cil_mem.h"
> +#include "cil_policy.h"
> #include "cil_tree.h"
> #include "cil_list.h"
> #include "cil_symtab.h"
> diff --git a/libsepol/cil/src/cil_reset_ast.c b/libsepol/cil/src/cil_reset_ast.c
> index 52e5f64011d2..3da1b9a64167 100644
> --- a/libsepol/cil/src/cil_reset_ast.c
> +++ b/libsepol/cil/src/cil_reset_ast.c
> @@ -2,6 +2,7 @@
> #include "cil_internal.h"
> #include "cil_log.h"
> #include "cil_list.h"
> +#include "cil_reset_ast.h"
> #include "cil_symtab.h"
>
> static inline void cil_reset_classperms_list(struct cil_list *cp_list);
> diff --git a/libsepol/src/kernel_to_cil.c b/libsepol/src/kernel_to_cil.c
> index c247b32f9e75..a146ac514018 100644
> --- a/libsepol/src/kernel_to_cil.c
> +++ b/libsepol/src/kernel_to_cil.c
> @@ -16,6 +16,7 @@
> #define IPPROTO_SCTP 132
> #endif
>
> +#include <sepol/kernel_to_cil.h>
> #include <sepol/policydb/avtab.h>
> #include <sepol/policydb/conditional.h>
> #include <sepol/policydb/hashtab.h>
> diff --git a/libsepol/src/kernel_to_conf.c b/libsepol/src/kernel_to_conf.c
> index 62bf706c1aa0..a22f196df9e9 100644
> --- a/libsepol/src/kernel_to_conf.c
> +++ b/libsepol/src/kernel_to_conf.c
> @@ -15,6 +15,7 @@
> #define IPPROTO_SCTP 132
> #endif
>
> +#include <sepol/kernel_to_conf.h>
> #include <sepol/policydb/avtab.h>
> #include <sepol/policydb/conditional.h>
> #include <sepol/policydb/hashtab.h>
> diff --git a/libsepol/src/services.c b/libsepol/src/services.c
> index 72b39657cd2e..6596431c38e2 100644
> --- a/libsepol/src/services.c
> +++ b/libsepol/src/services.c
> @@ -59,6 +59,7 @@
> #include <sepol/policydb/services.h>
> #include <sepol/policydb/conditional.h>
> #include <sepol/policydb/util.h>
> +#include <sepol/sepol.h>
>
> #include "debug.h"
> #include "private.h"
> --
> 2.30.0
>
