It is now obsolete with commit 621bb723a4e00cb93e8a94c6126de4976dde1d9e

On Mon, Feb 15, 2016 at 10:25 AM, Gao, Liming <[email protected]> wrote:
> Reviewed-by: Liming Gao <[email protected]>
>
> -----Original Message-----
> From: Ronny Chevalier [mailto:[email protected]]
> Sent: Friday, February 12, 2016 10:26 PM
> To: [email protected]
> Cc: Zhu, Yonghong; Gao, Liming
> Subject: [PATCH] BaseTools/GenFw: skip relocation entry if there is an error
>
> Otherwise a segfault occurs due to an out-of-bound access
>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Cc: Yonghong Zhu <[email protected]>
> Cc: Liming Gao <[email protected]>
> Signed-off-by: Ronny Chevalier <[email protected]>
> ---
>  BaseTools/Source/C/GenFw/Elf32Convert.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/BaseTools/Source/C/GenFw/Elf32Convert.c 
> b/BaseTools/Source/C/GenFw/Elf32Convert.c
> index 9bf5855..cdb3246 100644
> --- a/BaseTools/Source/C/GenFw/Elf32Convert.c
> +++ b/BaseTools/Source/C/GenFw/Elf32Convert.c
> @@ -670,6 +670,7 @@ WriteSections32 (
>              || Sym->st_shndx == SHN_ABS
>              || Sym->st_shndx > mEhdr->e_shnum) {
>            Error (NULL, 0, 3000, "Invalid", "%s bad symbol definition.", 
> mInImageName);
> +          continue;
>          }
>          SymShdr = GetShdrByIndex(Sym->st_shndx);
>
> --
> 2.7.1
>
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to