tyler.durden
/
audk
mirror of https://github.com/acidanthera/audk.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

NetworkPkg/HttpBootDxe: Fix for the issue that the HTTP boot option can't be booted more than once 

This issue can be reproduced by the following steps:
1. Boot to HTTP boot option and the boot file is a ISO file like Ubuntu PE image.
2. Exit from boot option (GRUB) and then back to boot manager menu.
3. Boot to the same HTTP boot option again or a HTTP boot option pointing to the same HTTP ISO file. It will fail to boot.
Root cause:
When booting a HTTP boot option, the HTTP boot driver will save the Boot File's information in its private data as cache data for skipping the Boot file discovery from next time boot. However, the cache data doesn't include ImageType data, which would cause HTTP boot driver using the invalid ImageType (ImageTypeMax) and then fail to boot the cached boot file. In other words, for the second time boot, the HttpBootLoadFile() doesn't update ImageType (it returns a valid ImageType), which causes that the HttpBootDxeLoadFile() skips to Register a RAM Disk for downloaded HTTP ISO file and then BDS code can't find the RAM disk to boot.
Solution:
Save ImageType to private data for next time HTTP boot.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Sunny Wang <sunnywang@hpe.com>
Reviewed-by: Fu Siyuan <siyuan.fu@intel.com>
This commit is contained in:
Sunny Wang 2016-05-05 10:05:00 +08:00 committed by Fu Siyuan
parent 9cc0168a70
commit 44a7d08b5a
2 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
NetworkPkg/HttpBootDxe/HttpBootDxe.h
View File

@ -189,6 +189,7 @@ struct _HTTP_BOOT_PRIVATE_DATA {
VOID *BootFileUriParser; VOID *BootFileUriParser;
UINTN BootFileSize; UINTN BootFileSize;
BOOLEAN NoGateway; BOOLEAN NoGateway;
HTTP_BOOT_IMAGE_TYPE ImageType;
// //
// URI string extracted from the input FilePath parameter. // URI string extracted from the input FilePath parameter.

6
NetworkPkg/HttpBootDxe/HttpBootImpl.c
View File

@ -2,6 +2,7 @@
The implementation of EFI_LOAD_FILE_PROTOCOL for UEFI HTTP boot. The implementation of EFI_LOAD_FILE_PROTOCOL for UEFI HTTP boot.
Copyright (c) 2015 - 2016, Intel Corporation. All rights reserved.<BR> Copyright (c) 2015 - 2016, Intel Corporation. All rights reserved.<BR>
(C) Copyright 2016 Hewlett Packard Enterprise Development LP<BR>
This program and the accompanying materials are licensed and made available under This program and the accompanying materials are licensed and made available under
the terms and conditions of the BSD License that accompanies this distribution. the terms and conditions of the BSD License that accompanies this distribution.
The full text of the license may be found at The full text of the license may be found at
@ -271,7 +272,7 @@ HttpBootLoadFile (
TRUE, TRUE,
&Private->BootFileSize, &Private->BootFileSize,
NULL, NULL,
ImageType &Private->ImageType
); );
if (EFI_ERROR (Status) && Status != EFI_BUFFER_TOO_SMALL) { if (EFI_ERROR (Status) && Status != EFI_BUFFER_TOO_SMALL) {
// //
@ -283,7 +284,7 @@ HttpBootLoadFile (
FALSE, FALSE,
&Private->BootFileSize, &Private->BootFileSize,
NULL, NULL,
ImageType &Private->ImageType
); );
if (EFI_ERROR (Status) && Status != EFI_BUFFER_TOO_SMALL) { if (EFI_ERROR (Status) && Status != EFI_BUFFER_TOO_SMALL) {
return Status; return Status;
@ -293,6 +294,7 @@ HttpBootLoadFile (
if (*BufferSize < Private->BootFileSize) { if (*BufferSize < Private->BootFileSize) {
*BufferSize = Private->BootFileSize; *BufferSize = Private->BootFileSize;
*ImageType = Private->ImageType;
return EFI_BUFFER_TOO_SMALL; return EFI_BUFFER_TOO_SMALL;
} }