[patch] tailmatch() returns true for invalid host.

2013-04-09 Thread YAMADA Yasuharu
Hello cURL hackers, I noticed that the cookie tailmatch() function returns true if following case: - cookie comain is example.com or .example.com. - hostname is abcexample.com Because tailmatch() does not check whether the charactor of ab'c'example.com position is '.'. So, the current cURL

--with-darwinssl on iOS ?

2013-04-09 Thread Paul Joyce
I have configured curl to use darwinssl, and here is the report (for the simulator config, but device config is the same): curl version: 7.28.0 Host setup: i386-apple-darwin Install prefix: /usr/local Compiler:

Re: [patch] tailmatch() returns true for invalid host.

2013-04-09 Thread Daniel Stenberg
On Tue, 9 Apr 2013, YAMADA Yasuharu wrote: I noticed that the cookie tailmatch() function returns true if following case: - cookie comain is example.com or .example.com. - hostname is abcexample.com Thanks. Have you tried to write up a test case for the test suite that repeats the problem

Re: A strange problem on IOS(Empty reply from server)

2013-04-09 Thread Daniel Stenberg
On Tue, 9 Apr 2013, zhangyongchao wrote: A strange problem here , I upload a image which the size is 341623, I trace the msg, a msg We are completely uploaded and fine is displayed; But then, it's seem dead lock no response, after about at least 3~5 minutes, libcurl display Empty reply from

Re: [patch] tailmatch() returns true for invalid host.

2013-04-09 Thread Daniel Stenberg
On Tue, 9 Apr 2013, Daniel Stenberg wrote: Thanks. Have you tried to write up a test case for the test suite that repeats the problem and proves your fix? I wrote up test 1216 and just pushed it to the git repo. I tried to repeat what you described but it doesn't seem to repeat the problem.

Recognizing a non-standard FTP response to PWD MKD commands

2013-04-09 Thread Bill Middlecamp
I would like to request that a change be added to libcURL so that it will accept a slightly non-compliant FTPD 257 response. I have written an FTP client to interface with a customized FTP daemon in a storage appliance for the video industry. The vendor of this appliance implemented a non-RFC

Re: A strange problem on IOS(Empty reply from server)

2013-04-09 Thread Nick Zitzmann
On Apr 8, 2013, at 8:24 PM, zhangyongchao zhangy...@hotmail.com wrote: I complied 7.29.0 source code and used in iPad3. A strange problem here , I upload a image which the size is 341623, I trace the msg, a msg We are completely uploaded and fine is displayed; But then, it's seem dead

Re: --with-darwinssl on iOS ?

2013-04-09 Thread Nick Zitzmann
On Apr 9, 2013, at 1:54 AM, Paul Joyce pjo...@we-do-it.com wrote: curl version: 7.28.0 […] It says HTTPS is in the list, so configure was happy enough. When I attempt to compile, I get an error …./curl-7.28.0/lib/curl_darwinssl.c:905:6: Use of undeclared identifier

Re: [PATCH] Utils: Fix lost OpenSSL output with -t.

2013-04-09 Thread Guenter
Hi Richard, On 04.04.2013 11:55, Richard Michael wrote: The OpenSSL pipe wrote to the final CA bundle file, but the encoded PEM output wrote to a temporary file. Consequently, the OpenSSL output was lost when the temp file was renamed to the final file at script finish (overwriting the final

Re: Name lookup/DNS resolution mechanism of libcurl

2013-04-09 Thread Michael-O
Am 2013-04-02 00:43, schrieb Guenter: Hi Michael, On 01.04.2013 23:24, Michael-O wrote: Am 2013-04-01 22:52, schrieb Guenter: 1st we need to know if this is also an issue with the default windows resolver; so have you also tested with a non-c-ares build? If it only occures with c-ares then

Re: Name lookup/DNS resolution mechanism of libcurl

2013-04-09 Thread Guenter
Hi Mike, On 09.04.2013 20:01, Michael-O wrote: Günter, were you able to look at the Makefiles of c-ares and curl? yes, today commited two patches - one for curl and one for c-ares ... Gün. --- List admin:

Re: Name lookup/DNS resolution mechanism of libcurl

2013-04-09 Thread Michael-O
Am 2013-04-09 20:31, schrieb Guenter: Hi Mike, On 09.04.2013 20:01, Michael-O wrote: Günter, were you able to look at the Makefiles of c-ares and curl? yes, today commited two patches - one for curl and one for c-ares ... Do you want to to clone from Git and test it?

curl mingw compile and shared v/s static libraries?

2013-04-09 Thread Ben Greear
I updated to the latest curl recently and now my mingw build is different. It used to produce these two lib files: /lib/.libs/libcurl.a /lib/.libs/libcurl.dll.a Now, it does not. I notice that it appears to only be able to do shared OR static libraries...I am pretty sure it used to do both.

Re: Name lookup/DNS resolution mechanism of libcurl

2013-04-09 Thread Guenter
Hi Mike, On 09.04.2013 20:46, Michael-O wrote: Am 2013-04-09 20:31, schrieb Guenter: yes, today commited two patches - one for curl and one for c-ares ... Do you want to to clone from Git and test it? sure, that would be great! But remember that Daniel might at any time tag and release ...

Re: Name lookup/DNS resolution mechanism of libcurl

2013-04-09 Thread Daniel Stenberg
On Tue, 9 Apr 2013, Guenter wrote: But remember that Daniel might at any time tag and release ... so only a short while for last-minute hot-fixes :-P EXactly. Please be careful with last-minute pushes now. My plan is to release what's HEAD in git in 14-15 hours or so. -- / daniel.haxx.se

Re: Recognizing a non-standard FTP response to PWD MKD commands

2013-04-09 Thread Daniel Stenberg
On Tue, 9 Apr 2013, Bill Middlecamp wrote: I would like to request that a change be added to libcURL so that it will accept a slightly non-compliant FTPD 257 response. Thanks a lot! Accepted, with a little edit of the code. I also added test case 1217 to verify your fix before I pushed. --

Re: Name lookup/DNS resolution mechanism of libcurl

2013-04-09 Thread Gisle Vanem
Guenter li...@gknw.net wrote: On 09.04.2013 20:01, Michael-O wrote: Günter, were you able to look at the Makefiles of c-ares and curl? yes, today commited two patches - one for curl and one for c-ares ... Thanks for fixing this. I wrote about this in this thread:

Re: [PATCH] Utils: Fix lost OpenSSL output with -t.

2013-04-09 Thread Guenter
On 09.04.2013 18:08, Guenter wrote: On 04.04.2013 11:55, Richard Michael wrote: The OpenSSL pipe wrote to the final CA bundle file, but the encoded PEM output wrote to a temporary file. Consequently, the OpenSSL output was lost when the temp file was renamed to the final file at script finish

Re: [PATCH] Utils: Fix lost OpenSSL output with -t.

2013-04-09 Thread Richard Michael
Hello Günter, Thanks for looking at the patch. I don't have much to add, except: On Wed, Apr 10, 2013 at 12:40 AM, Guenter li...@gknw.net wrote: it seems that my verification was incomplete; looking again at the patch I felt that it would not work on Windows, and in fact it didnt, sigh! So

Re: Name lookup/DNS resolution mechanism of libcurl

2013-04-09 Thread Guenter
Hi Gisle, On 10.04.2013 00:07, Gisle Vanem wrote: Thanks for fixing this. I wrote about this in this thread: http://curl.haxx.se/mail/lib-2013-01/0131.html earlier this year. Your patch below looks good here. Except: +#if !defined(USE_ARES) !defined(USE_THREADS_WIN32) # define

Re: [PATCH] Utils: Fix lost OpenSSL output with -t.

2013-04-09 Thread Guenter
Hi Richard, On 10.04.2013 01:15, Richard Michael wrote: Sorry you've had to spend time on this. naaa, no sorry - we thank you that you found the bug!! And even with your patch it was already better than before cause it was completely broken; with you patch it worked at least on Linux ... if

Re: [patch] tailmatch() returns true for invalid host.

2013-04-09 Thread YAMADA Yasuharu
The point is the second flag of cookie data file. This flag is tail match flag. I attach a patch for test1216. Please check it. Best regards, thanks === YAMADA Yasuharu ACCESS CO., LTD. yasuharu.yam...@access-company.com 2013/4/9 Daniel Stenberg dan...@haxx.se On Tue, 9 Apr

Re: Name lookup/DNS resolution mechanism of libcurl

2013-04-09 Thread Yang Tse
Hi Gün, On Wed, Apr 10, 2013, Guenter li...@gknw.net wrote: honestly I'm not happy with that hack, and if someone has something better in mind please speak up! Modify Makefile.m32 in order to make it define one out of three possibilities ENABLE_ARES, ENABLE_SYNC_DNS or ENABLE_THREADS_WIN32.

Re: curl mingw compile and shared v/s static libraries?

2013-04-09 Thread Yang Tse
On Tue, Apr 9, 2013, Ben Greear gree...@candelatech.com wrote: I updated to the latest curl recently and now my mingw build is different. [...] Now, it does not. I notice that it appears to only be able to do shared OR static libraries...I am pretty sure it used to do both. Is this on