Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Fix a memory leak in BIO_free() which no current BIO can trigger; OpenSSL | miod | 2014-07-11 | 1 | -4/+3 |
| | | | | PR #3439 via OpenSSL trunk | ||||
* | Missing calloc() return value check in dgram_sctp_ctrl(); from Kurt Roeckx via | miod | 2014-07-11 | 1 | -11/+21 |
| | | | | OpenSSL trunk | ||||
* | Only import cryptlib.h in the four source files that actually need it. | jsing | 2014-07-11 | 17 | -47/+40 |
| | | | | | | | | Remove the openssl public includes from cryptlib.h and add a small number of includes into the source files that actually need them. While here, also sort/group/tidy the includes. ok beck@ miod@ | ||||
* | Explicitly include <openssl/opensslconf.h> in every file that references | jsing | 2014-07-10 | 3 | -3/+10 |
| | | | | | | | | | an OPENSSL_NO_* define. This avoids relying on something else pulling it in for us, plus it fixes several cases where the #ifndef OPENSSL_NO_XYZ is never going to do anything, since OPENSSL_NO_XYZ will never defined, due to the fact that opensslconf.h has not been included. This also includes some miscellaneous sorting/tidying of headers. | ||||
* | Simplify realloc() usage; ok tedu@ | miod | 2014-07-10 | 1 | -15/+6 |
| | |||||
* | Stop including standard headers via cryptlib.h - pull in the headers that | jsing | 2014-07-10 | 12 | -39/+57 |
| | | | | | | are needed in the source files that actually require them. ok beck@ miod@ | ||||
* | delete some casts. ok miod | tedu | 2014-07-10 | 1 | -2/+2 |
| | |||||
* | replace getservbyname_r with getaddrinfo for portability | bcook | 2014-07-10 | 1 | -6/+11 |
| | | | | ok jsing@ | ||||
* | include <limits.h> for LONG_MIN/LONG_MAX. | bcook | 2014-07-09 | 1 | -2/+2 |
| | | | | | | | Also remove <sys/filio.h> added from previous commit. This was the wrong way to get FIONBIO. ok jsing@ | ||||
* | add <sys/filio.h> header for FIONBIO | bcook | 2014-07-09 | 1 | -1/+2 |
| | | | | ok beck@ | ||||
* | Pretty much everything has SO_REUSEADDR - nuke the #ifdefs. | jsing | 2014-07-08 | 1 | -5/+1 |
| | | | | ok beck@ | ||||
* | We have EAI_FAMILY - remove the #ifdefs. | jsing | 2014-07-08 | 1 | -5/+1 |
| | | | | ok beck@ | ||||
* | More KNF. | jsing | 2014-07-08 | 1 | -11/+21 |
| | |||||
* | Avoid locking in BIO_get_port() by using getservbyname_r() instead of | jsing | 2014-07-08 | 1 | -31/+31 |
| | | | | | | getservbyname(). While here, provide a common/single return path. ok deraadt@ | ||||
* | Simplify various BIO_sock_* fuctions - less code, better variable names, | jsing | 2014-07-08 | 1 | -27/+12 |
| | | | | | | correct types and fewer casts. ok deraadt@ miod@ | ||||
* | Replace 48 lines of code with a single inet_pton() call. The previous | jsing | 2014-06-24 | 1 | -48/+3 |
| | | | | | | | | | | | | | handrolled version could not even make use of sscanf(), since that would not work with a certain antiquated compiler. It is worth noting that there is a tiny change in behaviour - previously calling BIO_get_host_ip() with something that looked like it might be a valid IP address (for example, "1." or even ".") would result in it returning failure rather than trying a BIO_gethostbyname() - now we'll always try a BIO_gethostbyname() if it was not a valid IPv4 address. ok beck@ miod@ deraadt@ | ||||
* | Actually make BIO_set_tcp_ndelay() work - TCP_NODELAY will not magically | jsing | 2014-06-24 | 1 | -16/+3 |
| | | | | | | appear by itself. ok beck@ miod@ | ||||
* | BIO_sock_init() no longer does anything, so stop calling it. | jsing | 2014-06-22 | 1 | -10/+1 |
| | |||||
* | Just use SOMAXCONN and IPPROTO_TCP, since we know we have them. | jsing | 2014-06-22 | 1 | -14/+4 |
| | |||||
* | In BIO_get_port(), use strol() with appropriate range checks rather than | jsing | 2014-06-22 | 3 | -34/+50 |
| | | | | | | | an atoi() followed by an unsigned short cast. This stops things like "-1" and "66536" from being considered to be "valid" port numbers. ok beck@ deraadt@ | ||||
* | tags as requested by miod and tedu | deraadt | 2014-06-12 | 21 | -21/+22 |
| | |||||
* | Tsk. Tsk. Someone forgot to compile test the other half. | jsing | 2014-06-11 | 1 | -2/+2 |
| | |||||
* | Provide support for non-funopen systems. | deraadt | 2014-06-11 | 1 | -8/+30 |
| | | | | ok beck | ||||
* | c-file-style hints, begone; ok beck | deraadt | 2014-06-11 | 1 | -1/+1 |
| | |||||
* | Stop using DSO_global_lookup to reach getaddrinfo() and friends | deraadt | 2014-06-08 | 1 | -40/+3 |
| | | | | discussed with tedu, ok jsing | ||||
* | Fix memory leak. | logan | 2014-06-03 | 1 | -0/+1 |
| | | | | | | (From Martin Brejcha) OK from tedu@, miod@ and deraadt@ | ||||
* | Remove __bio_h__attr__ wrapper around __attribute__, since earlier statements | miod | 2014-06-01 | 1 | -10/+4 |
| | | | | | | in this file directly use __attribute__. ok deraadt@ | ||||
* | Fix some more nasty stringyness in here by using asprintf instead of cruft. | beck | 2014-05-30 | 1 | -10/+7 |
| | | | | gets rid of the second last use of the awful DECIMAL_SIZE. | ||||
* | more: no need for null check before free | deraadt | 2014-05-30 | 8 | -41/+19 |
| | | | | ok tedu guenther | ||||
* | remove CONST_STRICT. ok beck deraadt | tedu | 2014-05-30 | 1 | -7/+0 |
| | |||||
* | Everything sane has stdio, and FILE *. we don't need ifdefs for this. | beck | 2014-05-29 | 2 | -8/+0 |
| | | | | ok to firebomb from tedu@ | ||||
* | Any sane platform has stdio. Stop pretending we will ever use a platform | beck | 2014-05-29 | 2 | -4/+0 |
| | | | | | that does not. "fire bomb" tedu@ | ||||
* | Almost nothing actually needs to include <openssl/e_os2.h>, however by | jsing | 2014-05-24 | 1 | -1/+1 |
| | | | | | | | including it they get <openssl/opensslconf.h>. So instead of pulling in <openssl/e_os2.h>, just pull in <openssl/opensslconf.h>. "go ahead" miod@ | ||||
* | Spell ssize_t correctly (the incorrect spelling is ossl_ssize_t). | jsing | 2014-05-24 | 1 | -9/+9 |
| | | | | ok miod@ | ||||
* | mark BIO_snprintf as deprecated, so warnings will be generated for it's | beck | 2014-05-22 | 1 | -2/+2 |
| | | | | use. ok miod@ tedu@ | ||||
* | Sort and group includes. | jsing | 2014-05-05 | 2 | -10/+11 |
| | |||||
* | Repair BIO_socket_nbio operation. | jsing | 2014-05-05 | 1 | -2/+3 |
| | | | | ok miod@ | ||||
* | Use C99 initializers for the various FOO_METHOD structs. More readable, and | miod | 2014-04-27 | 14 | -148/+132 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | avoid unreadable/unmaintainable constructs like that: const EVP_PKEY_ASN1_METHOD cmac_asn1_meth = { EVP_PKEY_CMAC, EVP_PKEY_CMAC, 0, "CMAC", "OpenSSL CMAC method", 0,0,0,0, 0,0,0, cmac_size, 0, 0,0,0,0,0,0,0, cmac_key_free, 0, 0,0 }; ok matthew@ deraadt@ | ||||
* | Replace all use of ERR_add_error_data with ERR_asprintf_error_data. | beck | 2014-04-26 | 3 | -20/+19 |
| | | | | | | | | This avoids a lot of ugly gymnastics to do snprintfs before sending the bag of strings to ERR, and eliminates at least one place in dso_dlfctn.c where it was being called with the incorrect number of arguments and using random things off the stack as addresses of strings. ok krw@, jsing@ | ||||
* | This is not FreeBSD. | jsing | 2014-04-26 | 1 | -18/+0 |
| | | | | ok miod@ | ||||
* | The usual idiom to cope with systems not defining socklen_t is to add a | miod | 2014-04-23 | 2 | -49/+8 |
| | | | | | | | | | | | | | | | | | | #define socklen_t int somewhere (or a typedef, whatever gives you an integer type of the size your system expects as the 3rd argument of accept(2), really). OpenSSL here is a bit more creative by using an union of an int and a size_t, and extra code if sizeof(int) != sizeof(size_t) in order to recover the proper size. With a comment mentioning that this has no chance to work on a platform with a stack growing up and accept() returning an int, fortunately this seems to work on HP-UX. Switch to the light side of the force and declare and use socklen_t variables, period. If your system does not define socklen_t, consider bringing it back to your vendor for a refund. ok matthew@ tedu@ | ||||
* | close socket in failure cases too | tedu | 2014-04-23 | 1 | -0/+3 |
| | |||||
* | if realloc failed, BIO_accept would leak memory and return NULL, causing | tedu | 2014-04-23 | 1 | -2/+11 |
| | | | | caller to crash. Fix leak and return an error instead. from Chad Loder | ||||
* | wrong calloc, see people do care | deraadt | 2014-04-21 | 1 | -1/+2 |
| | |||||
* | Replace entire printf-like guts with calls to libc snprintf. | deraadt | 2014-04-21 | 1 | -769/+46 |
| | | | | | funopen(3) is used to interface to BIO descriptors. ok guenther | ||||
* | Bring malloc/calloc/realloc sequences to modern standard | deraadt | 2014-04-21 | 8 | -27/+19 |
| | | | | ok guenther | ||||
* | We'll interpret a (void) cast on snprintf() to mean it's been verified that | guenther | 2014-04-19 | 4 | -23/+23 |
| | | | | | | truncation is either desirable, not an issue, or is detected and handled later ok deraadt@ | ||||
* | use intrinsic strlcpy and strlcat everywhere so we only have one set of | beck | 2014-04-19 | 2 | -12/+12 |
| | | | | | | funcitons to check for incorrect use. keep BUF_strlcpy and BUF_strlcat for API comptibility only. ok tedu@ | ||||
* | unifdef NO_SOCK | tedu | 2014-04-18 | 4 | -8/+0 |
| | |||||
* | Put the final pieces from e_os.h in the required places, and remove it. | deraadt | 2014-04-18 | 3 | -36/+3 |
| | | | | | "dance on it's grave" says beck ok guenther beck |