Author: gonzo
Date: Tue Jan 10 20:08:21 2017
New Revision: 311888
URL: https://svnweb.freebsd.org/changeset/base/311888

Log:
  [efi] Fix off-by-one error in ARM .bss zeroing code in loader's _start
  
  __bss_end should not be included in .bss zeroing code. Otherwise first 4
  bytes of the section that follows .bss (in loader's case it's .sdata) are
  overwritten by zero.
  
  Reviewed by:  andrew
  MFC after:    3 days
  Differential Revision:        https://reviews.freebsd.org/D9108

Modified:
  head/sys/boot/efi/loader/arch/arm/start.S

Modified: head/sys/boot/efi/loader/arch/arm/start.S
==============================================================================
--- head/sys/boot/efi/loader/arch/arm/start.S   Tue Jan 10 19:28:40 2017        
(r311887)
+++ head/sys/boot/efi/loader/arch/arm/start.S   Tue Jan 10 20:08:21 2017        
(r311888)
@@ -161,7 +161,7 @@ _start:
        mov     r2, #0
 
 1:     cmp     r0, r1
-       bgt     2f
+       bge     2f
        str     r2, [r0], #4
        b       1b
 2:
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to