[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.
cheers,
felix