On Mon, Apr 8, 2024 at 11:08 AM Christian Göttsche
<cgoettsche@xxxxxxxxxxxxx> wrote:
>
> From: Christian Göttsche <cgzones@xxxxxxxxxxxxxx>
>
> If a policy version cannot be found include the policy target, and a
> module prefix for non kernel policies in the message.
>
> Signed-off-by: Christian Göttsche <cgzones@xxxxxxxxxxxxxx>
For these four patches:
Acked-by: James Carter <jwcart2@xxxxxxxxx>
> ---
> libsepol/src/write.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/libsepol/src/write.c b/libsepol/src/write.c
> index 283d11c8..2fcc1701 100644
> --- a/libsepol/src/write.c
> +++ b/libsepol/src/write.c
> @@ -2252,8 +2252,10 @@ int policydb_write(policydb_t * p, struct policy_file *fp)
> info = policydb_lookup_compat(p->policyvers, p->policy_type,
> p->target_platform);
> if (!info) {
> - ERR(fp->handle, "compatibility lookup failed for policy "
> - "version %d", p->policyvers);
> + ERR(fp->handle, "compatibility lookup failed for %s%s policy version %d",
> + p->target_platform == SEPOL_TARGET_SELINUX ? "selinux" : "xen",
> + p->policy_type == POLICY_KERN ? "" : " module",
> + p->policyvers);
> return POLICYDB_ERROR;
> }
>
> --
> 2.43.0
>
>
