[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Chicken-hackers] [patch] utils: qs not escaping pipes

From: Felix
Subject: Re: [Chicken-hackers] [patch] utils: qs not escaping pipes
Date: Fri, 05 Apr 2013 22:52:34 +0200 (CEST)

From: Peter Bex <address@hidden>
Subject: Re: [Chicken-hackers] [patch] utils: qs not escaping pipes
Date: Tue, 2 Apr 2013 23:22:38 +0200

> On Sun, Feb 17, 2013 at 03:53:42PM +0100, Peter Bex wrote:
>> Could someone more knowledgeable about Windows please take a look at
>> the version for that platform?  That implementation does not look correct
>> at all.  It would be great if we could have a correct version for Windows
>> as well.
> Here's a new patch for both Unix and Windows.  Again, I'd appreciate if
> someone could test this on Windows.
> The fact that Felix (silently!) had to push a fix for qs to also escape
> the pipe character really shows the need for a whitelist approach.
> Florian's fix takes care of this properly.  I hope this addition
> for Windows helps make it work everywhere, so we can finally get this
> problem fixed.

Thanks. The patch is severely broken (you char=? delim which is a string,
and some other things), but I was able to test a modified version
on Windows. I don't have a repository handy here, but will post an
improved version in the next days.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]