Fixes to OCSP print code.

Don't try to print request certificates if signature is not present.

Remove unnecessary test for certificates being NULL.

Fix typos in printed output.

Tidy up output.

Fix for typo in OCSP_SERVICELOC ASN1 template.

Also give a bit more info in CHANGES about the ASN1 revision.

fix indentation

Don't hold CRYPTO_LOCK_RSA during time-consuming operations.

Add a comment (intended change)

Import s2_pkt.c wbuf fixes from OpenSSL_0_9_6-stable branch.

Comment correction.

Obtain lock CRYPTO_LOCK_RSA before creating BN_MONT_CTX
structures and setting rsa->_method_mod_{n,p,q}.

Submitted by: "Reddie, Steven" <Steven.Reddie@ca.com>

undo previous change: '-prexit' is already available in current versions of s_client

Redo OCSP response printing. Remove duplicate or
obsolete code. Delete some redundant files.

When mentioning features that don't exist in current releases of
OpenSSL (such as the new undocumented '-prexit' option to s_client),
the FAQ should point out that they don't: The FAQ is not just part
of the release, it's current version is also published on the web.

Simplify preprocessor statements.

Add OCSP service locator extension.

Do not poll DEVRANDOM if weäre building without an file pointer API.
Spotted by "David Schwartz" <davids@webmaster.com>.

Make sure each FAQ item has an index entry.

Add support for the noCheck OCSP extension. This is
just a NULL and appears in a certificate.

Make mkdef.pl parse some ASN1 IMPLEMENT macros.

Initial support for variables in DEF files.

Add a couple of FAQs.

"Andrew W. Gray" <agray@iconsinc.com> says /GD is no longer a valid
compiler switch.

If CONF_get_string returns NULL and we want to tolerate this
(e.g., use a default), we have to call ERR_clear_error().

Locking issues.

Implement some standard OCSP extensions in the v3 code. These
are all raw print only extensions at present.

The C version of bn_sub_part_words is needed not only
in NO_ASM configurations

Very few in the "README" is up-to-date

Add OCSP nonce extension to supported extensions.

This is a little unusual because it can contain no
structure i.e. the extension OCTET STRING content
octets do not contain a DER encoded structure.

Make a note of the new engine.

This is an engine contributed by Broadcom - it is meant to support the
BCM5805 and BCM5820 units. So far I've merely taken a skim over the code
and changed a few things from their original contributed source
(de-shadowing variables, removing variables from the header, and
re-constifying some functions to remove warnings). If this gives
compilation problems on any system, please let me know. We will hopefully
know for sure whether this actually functions on a system with the relevant
hardware in a day or two.  :-)

New function X509V3_extensions_print() this removes extension duplication
from the print routines.

Reorganisation of OCSP code: initial print routines in ocsp_prn.c. Doesn't
work fully because OCSP extensions aren't reimplemented yet.

Implement some ASN1 functions needed to compile OCSP code.

typo

The BN_mul bug test apparently is no longer needed

First step towards SSL_peek fix.

Update.

New function X509_signature_print() to remove some duplicate
code from certificate, CRL and request printing routines.

Change the PKCS7 structure to use SEQUENCE OF for the
authenticated attributes: this is used to retain the
original encoding and not break signatures.

Support for a SET OF which reorders the STACK when
encoding a structure. This will be used with the
PKCS7 code.

c&p error spotted by Martin Forssen

Replace the old style OCSP ASN1 module.

Constification of the data of a hash table.  This means the callback
functions need to be constified, and therefore meant a number of easy
changes a little everywhere.

Now, if someone could explain to me why OBJ_dup() cheats...

Problem: bn_mul_normal() misbehaves if the size of b is 0.
Solution: multiply a with 0, putting the result in r, and return.

In bn_mul_recursive(), make sure the comba routines are only called
when both a and b are of the exact right size.  This may change to
something better later.

When using -pedantic, it's a good thing to define PEDANTIC as well.
Also, define a second debugging configuration without assembler.

Stop extension creation code core dumping.

This was caused by no initialising the buffer
to NULL when using the auto allocating version
if i2d.

Rewrite the extension code to use an ASN1_ITEM structure
for its ASN1 operations as well as the old style function
pointers (i2d, d2i, new, free). Change standard extensions
to support this.

Fix a warning in BN_mul(), bn_mul.c about uninitialised 'j'.

Add description of SSL_[CTX_]_check_private_key().

SSL_new() may potentially add a certfificate.  Therefore, wen
duplicating the certificate that is in the original SSL, remove the
one that SSL_new() provided, if any.
Spotted by: Mike Zeoli <zeoli@roguewave.com>

Change/add comments

Stop on bntest error.

*** empty log message ***

Make TYPE_RSA the default type instead of just setting it when -new is
given.  That also allows the arguments to come in any order (-new
last, for example).

Use continuation lines in test/bctest as far as it is possible
to dermine what the expression should look like.
Apparently CVS does not like lines longer than about 2^10 characters.

Don't throw away bctest's error messages.

Test for SCO bc bug

Next step in tidying up the LHASH code.

DECLARE/IMPLEMENT macros now exist to create type (and prototype) safe
wrapper functions that avoid the use of function pointer casting yet retain
type-safety for type-specific callbacks. However, most of the usage within
OpenSSL itself doesn't really require the extra function because the hash
and compare callbacks are internal functions declared only for use by the
hash table. So this change catches all those cases and reimplements the
functions using the base-level LHASH prototypes and does per-variable
casting inside those functions to convert to the appropriate item type.

The exception so far is in ssl_lib.c where the hash and compare callbacks
are not static - they're exposed in ssl.h so their prototypes should not be
changed. In this last case, the IMPLEMENT_LHASH_*** macros have been left
intact.

bn_part_sub_word prototype.

Merge from the ASN1 branch of new ASN1 code
to main trunk.

Lets see if the makes it to openssl-cvs :-)

Add manual pages for certficate/key loading and friends.

Placeholder for SCO bc bug detection

Fix some things that look like bugs.

One problem that looked like a problem in bn_recp.c at first turned
out to be a BN_mul bug.  An example is given in bn_recp.c; finding
the bug responsible for this is left as an exercise.

TEST_MUL and TEST_SQR added.

Sign-related fixes (and tests).

BN_mod_exp_mont does not work properly yet if modulus m
is negative (we want computations to be carried out
modulo |m|).

Move 'q->neg = 0' to those places where it is needed
(just in cases someone uses a negative modulus)

BN_mod_sqrt documentation/comment

Don't check for bc at all. We can now run a meaningful test even if
it is missing.

remove a comment that shouldn't have been there any more

Faster BN_mod_sqrt algorithm for p == 5 (8).

Printing "verify ..." should not be counted as a test for the
"xxx tests passed" message.

During the self test, we only want to know what bctest says on
stderr...

Have the self test use bctest to check that bc is sane.

Use bc's "print" feature whenever it is available,
not just on certain platforms.

Add a comment.

Workaround for broken (or missing) bc.

remove useless instruction

Intel assembler version for bn_sub_part_words().  I haven't got
reliable timings yet, please try it out!

another fix for the debug print

test_mod_mul is useful, let's run it more often.

move constants for debug functions to end of file

Update the internal docs.

Typo and additional information about cert-chain building.

Add EXAMPLES for SSL_CIPHER_description() output.

Discuss http://www.shoup.net/papers/oaep.ps.Z

On Windows, Rainbow uses _stdcall convention under Windows.
Spotted by plin <plin@rainbow.com>

push the flags too

looks like it works now

save registers in the debug output code (return value is overwritten too)

more of the same: add printf() for perlasm.

minor modification to the previous change

PERLASM - the wierdest programming language since Intercal.

add debug output facilities.

Use assert as in the rest of the BN library.

Improve formatting.

It's completely unnecessary to add a compression algorithm that is
really undefined.
Spotted by Jeffrey Altman <jaltman@columbia.edu>

Remove the last bn_wexpand()s that made us break constness.  Of
course, that means we need to handle the cases where the two arrays to
bn_mul_recursive() and bn_mul_part_recursive() differ in size.

I haven't yet changed the comments that describe bn_mul_recursive()
and bn_mul_part_recursive().  I want this to be tested by more people
before I consider this change final.  Please test away!

Change error message to "bignum too long"

Sync up with a minor change in lhash.h

ANSI C doesn't allow trailing semi-colons after a function's closing brace
so these macros probably shouldn't be used like that at all. So, this
change removes the misleading comment and also adds an implicit trailing
semi-colon to the DECLARE macros so they too don't require one.

Update the documentation to the current state of the LHASH changes. There
will probably be more when the lh_doall[_arg] callbacks are similarly
tidied up, but this 'pod' should now be current.

Make a note of the LHASH changes.

Make the remaining LHASH macro changes. This should leave no remaining
cases of function pointer casting in lh_new() calls - and leave only the
lh_doall and lh_doall_arg cases to be finished.

Fix warnings.

include <limits.h>

Don't allow BIGNUMs to become so large that computations with dmax
might overflow.

Use the new LHASH macros to declare type-safe wrapper functions that can
be used as the hash/compare callbacks without function pointer casting.

For now, this is just happening in the apps/ directory whilst a few people
check the approach. The rest of the library will be moved across to the
same idea if there's no problems with this.

Next step in tidying up the LHASH code. This commit defines DECLARE and
IMPLEMENT macros for defining wrapper functions for "hash" and "cmp" callbacks
that are specific to the underlying item type in a hash-table. This prevents
function pointer casting altogether, and also provides some type-safety
because the macro does per-variable casting from the (void *) type used in
LHASH itself to the type declared in the macro - and if that doesn't match the
prototype expected by the "hash" or "cmp" function then a compiler error will
result.

NB: IMPLEMENT macros are not required unless predeclared forms are required
(either in a header file, or further up in a C file than the implementation
needs to be). The DECLARE macros must occur after the type-specific hash/cmp
callbacks are declared. Also, the IMPLEMENT and DECLARE macros are such that
they can be prefixed with "static" if desired and a trailing semi-colon should
be appended (making it look more like a regular declaration and easier on
auto-formatting text-editors too).

Now that these macros are defined, I will next be commiting changes to a
number of places in the library where the casting was doing bad things. After
that, the final step will be to make the analogous changes for the lh_doall
and lh_doall_arg functions (more specifically, their callback parameters).