Browse Source

ucode: add temporary fix for integer formatting on 32bit systems

The ucode VM always passes 64bit integer values to sprintf implementation
while the `%d` format expects 32bit integers on 32bit platforms, leading
to incorrect formatting results.

Temporarily solve the issue by casting the numeric argument to int until
a more thorough fix arrives with the next update.

Fixes: FS#4234
Signed-off-by: Jo-Philipp Wich <[email protected]>
Jo-Philipp Wich 4 years ago
parent
commit
3d3d03479d

+ 1 - 1
package/utils/ucode/Makefile

@@ -8,7 +8,7 @@
 include $(TOPDIR)/rules.mk
 
 PKG_NAME:=ucode
-PKG_RELEASE:=1
+PKG_RELEASE:=2
 
 PKG_SOURCE_PROTO:=git
 PKG_SOURCE_URL=https://github.com/jow-/ucode.git

+ 11 - 0
package/utils/ucode/patches/100-fix-int-format-on-32bit-system.patch

@@ -0,0 +1,11 @@
+--- a/lib.c
++++ b/lib.c
+@@ -1438,7 +1438,7 @@ uc_printf_common(uc_vm_t *vm, size_t nar
+ 
+ 			switch (t) {
+ 			case UC_INTEGER:
+-				ucv_stringbuf_printf(buf, sfmt, arg.n);
++				ucv_stringbuf_printf(buf, sfmt, (int)arg.n);
+ 				break;
+ 
+ 			case UC_DOUBLE: