| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
| |
1. it's not a bug; it's a caveat.
2. "slightly unsafe" gives me the willies.
3. one .Xr to malloc should suffice
ok deraadt jmc
|
|
|
|
|
| |
otherwise we end up needlessly replacing a NUL with a NUL.
OK deraadt@
|
| |
|
|
|
|
| |
OK deraadt@
|
|
|
|
| |
review by millert, binary checking process with doug, concept with guenther
|
|
|
|
|
|
|
|
| |
using the name of relayd relay or smtpd pki, use a 32 byte arc4random
buffer that should be unique for the context. This fixes an issue in
OpenSMTPD when a long pki name could break the configuration.
OK gilles@ benno@
|
|
|
|
|
|
|
|
| |
since yesterday, "mandoc -Tlint -Wfatal" can no longer fail.
Instead, as suggested by deraadt@, provide a manlint target
that is *not* run during make build, but can be run
whenever you want to check syntax of manuals.
"nice stuff" deraadt@
|
|
|
|
| |
ok tedu@
|
|
|
|
| |
other systems to fit into the same mold, so add copyright
|
| |
|
|
|
|
|
|
|
|
| |
Use the same logic from the read, write, accept functions to inform
the caller wether a readable or writable socket is needed. After
that event, the connect function must be called again. All the
checks before connecting are done only once.
OK tedu@
|
|
|
|
| |
OK jmc@
|
|
|
|
|
|
| |
proxy. Implement the -proxy feature in the same hackish way as
-starttls.
OK jsing@
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Check the return value of every relevant function call.
- If BIO_new() returns NULL instead of a valid BIO, do not attempt to
blindly use the NULL value as a BIO throughout the rest of the code.
- Ensure that bio_out is freed by BIO_free_all() at the end of all
error paths.
ok doug@
|
|
|
|
| |
nor are they the same size.
|
|
|
|
|
|
|
|
| |
Free data->saved_message.data. Based on OpenSSL commit:
41cd41c4416f545a18ead37e09e437c75fa07c95 except this version sets a->ptr
to NULL to avoid accidental reuse and handles malloc failing.
ok beck@, input + ok miod@
|
|
|
|
| |
input + ok jsing@
|
|
|
|
|
| |
Minor KNF in a few places too.
input + ok jsing@
|
|
|
|
|
|
|
| |
Also, removed a few useless if null checks.
input from bcook@
input + ok jsing@
|
| |
|
|
|
|
| |
ok jsing@ beck@
|
| |
|
|
|
|
|
|
|
| |
remember to set EACCES in bcrypt_checkpass for hash differences.
the higher level crypt_checkpass function will reset errno to EACCES in
all cases, which is probably the right behavior, but this change gives code
working with the lower level functions the correct errno if they care.
|
| |
|
|
|
|
|
|
| |
patch from Kinichiro Inoguchi, tested on HP-UX 11.31
ok deraadt@
|
|
|
|
| |
what's going on.
|
|
|
|
| |
ok doug@
|
|
|
|
|
| |
guenther suggested using thread time, which actually may improve accuracy
if somebody puts this in a threaded program.
|
|
|
|
|
|
| |
this man page.
ok jsing@
|
|
|
|
|
|
|
|
| |
Instead of asserting, return an error code for I/O errors. This is based
on OpenSSL commit 2521fcd8527008ceb3e4748f95b0ed4e2d70cfef. Added checks
for two calloc()s while I'm here.
ok miod@
|
|
|
|
|
|
| |
also check the return value of an adjacent malloc() call.
ok jsing@
|
|
|
|
|
| |
a compiler warning about shadowing a global declaration.
OK jsing@
|
|
|
|
|
|
|
| |
apart from introducing a bug where the -elapsed option is not listed in
the usage output when it should be.
feedback/ok bcook@ jsing@
|
|
|
|
| |
option.
|
|
|
|
| |
With input from doug@
|
|
|
|
|
| |
because the API design isn't fully settled.
Requested by jsing@ and tedu@.
|
|
|
|
| |
patch from Sunil at Nimmagadda dot net
|
|
|
|
| |
ok deraadt miod
|
|
|
|
|
|
|
| |
afterwards. openssl has a more complicated fix, but it's less intrusive
for now to simply hoist the expensive part (fake key generation) up without
sweating a branch or two.
ok bcook jsing
|
| |
|
| |
|
|
|
|
| |
option type. In this case process the option as per normal.
|
| |
|
|
|
|
|
| |
arbitrary number of arguments. This will allow for more complex option
handling as required by some of the openssl(1) applications.
|
| |
|
|
|
|
| |
(as currently only implemented by some of the openssl(1) applications).
|
| |
|
|
|
|
|
| |
that it has consumed. This allows for the handling of multiple unnamed
arguments, including lists of filenames.
|
| |
|
|
|
|
|
|
|
| |
silently accept multiple unnamed arguments, ignoring all except the last.
This behaviour was already inconsistent between openssl(1) applications;
apply the principal of least surprise. This will also simplify the addition
of upcoming functionality.
|