From 56ed37f59fa1dcb4fbcf18873ace39cb517c6e42 Mon Sep 17 00:00:00 2001
From: n4l5u0r <rouslan@blooo.io>
Date: Mon, 1 Nov 2021 19:26:24 +0000
Subject: [PATCH] fix: Change max length strnlen in parsing data

---
 src/handle_query_contract_ui.c |   2 +-
 tests/elfs/paraswap_nanos.elf  | Bin 231056 -> 231048 bytes
 tests/elfs/paraswap_nanox.elf  | Bin 430260 -> 430256 bytes
 3 files changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/handle_query_contract_ui.c b/src/handle_query_contract_ui.c
index afeb662..c485565 100644
--- a/src/handle_query_contract_ui.c
+++ b/src/handle_query_contract_ui.c
@@ -7,7 +7,7 @@ static void prepend_ticker(char *dest, size_t destsize, const char *ticker) {
         THROW(0x6503);
     }
     size_t ticker_len = strnlen(ticker, MAX_TICKER_LEN);
-    size_t dest_len = strnlen(dest, destsize - ticker_len);
+    size_t dest_len = strnlen(dest, destsize);
 
     if (dest_len + ticker_len >= destsize) {
         THROW(0x6503);
diff --git a/tests/elfs/paraswap_nanos.elf b/tests/elfs/paraswap_nanos.elf
index 66a916767c26aeb520325708fa2030e95a1e368b..724ace14efde26b28e9eb891c4488f6f201c7432 100755
GIT binary patch
delta 207
zcmbQx%h%D%H$j1MLZjkVMaJ)Z%o=W%+oky#7X~s)Zr_{0xQCf>$@Z!gMr}4m=k3d~
z7%ll2)wVyVU}R=toIL$W4Wl39&*{dsj3$f`(=%!r^%>W1pHa)GYse_S{n$FjZFY<z
z+iTx4K4oSVZ)O#0*`D)}agE*d&E`yk+cgZB)@<H><1tguLdI#+Q(Kvp8AYadwlb?T
pZkWCiM5#}|38F4cXKVvf4AV71l*9BuASH37iJ4(10|GEYnE(pBKSKZj

delta 194
zcmeC^<(mLR3XC%v6}Kufe&=Jhl+xHP&Cj?nkWp^?-UP-y%#16ySEVp&voU&XUzWva
z$;YU@{XqpIGYjL4=}&4H{TTmEH?CzgVT_%gQOl^$xMll{T1H(%MwRWy)-i6gW0c%p
z`<C%3GnZa7t2mn;s}Nhp_Vka8YwV_fFk}+hu3^BmV)OPhkC`ScWSl)cwv}0#QDS;y
oE3-P|mgy@&l=k$CAnL~SpCF23x?&rU@|f-kqV7yDY-1J&0E{z18UO$Q

diff --git a/tests/elfs/paraswap_nanox.elf b/tests/elfs/paraswap_nanox.elf
index 715811c159ab3f5ebb4747f7842a933b91e8855e..636525421fab9f8ae1910200db627fa50df2fa0a 100755
GIT binary patch
delta 436
zcmWm4Ur1AN90l;5@9*BLO(eC(V#%1vP_iVD{tP0eauPYB2vL0MA@M=jC`r&DV@jBc
zca8i>Ok_h83qd71uQhuxh69oCA*1MV4?P76Hfs0b@ZoUI`?PN@?Xv^8uwJ*|SB>FN
zeACZOKi+k~!)PBaTZnLZ0MiO9ycol41H;S>V)`T|*fRl-;2fiq*fQ{qS8TX}m-<Em
zk8q$rByba5JW3!{i4o2%peX3)YzD^-+~(2}))gw*{T7)vZ1AXnXUAZ2<PX9X7-aSz
zif&Z!qg!kXPk6UlhTM3}FQ=us40#@&k(?mJ&q2uxo-^4f`+_aTnxr5&#olI#n)tz7
zi?cWI%_TV$%rY5re%~3p>io{Lw^hOlL(E*4`wE|U(C+kdUc2S=dpg-E<MOwTzKB4O
zW9=R3+btVYE|<9SOncQs@OQl`!1Tt=nlb(gs<SB_uQEI@0$Ejx!9dn*+eWp1Fk{uD
wT#wIK4OrBx4m<j*!z*2M*waC~#HfyxFn^9(u9SvtozPRZRfVT|-nP8|0rc{dkN^Mx

delta 466
zcmXYtT}YF07{%Z7{I^$n(>_F7yseRnq$LCGQwWnj456?h&=_4RT@+g(s3@qYm4t0$
z<UdM~?ZUnwGNf}^sDv>lg>+Rxgdprf2!a$7+FyI%Jiq5~&eiie=}slx*ghPdEEx}|
zTDM|gCcv2hUY>)?a2t-<xXh<FFrx5^M<RIWK`+zo7%9R7UL8iUpqk<PnDgKVkH_Eu
zQGGg&+xV;B#&HHAuEjA{fIdDN$GYGqM^o7D!9`9?U{axwEiaI&2f3C-?-n@h{*8vM
z+qzaQYpK%On0r2*ain@m-Z|L8_m0GDbo0_q>2T1=k9#DThXt;bN?K6I*>af|++m_Z
zz6mmn9F(kJFGE2Id$Gv$A(L0}@ljb3JYk~N^nNmO!t`nwIw=hbJxraFD+(X@=dAGs
ztZOvBStriRkf4<A1%cob{Y}!gkT<L>ORRY&eX12)E>l@J!2z$&4tfji>-=7>4vgu@
zFI9BcKdo}|&_C_1&tR9n7PZTeulu9+emv994Zi5l2BUh_;Hxf)<uIrl4HosS9Cr8h
O+t!=>&HtE<|K~r}Xp^D<