| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
| |
(Thanks to Brent Cook)
OK from jsing@
|
|
|
|
| |
includes) follows this commit message:
|
|
|
|
|
|
|
|
| |
I just spent too long chasing a bug in here and really should have done
this first. Gem of the day... is it an if test or a for loop? No, it is a
super ifloop!
if (!(flags & PKCS7_NOVERIFY)) for (k = 0; k < sk_X509_num(signers); k++) {
|
|
|
|
|
|
|
|
| |
function that ended with:
if (ret & 0x01)
if (ret & V_ASN1_CONSTRUCTED)
}
|
| |
|
|
|
|
| |
Substantially expand the conditional to reduce potential for error.
|
| |
|
|
|
|
|
|
|
| |
(Thanks to Brent Cook)
OK from tedu@
|
|
|
|
|
|
|
|
| |
(Thanks to Brent Cook)
With help from tedu@
OK from tedu@
|
|
|
|
|
|
| |
(Thanks to Brent Cook)
OK from tedu@
|
|
|
|
|
|
| |
are the current range checks. Help from millert and lteo. Please test
now that it is deployed and let us know if any numbers are off..
ok lteo
|
|
|
|
|
|
| |
fields (i.e. the flags field) before using it. This is currently harmless,
but might not be if we end up invoking other BN functions checking for
constant-time processing requirement in the future.
|
|
|
|
| |
portable code path must handle that; with brent cook
|
| |
|
|
|
|
|
|
| |
improve the random stream itself (it doesn't), but to introduce
noise in the arc4random calling pattern. Thanks to matthew@ who
pointed out bias in a previous diff, ok deraadt@ matthew@
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
it to hide memory leaks in the error paths, and fix aforementioned memory
leaks.
ok jsing@ logan@ deraadt@
|
|
|
|
|
| |
mechanism, to aid in portability to other systems as requested.
ok matthew
|
|
|
|
|
| |
cases where errno can be trashed.
ok jsing
|
| |
|
|
|
|
|
| |
use the address, not what it points to (which is always the same)
ok deraadt@
|
|
|
|
| |
ok deraadt@
|
| |
|
|
|
|
|
| |
by getauxval if we have it.
ok deraadt@
|
|
|
|
|
| |
we are running supports it.
from enh@google.com
|
|
|
|
|
|
| |
which we don;t have in base. after some discussion with jca, i've not removed
these references, but tried to make it clearer it's distributed with openssl
and not included in base;
|
| |
|
| |
|
|
|
|
| |
is incorrect code.
|
|
|
|
| |
of CRYPTO_EX_DATA; riding upon the libcrypto major bump.
|
|
|
|
|
| |
should not know anything about. Verified not to be used in ports; riding upon
the recent libcrypto major bump.
|
|
|
|
| |
ChaCha context. Other changes will also ride this crank.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
ensure that we save it and consume it on subsequent writes. Otherwise we
end up discarding part of the key stream and instead generate a new block
at the start of the next write.
This was only an issue for callers that did multiple writes that are not
multiples of 64 bytes - in particular, the ChaCha20Poly1305 usage does not
hit this problem since it performs encryption in a single-shot. For the
same reason, this is also a non-issue when openssl(1) is used to encrypt
with ChaCha.
Issue identified by insane coder; reported to bugs@ by Joseph M. Schwartz.
ok beck@
|
|
|
|
|
|
|
| |
single-shot CRYPTO_chacha_20() interface (the ChaCha interface was already
tested via the EVP regress, but not extensively). The additional ChaCha
tests include single-shot writes, along with partial/single-byte writes
that currently fail due to a bug in the underlying implementation.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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@
|
|
|
|
|
|
| |
appear by itself.
ok beck@ miod@
|
|
|
|
|
|
| |
Thanks to Brenk Cook.
OK from miod@
|
|
|
|
| |
ok miod
|
|
|
|
| |
RANDOM_UUID is an enum member.
|
| |
|
|
|
|
|
|
| |
multiline comments
ok jsing@
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
| |
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@
|
|
|
|
|
| |
BIO_get_port() and fails since the current code believes that "-1" is a
valid port.
|
| |
|