<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, 7 Sep 2019, 11:16 pm Alex Henrie, <<a href="mailto:alexhenrie24@gmail.com">alexhenrie24@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Sat, Sep 7, 2019 at 11:28 AM Alex Henrie <<a href="mailto:alexhenrie24@gmail.com" target="_blank" rel="noreferrer">alexhenrie24@gmail.com</a>> wrote:<br>
><br>
> On Sat, Sep 7, 2019 at 1:46 AM Alexandre Julliard <<a href="mailto:julliard@winehq.org" target="_blank" rel="noreferrer">julliard@winehq.org</a>> wrote:<br>
> ><br>
> > Alex Henrie <<a href="mailto:alexhenrie24@gmail.com" target="_blank" rel="noreferrer">alexhenrie24@gmail.com</a>> writes:<br>
> ><br>
> > > I think that instead of making the buffer bigger, it would be better<br>
> > > to add kernelbase to the IMPORTS line of<br>
> > > dlls/advpack/tests/Makefile.in, include shlwapi.h in files.c, and then<br>
> > > change lstrcatA to StrCatBuffA.<br>
> ><br>
> > Arguably such buffer overflows in tests don't matter either way, but<br>
> > using a function that would silently truncate the path is not an<br>
> > improvement in general.<br>
><br>
> That makes sense. However, I misspoke when I said to change lstrcatA<br>
> to StrCatBuffA: The compiler warning is about sprintf, not strcat. So<br>
> in this case I think it would make the most sense to replace the<br>
> sprintf with lstrcpyA and lstrcatA similar to the rest of the tests in<br>
> the function.<br>
<br>
Actually, PathCombineA would be even better.<br>
<br>
-Alex<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Sorry for being late, I was a bit busy.</div><div dir="auto"><br></div><div dir="auto">I'll use PathCombineA then.</div><div dir="auto"><br></div><div dir="auto">Isira</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
</blockquote></div></div></div>