<br><br><div class="gmail_quote">On Thu, Jun 24, 2010 at 10:33 AM, Bert Freudenberg <span dir="ltr"><<a href="mailto:bert@freudenbergs.de">bert@freudenbergs.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div><div></div><div class="h5"><br>
<br>
On 24.06.2010, at 19:30, Henrik Johansen wrote:<br>
<br>
><br>
> There's a bug in the alias resolution sqGetFilenameFromString in sqUnixMain.c<br>
><br>
> In the line<br>
> filenameLength = readLink(aCharBuffer, aCharBuffer, MAXPATHLEN)<br>
> It replaces the old link path with the resolved link path, however links may be relative, eg. /bin/sh linking to bash rather than /bin/bash.<br>
><br>
> Thus, file opening, and other operations using this function will fail if encountering a relative link.<br>
><br>
> F.ex. print (on Ubuntu, where sh -> dash):<br>
><br>
> |openedRelative|<br>
> openedRelative := nil.<br>
> [FileStream readOnlyFileNamed: '/bin/sh' do: [:fs | openedRelative := true]] on: FileDoesNotExistException do: [openedRelative := false].<br>
> openedRelative<br>
><br>
> and you get false.<br>
><br>
> Cheers,<br>
> Henry<br>
<br>
</div></div>Yeah I've been bitten by that too. Do you have a patch? Actually, Ian prefers to be sent a fully patched file ...<br></blockquote><div><br></div><div>You'll find its fixed in the Cog sources. Alas there are other changes there-in, but at least that function is not affected by other changes.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<font color="#888888"><br>
- Bert -<br>
<br>
<br>
</font></blockquote></div><br>