Re: [Qemu-devel] [PATCH] fips: fix build on !Linux

[Anthony Liguori]

Anthony Liguori <address@hidden> writes:

> Commit 0f66998 makes -enable-fips conditional on Linux hosts but then uses it
> unconditionally in vl.c.
>
> Fix this by moving the fips handling to os-posix.c and adding a
> condition.

Applied.

Regards,

Anthony Liguori

>
> Cc: Paul Moore <address@hidden>
> Signed-off-by: Anthony Liguori <address@hidden>
> ---
>  os-posix.c |    5 +++++
>  vl.c       |    3 ---
>  2 files changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/os-posix.c b/os-posix.c
> index daf3d6f..79fa228 100644
> --- a/os-posix.c
> +++ b/os-posix.c
> @@ -188,6 +188,11 @@ void os_parse_cmd_args(int index, const char *optarg)
>      case QEMU_OPTION_daemonize:
>          daemonize = 1;
>          break;
> +#if defined(CONFIG_LINUX)
> +    case QEMU_OPTION_enablefips:
> +        fips_set_state(true);
> +        break;
> +#endif
>      }
>      return;
>  }
> diff --git a/vl.c b/vl.c
> index 8cda85f..6d2ce45 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -3199,9 +3199,6 @@ int main(int argc, char **argv, char **envp)
>              case QEMU_OPTION_qtest_log:
>                  qtest_log = optarg;
>                  break;
> -            case QEMU_OPTION_enablefips:
> -                fips_set_state(true);
> -                break;
>              default:
>                  os_parse_cmd_args(popt->index, optarg);
>              }
> -- 
> 1.7.5.4