summaryrefslogtreecommitdiffstats
path: root/lib/libfetch
Commit message (Collapse)AuthorAgeFilesLines
* Mark libfetch as c99-clean. Always build at WARNS level 2 rather thandes2003-05-011-2/+2
| | | | | juggling with levels 2 and 3, as this has lead to world breakage for NOCRYPT users in the past.
* Don't dereference flags if NULL (see http.c rev 1.87)des2003-03-291-1/+1
|
* The flags passed in to _ftp_get_proxy may be nullmtm2003-03-191-1/+1
| | | | Approved by: des, markm (mentor)(implicit)
* Don't parse the proxy URL unless we're actually going to use it. No realdes2003-03-112-8/+16
| | | | functional difference, but debugging output will be less confusing.
* Add and document support for a FETCH_BIND_ADDRESS environment variabledes2003-03-033-4/+37
| | | | | | specifying a local address to bind sockets to. Caveat: lightly tested. PR: bin/37572
* No need for the OPENSSL_NO_KRB5 switch anymore.markm2003-01-311-1/+1
| | | | Fixed by: nectar
* Adjust for OpenSSL 0.9.7.markm2003-01-281-1/+1
|
* style(9): add parentheses to sizeof even when not strictly required.des2003-01-285-22/+22
| | | | MFC after: 3 days
* Fix signed/unsigned comparison warning/error from 'make release'jwd2003-01-281-4/+4
|
* Document .netrc support.des2003-01-221-0/+8
|
* Hook in the .netrc code + don't use pointers as if they were booleans.des2003-01-221-4/+6
|
* Experimental support for .netrc.des2003-01-222-0/+92
|
* Initialization paranoia.des2003-01-221-0/+8
|
* A negative offset means "get it all".des2003-01-221-2/+2
|
* Set auto-retry mode to avoid some spurious errors.des2003-01-031-0/+1
| | | | | Submitted by: Andre Albsmeier <andre.albsmeier@siemens.com> MFC after: 7 days
* english(4) police.schweikh2002-12-271-2/+2
|
* Reduce WARNS level to 3 in the non-crypto case to fix make release on sparc64.jake2002-12-041-1/+1
| | | | | | | This may only be turning up now to changes in the cpp predefines for sparc64, which may be turning on more strict alignment checking. Approved by: re
* Implement and document support for an HTTP_REFERER environment variable.des2002-11-282-12/+26
| | | | | | | PR: 28171 Submitted by: Andre Albsmeier <andre.albsmeier@mchp.siemens.de> Approved by: re (bmah) MFC after: 1 week
* Fix two typos.fenner2002-11-081-2/+2
|
* Use __func__ and break a long line.des2002-10-301-5/+4
|
* Fix a bug in fenner's _fetch_writev() patch (rev 1.36)des2002-10-301-1/+6
| | | | Submitted by: fenner
* Recognize the (incorrect) error code a MediaHawk server sends in reply todes2002-10-301-0/+1
| | | | | | unrecognized commands such as MDTM. Requested by: Stephen Roome <stephen_roome@pepcross.com>
* Clarify my feelings towards fetch / libfetch.des2002-10-301-1/+0
|
* Recommit the non-broken parts of 1.34 and 1.37.des2002-10-301-6/+11
| | | | Change the type and name of a variable introduced in 1.33.
* Don't forget to set the error flag when _fetch_read() returns -1.des2002-10-301-2/+6
|
* Reinstate revs 1.35-36 and 1.38. Revisions 1.34 and 1.37 were specificallyimp2002-10-301-12/+38
| | | | | | | | the root cause of the bus errors I was experiencing. Submitted by: fenner Tested by: obrien Prompted by: peter
* Fix `pkg_add -r' by backing out revs 1.34-1.38.obrien2002-10-291-52/+19
| | | | | Revs 1.37-8 produce a bus error in some environments. Revs 1.34-6 do not bus error, but write corrupted files.
* libfetch is now WARNS5 clean in the non-SSL case.des2002-10-281-1/+1
|
* Fix an off-by-one error (> where >= should have been used) which causeddes2002-10-281-6/+5
| | | | | | | | | _fetch_writev() to incorrectly report EPIPE in certain cases. Also fix a number of const warnings by using __DECONST(), plus a signed / unsigned comparison by casting the rhs to ssize_t. Submitted by: fenner, Craig Rodrigues <rodrigc@attbi.com>
* Up WARNS to 3 if not building with crypto.fenner2002-10-281-1/+3
| | | | Approved by: des
* Slight amendment to rev 1.34: instead of considering any short read andes2002-10-271-6/+8
| | | | | | error, only report an error if no data was read at all (unless len was 0 to start with). Otherwise, the final read of practically any transfer will end in a fatal error.
* Introduce _fetch_writev(), which is the conn_t version of writev(2). Indes2002-10-272-7/+38
| | | | | | | | | | | | | | the SSL case, it is no different from the old _fetch_write(), but in the non-SSL case it uses writev(2) to send the entire vector as a single packet (provided it can fit in one packet). Implement _fetch_write() and _fetch_putln() in terms of _fetch_writev(). This should improve performance in the non-SSL case (by reducing protocol overhead) and solve the problem where too-smart-for-their-own-good firewalls reject FTP packets that do not end in CRLF. PR: bin/44123 Submitted by: fenner
* Eliminate two cases of undefined behaviour: total in _fetch_write() wasdes2002-10-272-0/+2
| | | | | | | | not initialized before use, and _http_growbuf() did not return a value on success. Reported by: Peter Edwards <pmedwards@eircom.net> MFC after: 2 weeks
* Back out the previous commit, and fix the bug rather than try to hide itsdes2002-10-271-10/+18
| | | | | | | symptoms: make timeouts and short transfers fatal, and set errno to an appropriate value (ETIMEDOUT for a timeout, EPIPE for a short transfer). MFC after: 2 weeks
* The FTP connection caching needs a better interface -- connections arenjl2002-10-251-0/+4
| | | | | | | | | | | | | closed through _fetch_close() which is the only one who knows the connection REALLY was closed (since ref -> 0). However, FTP keeps its own local cached_connection and checks if it is valid by comparing it to NULL. This is bogus since it may have been freed elsewhere by _fetch_close(). This change checks if we are closing the cached_connection and the ref is 1 (soon to be 0). If so, set cached_connection to NULL so we don't accidentally reuse it. The REAL fix should be to move connection caching to the common.c level (_fetch_* functions) and NULL the cache(s) in _fetch_close(). Then all layers could benefit from caching.
* Use a warns setting we can catch regressions with.obrien2002-10-201-3/+1
|
* When recycling a cached connection, increment the reference count so thattjr2002-10-201-1/+1
| | | | | the heap block does not get freed and reused. This should fix the pkg_add -r crashes that have been happening for months.
* HTTP authentication got broken in rev. 1.49.ru2002-10-031-4/+4
|
* Zap now-unused SHLIB_MINORpeter2002-09-281-1/+0
|
* Fix an infinite loop when _fetch_read() can return 0 (if thealfred2002-09-201-1/+5
| | | | | connection is broken), take this into account and return at this point.
* Make _fetch_connect() always set the error code.fenner2002-09-172-4/+5
| | | | Tell ftp that _fetch_connect() always sets the error code (http already knew)
* Surround prototypes with __BEGIN_DECLS / __END_DECLS.des2002-07-221-0/+4
| | | | | PR: misc/40399 MFC after: 3 days
* Cope with 2292bis-01 getaddrinfo (no NI_WITHSCOPEID, always attachume2002-07-021-0/+1
| | | | | | | scope identifier). Approved by: des MFC after: 3 weeks
* Reintroduce debugging code that somehow got lost in a previous revision.des2002-06-241-0/+2
|
* Fix incorrect library ordering. I thought I'd committed this already...des2002-06-211-2/+2
|
* Don't try to dereference conn when we know it's NULL.des2002-06-191-2/+5
|
* Add a reference count to struct fetchconn so we don't prematurely close anddes2002-06-113-1/+22
| | | | free a cached FTP connection.
* libfetch now depends on libcrypto and libssl.ru2002-06-061-5/+6
|
* Make SSL support conditional on NOCRYPT.des2002-06-054-0/+21
|
* During buildworld, "regular" libraries are built before crypto stuff, sodes2002-06-051-2/+2
| | | | | libfetch can't depend on lib{crypto,ssl}. Move the dependency to fetch until we can figure out how to fix this.
OpenPOWER on IntegriCloud