Skip to content

Commit

Permalink
sysemu/tpm: Clean up global variable shadowing
Browse files Browse the repository at this point in the history
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>
Message-ID: <20231004120019.93101-16-philmd@linaro.org>
Reviewed-by: Stefan Berger <stefanb@linux.ibm.com>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
  • Loading branch information
philmd authored and Markus Armbruster committed Oct 6, 2023
1 parent 48176a1 commit fcd9a35
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion include/sysemu/tpm.h
Original file line number Diff line number Diff line change
Expand Up @@ -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);

Expand Down
6 changes: 3 additions & 3 deletions softmmu/tpm.c
Original file line number Diff line number Diff line change
Expand Up @@ -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;
}
Expand Down

0 comments on commit fcd9a35

Please sign in to comment.