<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 30 August 2016 at 10:21, Jussi Kukkonen <span dir="ltr">&lt;<a href="mailto:jussi.kukkonen@intel.com" target="_blank">jussi.kukkonen@intel.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div class="h5">On 30 August 2016 at 05:17, Edwin Plauchu &lt;<a href="mailto:edwin.plauchu.camacho@linux.intel.com" target="_blank">edwin.plauchu.camacho@linux.<wbr>intel.com</a>&gt; wrote:<br>&gt;<br>&gt; From: Edwin Plauchu &lt;<a href="mailto:edwin.plauchu.camacho@intel.com" target="_blank">edwin.plauchu.camacho@intel.<wbr>com</a>&gt;<br>&gt;<br>&gt; This fixes commit 763a3d424bccf559a8d6add3dc1f27<wbr>46c82f2933<br>&gt;<br>&gt; Output was strange when using unzip to extract zip file.<br>&gt; This patch fixed so.<br>&gt;<br>&gt; [YOCTO #9551]<br>&gt;<br>&gt; Signed-off-by: Edwin Plauchu &lt;<a href="mailto:edwin.plauchu.camacho@intel.com" target="_blank">edwin.plauchu.camacho@intel.<wbr>com</a>&gt;<br>&gt; ---<br>&gt;  .../unzip/unzip/fix-security-<wbr>format.patch          | 198 ++++++++-------------<br>&gt;  1 file changed, 78 insertions(+), 120 deletions(-)<br>&gt;<br>&gt; diff --git a/meta/recipes-extended/unzip/<wbr>unzip/fix-security-format.<wbr>patch b/meta/recipes-extended/unzip/<wbr>unzip/fix-security-format.<wbr>patch<br>&gt; index c82f502..8e9b06c 100644<br>&gt; --- a/meta/recipes-extended/unzip/<wbr>unzip/fix-security-format.<wbr>patch<br>&gt; +++ b/meta/recipes-extended/unzip/<wbr>unzip/fix-security-format.<wbr>patch<br>&gt; @@ -9,131 +9,89 @@ Upstream-Status: Pending<br>&gt;<br>&gt;  Signed-off-by: Edwin Plauchu &lt;<a href="mailto:edwin.plauchu.camacho@intel.com" target="_blank">edwin.plauchu.camacho@intel.<wbr>com</a>&gt;<br>&gt;<br>&gt; -diff --git a/unzpriv.h b/unzpriv.h<br>&gt; -index c8d3eab..85e693a 100644<br>&gt; ---- a/unzpriv.h<br>&gt; -+++ b/unzpriv.h<br>&gt; -@@ -1006,7 +1006,7 @@<br>&gt; - #    define LoadFarStringSmall(x)   Qstrfix(x)<br>&gt; - #    define LoadFarStringSmall2(x)  Qstrfix(x)<br>&gt; - #  else<br>&gt; --#    define LoadFarString(x)        (char *)(x)<br>&gt; -+#    define LoadFarString(x)        &quot;%s&quot;,(char *)(x)<br><br></div></div>LoadFarString is used a _lot_ in various different ways, usually as argument to another macro or function. This sort of macro change seems potentially very dangerous in C, have you checked this is ok in every case?.<br></div></blockquote><div><br></div><div>Edwin just pointed out to me that the patch actually removes this macro change, and he&#39;s correct. My patch-of-patch reading failure.</div><div><br></div><div>So I&#39;m fine with this patch, sorry for noise.</div><div>  Jussi</div><div><br></div></div></div></div>