Author: juhosg
Date: 2016-03-21 14:15:29 +0100 (Mon, 21 Mar 2016)
New Revision: 49062

Modified:
   trunk/target/linux/ramips/image/lzma-loader/src/head.S
Log:
ramips: lzma-loader: fix O32 ABI conformance

According to the calling convention of the o32 ABI the
caller function must reserve stack space for $a0-$a3
registers in case the callee needs to save its arguments.

The assembly startup code does not reserve stack space
for these registers thus when the main C function needs
to save its arguments, that will cause a stack overflow.

Fix the assembly code to reserve stack space for the
registers to avoid that.

Untested. It seems that the lzma-loader is not used at all?

Signed-off-by: Gabor Juhos <[email protected]>

Modified: trunk/target/linux/ramips/image/lzma-loader/src/head.S
===================================================================
--- trunk/target/linux/ramips/image/lzma-loader/src/head.S      2016-03-21 
13:15:25 UTC (rev 49061)
+++ trunk/target/linux/ramips/image/lzma-loader/src/head.S      2016-03-21 
13:15:29 UTC (rev 49062)
@@ -109,6 +109,9 @@
        /* Setup new "C" stack */
        la      sp, _stack
 
+       /* reserve stack space for a0-a3 registers */
+       subu    sp, 16
+
        /* jump to the decompressor routine */
        la      t0, loader_main
        jr      t0
_______________________________________________
openwrt-commits mailing list
[email protected]
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-commits

Reply via email to