summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* TODO-RELEASE: one fixed, one postponed, one addedDaniel Stenberg2010-09-291-6/+12
| | | | As we're already in feature freeze, I pushed the feature onwards.
* Renamed test1204 to test1117 to move it into the normal rangeDan Fandrich2010-09-292-4/+4
|
* Add gopher protocol definition to ILE/RPG binding.Patrick Monnerat2010-09-292-1/+16
| | | | OS400 compile script in test dir updated for chkhostname.
* krb5-gssapi: Remove several memory leaks.Julien Chaffraix2010-09-282-5/+14
| | | | | | | | | | | Remove a leak seen on Kerberos/MIT (gss_OID is copied internally and we were leaking it). Now we just pass NULL as advised in RFC2744. |tmp| was never set back to buf->data. Cleaned up Curl_sec_end to take into account failure in Curl_sec_login (where conn->mech would be NULL but not conn->app_data or conn->in_buffer->data).
* security.c: Remove Curl_sec_fflush_fd.Julien Chaffraix2010-09-283-16/+0
| | | | | The current implementation would make us send wrong data on a closed socket. We don't buffer our data so the method can be safely removed.
* security.c: We should always register the socket handler.Julien Chaffraix2010-09-281-6/+8
| | | | | | | | | Following a change in the way socket handler are registered, the custom recv and send method were conditionaly registered. We need to register them everytime to handle the ftp security extensions. Re-added the clear text handling in sec_recv.
* security.c: Fix Curl_sec_login after rewrite.Julien Chaffraix2010-09-283-5/+4
| | | | | | Curl_sec_login was returning the opposite result that the code in ftp.c was expecting. Simplified the return code (using a CURLcode) so to see more clearly what is going on.
* security.c: Readd the '\n' to the infof() calls.Julien Chaffraix2010-09-281-8/+8
| | | | | They are not automatically added and make the output of the verbose mode a lot more readable.
* security.c: Fix typo (PSBZ -> PBSZ)Julien Chaffraix2010-09-281-1/+1
|
* security.c: Fix ftp_send_command.Julien Chaffraix2010-09-281-2/+5
| | | | | My use of va_args was completely wrong. Fixed the usage so that we send the right commands!
* curl_easy_escape: don't escape "unreserved" charactersDaniel Stenberg2010-09-283-6/+9
| | | | | | | | According to RFC3986 section 2.3 the letters -, ., _ and ~ should not be percent-encoded. Reported by: Miguel Diaz Bug: http://curl.haxx.se/mail/lib-2010-09/0227.html
* multi: don't expire timeouts at disonnect or doneDaniel Stenberg2010-09-282-3/+4
| | | | | | | | | | | The functions Curl_disconnect() and Curl_done() are both used within the scope of a single request so they cannot be allowed to use Curl_expire(... 0) to kill all timeouts as there are some timeouts that are set before a request that are supposed to remain until the request is done. The timeouts are now instead cleared at curl_easy_cleanup() and when the multi state machine changes a handle to the complete state.
* Changed the TPF make file to get source files from Makefile.incDan Fandrich2010-09-272-91/+3
| | | | Patch was fixed and validated by David McCreedy.
* Added test case 1204 to test HTTP range failureDan Fandrich2010-09-272-1/+81
| | | | This is an attempt to reproduce bug #3076808
* multi_runsingle: set timeout error messagesDirk Manske2010-09-271-0/+13
| | | | | With the latest changes to fix the timeout handling with multi interface we lost the timeout error messages. This patch brings them back.
* TODO-RELEASE: updated list of issues to work onDaniel Stenberg2010-09-271-1/+17
|
* parsedate: allow time specified without secondsDaniel Stenberg2010-09-273-0/+9
| | | | | | | | | The date format in RFC822 allows that the seconds part of HH:MM:SS is left out, but this function didn't allow it. This change also includes a modified test case that makes sure that this now works. Reported by: Matt Ford Bug: http://curl.haxx.se/bug/view.cgi?id=3076529
* TFTP: re-indented the source codeDaniel Stenberg2010-09-261-61/+61
| | | | | Just made sure that the good old curl indentation style is used all over this file.
* TFTP: Work around tftpd-hpa upload bugTim Newsome2010-09-261-1/+7
| | | | | | tftpd-hpa has a bug where it will send an incorrect ack when the block counter wraps and tftp options have been sent. Work around that by accepting an ack for 65535 when we're expecting one for 0.
* Revert "security.c: buffer_read various fixes."Daniel Stenberg2010-09-241-6/+4
| | | | This reverts commit fbb38de415b7bb7d743e53a7b4b887ffb12b3e5b.
* security.c: removed superfluous parenthesesDaniel Stenberg2010-09-221-15/+15
| | | | | And also removed the FIXME where memory was zeroed just before freed, and some other minor whitespace changes.
* security.c: Update the #include statements after the rewrite.Julien Chaffraix2010-09-221-7/+4
|
* security.c: sec_write tweaksJulien Chaffraix2010-09-221-6/+10
| | | | | | - |fd| is now a curl_socket_t and |len| a size_t to avoid conversions. - Added 2 FIXMEs about the 2 unsigned -> signed conversions. - Included 2 minor changes to Curl_sec_end.
* security.c: _sec_send tweaksJulien Chaffraix2010-09-221-6/+7
| | | | | | - Renamed the method to sec_send now that we renamed sec_send to do_sec_send. - Some more variable renaming.
* security.c: sec_read tweaksJulien Chaffraix2010-09-221-19/+21
| | | | | - Renamed the function to sec_recv. - Renamed the parameters and variable to match the rest of the code.
* security.c: Curl_sec_fflush_fd tweaksJulien Chaffraix2010-09-221-5/+7
| | | | | - Use an early return as it makes the code more readable. - Added a FIXME about a conversion.
* security.c: sec_send tweaksJulien Chaffraix2010-09-221-27/+32
| | | | | | | | | | - Renamed it to do_sec_send as it is the function doing the actual transfer. - Do not return any values as no one was checking it and it never reported a failure (added a FIXME about checking for errors). - Renamed the variables to make their use more specific. - Removed some casts (int -> curl_socket_t, ...) - Avoid doing the htnl <-> nthl twice by caching the 2 results.
* security.c: Curl_sec_read_msg tweaksJulien Chaffraix2010-09-221-22/+30
| | | | | | | - Renamed the variables name to better match their intend. - Unified the |decoded_len| checks. - Added some FIXMEs to flag some improvement that did not go in this change.
* security.c: Curl_sec_set_protection_level tweakingJulien Chaffraix2010-09-222-37/+58
| | | | | | | | | | - Removed sec_prot_internal as it is now inlined in the function (this removed a redundant check). - Changed the prototype to return an error code. - Updated the method to use the new ftp_send_command function. - Added a level_to_char helper method to avoid relying on the compiler's bound checks. This default to the maximum security we have in case of a wrong input.
* security.c: factored the logic from Curl_sec_login into a dedicated method ↵Julien Chaffraix2010-09-221-38/+77
| | | | | | | that better reflect its intent. Introduced a helper method ftp_send_command that synchronously send an FTP query.
* security.c: Remove out_buffer as it was never written into.Julien Chaffraix2010-09-222-5/+1
|
* security.c: buffer_read various fixes.Julien Chaffraix2010-09-221-4/+6
| | | | | | | Tighten the type of the |data| parameter to avoid a cast. Also made it const as we should not modify it. Added a DEBUGASSERT on the size to be written while changing it.
* security.c: Made block_write return a CURLcode.Julien Chaffraix2010-09-221-19/+28
| | | | | While doing so, renamed it to socket_write to better match its function.
* security.c: Made block_read and sec_get_data return CURLcode.Julien Chaffraix2010-09-221-35/+40
| | | | | | | | | To do so, made block_read call Curl_read_plain instead of read. While changing them renamed block_read to socket_read and sec_get_data to read_data to better match their function. Also fixed a potential memory leak in block_read.
* Security.c: Fix headers guard to match the rest of the code.Julien Chaffraix2010-09-221-3/+4
|
* configure: Fix the LDAPS disable messageJulien Chaffraix2010-09-221-2/+4
| | | | | | | | | | ... for example when LDAP is not compiled. Fixed the logic to match the rest of the options' message that is we update the default message only if the option is not disabled after the different checks. Reported by: Guenter Knauf
* RELEASE-NOTES: sync with 8665d4e5 and c-ares >= 1.6.0 noteDaniel Stenberg2010-09-211-0/+2
|
* parse_remote_port: ignore colons without port numberDaniel Stenberg2010-09-211-0/+5
| | | | | | | | | Obviously, browsers ignore a colon without a following port number. Both Firefox and Chrome just removes the colon for such URLs. This change does not remove the colon for URLs sent over a HTTP proxy, so we should consider doing that change as well. Reported by: github user 'kreshano'
* RELEASE-NOTES: in sync with 19f45eaa799Daniel Stenberg2010-09-211-1/+4
|
* duphandle: use ares_dup()Daniel Stenberg2010-09-213-3/+5
| | | | | | | | | | curl_easy_duphandle() was not properly duping the ares channel. The ares_dup() function was introduced in c-ares 1.6.0 so by starting to use this function we also raise the bar and require c-ares >= 1.6.0 (released Dec 9, 2008) for such builds. Reported by: Ning Dong Bug: http://curl.haxx.se/mail/lib-2010-08/0318.html
* MacOSX-Framework: updates for SnowleopardHendrik Visage2010-09-211-32/+86
| | | | | | | | | | | 1) PPC64 appears to be an 10.5 only supported architecture, so I forced 10.5 for 64bit if there is a need for PPC64, else 64bit only does x86_64 2) proper "make clean" after every ./configure. fixes a bug where subsequent runs the 32bit do not get compiled 3) Added a version numbering curl-$VERSION} rather than the "stock standard" A
* RELEASE-NOTES: synced with 5fcc4332d62feDaniel Stenberg2010-09-211-2/+4
| | | | Removed the duplicate entry of Kamil in the credits.
* configure: don't enable RTMP if the lib detect failsDaniel Stenberg2010-09-201-13/+14
| | | | | | | librtmp is often statically linked and using sub dependencies like OpenSSL, so we need to make sure we can actually link with it properly before enabling it. Otherwise we easily end up trying to link with a RTMP lib that fails.
* TODO: added 8.4 non-gcrypt under GnuTLSDaniel Stenberg2010-09-201-0/+12
| | | | We must not assume gcrypt just because of GnuTLS
* configure: check for gcrypt if using GnuTLSDaniel Stenberg2010-09-201-0/+14
| | | | | | | | | | | | | | | | | 1 - libcurl assumes that there are gcrypt functions available when GnuTLS is. 2 - GnuTLS can be built to use libnettle instead as crypto library, which breaks assumption (1) This change makes configure make sure that if GnuTLS is requested and detected, it also makes sure that gcrypt is present or it errors out. This is mostly a way to make the user more aware of this flaw, the correct fix would be to detect which crypto layer that is in use and adapt our code to use that instead of blindly assuming gcrypt. Reported by: Michal Gorny Bug: http://curl.haxx.se/bug/view.cgi?id=3071038
* RELEASE-NOTES: sync from d2a7fd2fe65b to HEADDaniel Stenberg2010-09-181-1/+13
|
* FTP: fix bad check of Curl_timeleft() return codeDaniel Stenberg2010-09-181-1/+1
| | | | | When it returns 0 it means no timeout. Only a negative value means that we're out of time.
* LDAP: moved variable declaration to avoid compiler warnDaniel Stenberg2010-09-181-1/+1
| | | | | | If built without HTTP or proxy support it would cause a compiler warning due to the unused variable. I moved the declaration of it into the only scope it is used.
* LDAP: Use FALSE instead of bool_false when setting bits.closeTor Arntsen2010-09-181-1/+1
| | | | | | | | | | | | bool_false is the internal name used in the setup_once.h definition we fall back to for non-C99 non-stdbool systems, it's not the actual name to use in assignments (we use bool_false, bool_true there to avoid global namespace problems, see comment in setup_once.h). The correct C99 value to use is 'false', but let's use FALSE as used elsewhere when assigning to bits.close. FALSE is set equal to 'false' in setup_once.h when possible. This fixes a build problem on C99 targets.
* LDAP: Add missing declaration for 'result'Tor Arntsen2010-09-181-0/+1
|
View Lorry Controller Status