On 10/4/23 08:00, Philippe Mathieu-Daudé wrote:
> Fix:
>
> softmmu/tpm.c:178:59: error: declaration shadows a variable in the global scope [-Werror,-Wshadow]
> int tpm_config_parse(QemuOptsList *opts_list, const char *optarg)
> ^
> /Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/getopt.h:77:14: note: previous declaration is here
> extern char *optarg; /* getopt(3) external variables */
> ^
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Stefan Berger <stefanb@linux.ibm.com>
> ---
> include/sysemu/tpm.h | 2 +-
> softmmu/tpm.c | 6 +++---
> 2 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/include/sysemu/tpm.h b/include/sysemu/tpm.h
> index 66e3b45f30..1ee568b3b6 100644
> --- a/include/sysemu/tpm.h
> +++ b/include/sysemu/tpm.h
> @@ -17,7 +17,7 @@
>
> #ifdef CONFIG_TPM
>
> -int tpm_config_parse(QemuOptsList *opts_list, const char *optarg);
> +int tpm_config_parse(QemuOptsList *opts_list, const char *optstr);
> int tpm_init(void);
> void tpm_cleanup(void);
>
> diff --git a/softmmu/tpm.c b/softmmu/tpm.c
> index 578563f05a..7164ea7ff1 100644
> --- a/softmmu/tpm.c
> +++ b/softmmu/tpm.c
> @@ -175,15 +175,15 @@ int tpm_init(void)
> * Parse the TPM configuration options.
> * To display all available TPM backends the user may use '-tpmdev help'
> */
> -int tpm_config_parse(QemuOptsList *opts_list, const char *optarg)
> +int tpm_config_parse(QemuOptsList *opts_list, const char *optstr)
> {
> QemuOpts *opts;
>
> - if (!strcmp(optarg, "help")) {
> + if (!strcmp(optstr, "help")) {
> tpm_display_backend_drivers();
> return -1;
> }
> - opts = qemu_opts_parse_noisily(opts_list, optarg, true);
> + opts = qemu_opts_parse_noisily(opts_list, optstr, true);
> if (!opts) {
> return -1;
> }