Re: [edk2] [PATCH 1/2] BaseTools/PeCoffLoader: fix handling of ARM MOVW/MOVT instruction relocs

2015-10-06 Thread Leif Lindholm
On Tue, Sep 29, 2015 at 10:28:59AM +0200, Ard Biesheuvel wrote: > The handling of ARM MOVW/MOVT relocations sets the FixupData twice (once > incorrectly), but fails to advance the *FixupData pointer afterwards. > This is not actually a problem, since the fixup data is never used but > let's fix it

[edk2] [PATCH 1/2] BaseTools/PeCoffLoader: fix handling of ARM MOVW/MOVT instruction relocs

2015-09-29 Thread Ard Biesheuvel
The handling of ARM MOVW/MOVT relocations sets the FixupData twice (once incorrectly), but fails to advance the *FixupData pointer afterwards. This is not actually a problem, since the fixup data is never used but let's fix it anyway in case anyone reuses this code. Contributed-under: TianoCore