diff options
| author | Michael Brown <mcb30@ipxe.org> | 2014-03-14 15:15:14 +0000 |
|---|---|---|
| committer | Michael Brown <mcb30@ipxe.org> | 2014-03-14 16:20:55 +0000 |
| commit | e662912c53825ed9ab0cd02c41b2ab460716c3c7 (patch) | |
| tree | c8366704d57ac2d920310762d337d35ebc10d9cc /src/arch/x86/prefix | |
| parent | 87465258abbd1056c8be4e934385d6c7197dc655 (diff) | |
| download | ipxe-e662912c53825ed9ab0cd02c41b2ab460716c3c7.tar.gz | |
[efi] Avoid accidentally calling main() twice
EFIRC() uses PLATFORM_TO_ERRNO(), which evaluates its argument twice
(and can't trivially use a braced-group expression or an inline
function to avoid this, since it gets used outside of function
context).
The expression "EFIRC(main())" will therefore end up calling main()
twice, which is not the intended behaviour. Every other instance of
EFIRC() is of the simple form "EFIRC(rc)", so fix by converting this
instance to match.
Signed-off-by: Michael Brown <mcb30@ipxe.org>
Diffstat (limited to 'src/arch/x86/prefix')
| -rw-r--r-- | src/arch/x86/prefix/efiprefix.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/arch/x86/prefix/efiprefix.c b/src/arch/x86/prefix/efiprefix.c index a847045a5..475ea505c 100644 --- a/src/arch/x86/prefix/efiprefix.c +++ b/src/arch/x86/prefix/efiprefix.c @@ -33,11 +33,15 @@ FILE_LICENCE ( GPL2_OR_LATER ); EFI_STATUS EFIAPI _efi_start ( EFI_HANDLE image_handle, EFI_SYSTEM_TABLE *systab ) { EFI_STATUS efirc; + int rc; /* Initialise EFI environment */ if ( ( efirc = efi_init ( image_handle, systab ) ) != 0 ) return efirc; /* Call to main() */ - return EFIRC ( main () ); + if ( ( rc = main() ) != 0 ) + return EFIRC ( rc ); + + return 0; } |