Crash in Curl_disconnect

2011-08-25 Thread Giany
Hello, We are using the curl library in Asterisk (Voip stuff) in order to make some http requests. We have noticed that the library crashes when multiple requests are sent to a ip that gives a "request timed out" response. Our application crashes in the curl library: (gdb) bt #0  0x003e8ce3

Problem with NTLM_SSO and signals

2011-08-25 Thread Dan Fandrich
libcurl calls 'fork' and 'waitpid' in only one case: to run the helper app in the newly-added NTLM_SSO authentication method. But, forking in a library is somewhat dangerous as it can interfere with the parent application's handling of signals. Here's one failure scenario: an application can insta

RE: [PATCH] SMTP Modifications

2011-08-25 Thread Steve Holme
Hi Yang, > In order to consider that work complete, it would be convenient to > better integrate samba's winbind daemon auth helper > implementation when used. > > Steve now that you are more familiar with git these changes/fixes > might disturb your work less. No worries. I have enclosed a patc

Re: RE: RE: RE: Linker errors when statically compiled linkinglibcurl+openssl to my project

2011-08-25 Thread Gisle Vanem
"Steve Holme" wrote: I use Visual Studio (and have solution and project files for .NET 2003 (v7.1), 2005 (v8.0), 2008 (v9.0) and 2010 (v10.0)) and have them configured to build against OpenSSL (as a DLL rather than statically) and to include Visual Studio uses a 2 pass linker. gnu-ld does not

Re: [PATCH] SMTP Modifications

2011-08-25 Thread Yang Tse
2011/8/25 Steve Holme wrote: > If my post NTLM patch(es) are polished enough we might be able > to squeeze them into v7.22.0 ;-) ? In order to consider that work complete, it would be convenient to better integrate samba's winbind daemon auth helper implementation when used. At many places the f

RE: [PATCH] SMTP Modifications

2011-08-25 Thread Steve Holme
> Ha, ha left out ther references in previous message, I thought I was going mad for a minute as I couldn't see them in your previous message ;-) S. --- List admin: http://cool.haxx.se/list/listinfo/curl-library Etiquette: http://c

Re: [PATCH] SMTP Modifications

2011-08-25 Thread Yang Tse
Ha, ha left out ther references in previous message, Following sections of mentioned RFCs indicate that at least SASL for POP3, SMTP and IMAP follow same scheme relative to the 'initial-response' being optional for each respective authenticate-command. POP3 SASL - rfc5034 (section 5) http://tools

Re: [PATCH] SMTP Modifications

2011-08-25 Thread Yang Tse
2011/8/25 Daniel Stenberg wrote: > Oh, I didn't mean to suggest or imply that the "rules" have been violated in > any way. I was just thinking that this auth bits stuff we're discussing > might. Yes certainly. Any new option/symbol/function in external headers would easily pinpoint such a violati

Re: [PATCH] SMTP Modifications

2011-08-25 Thread Yang Tse
2011/8/25 Steve Holme wrote: > Hi Dan, > >> As I brought up in March when this first came up, since >> SMTP uses SASL for authentication, I suggest renaming >> these to something like CURL_SASL_AUTH_x so that >> they can be used for other SASL-using protocols in the future. > > I totally agree but

Re: sftp download issue (Daniel Stenberg)

2011-08-25 Thread Daniel Stenberg
On Thu, 25 Aug 2011, Sumukh Anantha Manohar wrote: Eh, so does it work for _any_ sftp files at all? If so, which? Even if i try to download a 2MB file with sftp protocol, sometimes it will download completely and sometimes it will show the error : **curl: (18) transfer closed with 574 byte

RE: RE: RE: RE: Linker errors when statically compiled linking libcurl+openssl to my project

2011-08-25 Thread Steve Holme
Hiya, > > Can you add -winmm and see if that decreases the number of errors by 1? If    > > not, then there is something else going wrong (as the unresolved externals > > for ws2_32 have got me baffled at the minute)! > > Looks like this changes nothing :( Mmmm very strange. I must admit I don't

RE: [PATCH] SMTP Modifications

2011-08-25 Thread Steve Holme
Hi all, > Patch requested today should show that there's little left. But > functionality this patch would provide already existed in first > patches provided Jul-29 and Aug-02 by Steve. Given that this, potentially, is part of much larger work would it be best to leave this until v7.22.1, along

Re: [PATCH] SMTP Modifications

2011-08-25 Thread Daniel Stenberg
On Thu, 25 Aug 2011, Yang Tse wrote: We do have these feature freeze periods in order to "settle" and iron out bugs. Yes certainly. I've attached chronology that shows that feature freeze period has not been ignored. Everything pushed after Aug-16 have been fixes for stuff already in master

RE: [PATCH] SMTP Modifications

2011-08-25 Thread Steve Holme
Hi Dan, > As I brought up in March when this first came up, since > SMTP uses SASL for authentication, I suggest renaming > these to something like CURL_SASL_AUTH_x so that > they can be used for other SASL-using protocols in the future. I totally agree but can the optional initial response flag

Re: [PATCH] SMTP Modifications

2011-08-25 Thread Yang Tse
2011/8/25 Daniel Stenberg wrote: > I think adding it among the bits is better than having it as a completely > separate option. I also prefer that one. > But also, as Steve brought up, we're in feature freeze right now and even if > it isn't a rule we can ignore at times I think we first need to

Re: [PATCH] SMTP Modifications

2011-08-25 Thread Dan Fandrich
On Thu, Aug 25, 2011 at 06:52:51PM +0200, Yang Tse wrote: > #define CURLMAIL_AUTH_LOGIN (1<<0) > #define CURLMAIL_AUTH_PLAIN (1<<1) > #define CURLMAIL_AUTH_CRAM_MD5(1<<2) > #define CURLMAIL_AUTH_DIGEST_MD5 (1<<3) > #define CURLMAIL_AUTH_GSSAPI (1<<4) > #define CURLMAIL_AUTH_EX

Re: [PATCH] SMTP Modifications

2011-08-25 Thread Daniel Stenberg
On Thu, 25 Aug 2011, Yang Tse wrote: Something very similar is already done for other protocols ie CURLOPT_HTTPAUTH. Daniel, should we use an independent CURLOPT_MAIL_AUTH_DISABLE_INITIAL_RESPONSE option or do we simply let it be a bit for the CURLOPT_MAILAUTH one? I don't mind having it a

Re: [PATCH] SMTP Modifications

2011-08-25 Thread Yang Tse
Hi, In case a single CURLOPT_MAILAUTH 'bitmasked' option is favoured relevant bits in curl.h could be something like the following, in which lower bits are used for authentication mechanisms and higher bits for parametrization purposes. Bits in between free for future expansion. #define CURLMAIL_

Re: can Curl use socket() created by application?

2011-08-25 Thread Dan Fandrich
On Thu, Aug 25, 2011 at 11:58:16AM -0400, amit paliwal wrote: > i fixed it, there was some problem at application side . Thanks For the > great help. Just one query, do I need to set these socket related options once > only or every-time when I call curl_easy_perform() I need to set them? Onc

Re: can Curl use socket() created by application?

2011-08-25 Thread amit paliwal
On Wed, Aug 24, 2011 at 6:21 PM, Daniel Stenberg wrote: > On Wed, 24 Aug 2011, amit paliwal wrote: > > I am using correct version of the library here. Once I connected to the >> server, I have done some manual send() and recv() which works fine, after >> that I tried sending HTTP POST to the ser

RE: [PATCH] SMTP Modifications

2011-08-25 Thread Steve Holme
Hi, > It seems to me that you want to provide the capability of _not_ sending > the 'client initial response' in the AUTH command in order to allow > interoperability with servers that support the AUTH extension but that > don't tolerate the optional client initial response argument. Exactly that

Re: [PATCH] SMTP Modifications

2011-08-25 Thread Yang Tse
2011/8/24 Steve Holme wrote: > Please find attached my next modification for the SMTP protocol - which adds > the ability to specify whether authenticated connections (AUTH PLAIN and > AUTH LOGIN) send the initial response in the AUTH command or in the next > command. This can be set via the CURLO

Re: [PATCH] Added AUTH NTLM for SMTP

2011-08-25 Thread Yang Tse
Hi Steve, 2011/8/24 Yang Tse wrote: > Right now I see nothing relative to all this in already provided > patches. I'll dig a bit further in case it is hidden somewhere, but > you are the one who actually knows by heart where should we arrive. Yep, your initial consolidated patch already included

RE: RE: RE: RE: Linker errors when statically compiled linking libcurl+openssl to my project

2011-08-25 Thread Oleksiy
> I just had to check MSDN for this one - and it says this function is defined > in winmm.lib. > > I'm not too familiar with librtmp, so I just downloaded the source for it > and checked out it's makefile which includes the following when building > with mingw: > >

RE: RE: Linker errors when statically compiled linking libcurl+openssl to my project

2011-08-25 Thread Steve Holme
Hiya, > ws2_32.lib was already linked, I added  gdi32.lib. Errors decreased. The following remain > > ../lib/librtmp.a(rtmp.o):rtmp.c:(.text+0x1b3b): undefined reference to `inet_addr@4' > ../lib/librtmp.a(rtmp.o):rtmp.c:(.text+0x732c): undefined reference to `htonl@4' > ../lib/librtmp.a(rtmp.o):r

Re: Re: Re: Problem using static libraries libcurl with OpenSSL, minGW, Eclipse

2011-08-25 Thread Oleksiy
As I was told on StackOverflow, maybe the openssl-libs I link against seem to be import-libraries. > > "That means they only contain the information your code needs to call functions and then load and call the corresponding functions from the dll. > So the problem i

RE: RE: Linker errors when statically compiled linking libcurl+openssl to my project

2011-08-25 Thread Oleksiy
>Looking at rand_win.c it uses CreateDCA, CreateCompatiableDC(), > GetDeciceCaps() etc... which are defined in gdi32.lib. > > > These are defined in ws2_32.lib. > > What libraries is your application linking against? I'm not a MinGW / > Eclipse user but it looks like

RE: Linker errors when statically compiled linking libcurl+openssl to my project

2011-08-25 Thread Steve Holme
Hi, > I am trying to statically link libcurl (with SSL) into my project but I am encountering lots of linker errors: > > ../lib/libcrypto.a(rand_win.o):rand_win.c:(.text+0xa9c): undefined reference to `CreateDCA@16' > ../lib/libcrypto.a(rand_win.o):rand_win.c:(.text+0xaa9): undefined reference to

Linker errors when statically compiled linking libcurl+openssl to my project

2011-08-25 Thread Oleksiy
I am trying to statically link libcurl (with SSL) into my project. Downloaded from here - http://www.gknw.net/mirror/curl/win32/curl-7.21.7-devel-mingw32.zip but I am encountering lots of linker errors: ../lib/libcrypto.a(rand_win.o):rand_win.c:(.text+0xa9c): undefined reference to

Re: Re: Re: Problem using static libraries libcurl with OpenSSL, minGW, Eclipse

2011-08-25 Thread Oleksiy
Gisle Vanem wrote: >"Oleksiy" wrote: > > > g++ -DCURL_STATICLIB -O0 -g3 -Wall -c -fmessage-length=0 -osrc\main.o > > ..\src\main.cpp > > > > g++ -L..\lib -oYTUploader.exe src\main.o -lcurl -lws2_32 -lwldap32 > > -leay32 -lssleay32 -lz > > The same thing > >