<feed xmlns='http://www.w3.org/2005/Atom'>
<title>openssl/crypto/stack/stack.c, branch rsaref</title>
<subtitle>TLS/SSL and crypto library.</subtitle>
<id>http://git.bktus.com/gpgfrontend/openssl/atom?h=rsaref</id>
<link rel='self' href='http://git.bktus.com/gpgfrontend/openssl/atom?h=rsaref'/>
<link rel='alternate' type='text/html' href='http://git.bktus.com/gpgfrontend/openssl/'/>
<updated>2000-11-07T13:49:47Z</updated>
<entry>
<title>This commit was manufactured by cvs2svn to create tag 'rsaref'.</title>
<updated>2000-11-07T13:49:47Z</updated>
<author>
<name>cvs2svn</name>
<email>cvs2svn</email>
</author>
<published>2000-11-07T13:49:47Z</published>
<link rel='alternate' type='text/html' href='http://git.bktus.com/gpgfrontend/openssl/commit/?id=9d98df23f38afa21e159d910335b49fb196206e8'/>
<id>urn:sha1:9d98df23f38afa21e159d910335b49fb196206e8</id>
<content type='text'>
</content>
</entry>
<entry>
<title>Some platforms define NULL as ((void *)0).  Unfortunately, a void*</title>
<updated>2000-09-17T14:46:09Z</updated>
<author>
<name>Richard Levitte</name>
<email>levitte@openssl.org</email>
</author>
<published>2000-09-17T14:46:09Z</published>
<link rel='alternate' type='text/html' href='http://git.bktus.com/gpgfrontend/openssl/commit/?id=60dae9985d0120c62552776919ca474266649908'/>
<id>urn:sha1:60dae9985d0120c62552776919ca474266649908</id>
<content type='text'>
can't be used as a function pointer according the the standards.  Use
a 0 instead and there will be no trouble.
</content>
</entry>
<entry>
<title>Safe stack reorganisation in terms of function casts.</title>
<updated>2000-06-16T23:29:26Z</updated>
<author>
<name>Dr. Stephen Henson</name>
<email>steve@openssl.org</email>
</author>
<published>2000-06-16T23:29:26Z</published>
<link rel='alternate' type='text/html' href='http://git.bktus.com/gpgfrontend/openssl/commit/?id=3aceb94b9e6ac4722b96d4fb1cf7e1a4358941e0'/>
<id>urn:sha1:3aceb94b9e6ac4722b96d4fb1cf7e1a4358941e0</id>
<content type='text'>
After some messing around this seems to work but needs
a few more tests. Working out the syntax for sk_set_cmp_func()
(cast it to a function that itself returns a function pointer)
was painful :-(

Needs some testing to see what other compilers think of this
syntax.

Also needs similar stuff for ASN1_SET_OF etc etc.
</content>
</entry>
<entry>
<title>Harmonize indentation.</title>
<updated>2000-06-08T22:41:03Z</updated>
<author>
<name>Bodo Möller</name>
<email>bodo@openssl.org</email>
</author>
<published>2000-06-08T22:41:03Z</published>
<link rel='alternate' type='text/html' href='http://git.bktus.com/gpgfrontend/openssl/commit/?id=3dd985ee2ab05b4eda53b67395d85269acad74bd'/>
<id>urn:sha1:3dd985ee2ab05b4eda53b67395d85269acad74bd</id>
<content type='text'>
</content>
</entry>
<entry>
<title>There have been a number of complaints from a number of sources that names</title>
<updated>2000-06-01T22:19:21Z</updated>
<author>
<name>Richard Levitte</name>
<email>levitte@openssl.org</email>
</author>
<published>2000-06-01T22:19:21Z</published>
<link rel='alternate' type='text/html' href='http://git.bktus.com/gpgfrontend/openssl/commit/?id=26a3a48d65c7464b400ec1de439994d7f0d25fed'/>
<id>urn:sha1:26a3a48d65c7464b400ec1de439994d7f0d25fed</id>
<content type='text'>
like Malloc, Realloc and especially Free conflict with already existing names
on some operating systems or other packages.  That is reason enough to change
the names of the OpenSSL memory allocation macros to something that has a
better chance of being unique, like prepending them with OPENSSL_.

This change includes all the name changes needed throughout all C files.
</content>
</entry>
<entry>
<title>This is the first of two commits (didn't want to dump them all into the</title>
<updated>2000-06-01T02:15:40Z</updated>
<author>
<name>Geoff Thorpe</name>
<email>geoff@openssl.org</email>
</author>
<published>2000-06-01T02:15:40Z</published>
<link rel='alternate' type='text/html' href='http://git.bktus.com/gpgfrontend/openssl/commit/?id=7bb7043580900b8f06cb418b46004b755ff0fc96'/>
<id>urn:sha1:7bb7043580900b8f06cb418b46004b755ff0fc96</id>
<content type='text'>
same one). However, the first will temporarily break things until the
second comes through. :-)

The safestack.h handling was mapping compare callbacks that externally
are of the type (int (*)(type **,type **)) into the underlying callback
type used by stack.[ch], which is (int (*)(void *,void *)). After some
degree of digging, it appears that the callback type in the underlying
stack code should use double pointers too - when the compare operations
are invoked (from sk_find and sk_sort), they are being used by bsearch
and qsort to compare two pointers to pointers. This change corrects the
prototyping (by only casting to the (void*,void*) form at the moment
it is needed by bsearch and qsort) and makes the mapping in safestack.h
more transparent. It also changes from "void*" to "char*" to stay in
keeping with stack.[ch]'s assumed base type of "char".

Also - the "const" situation was that safestack.h was throwing away
"const"s, and to compound the problem - a close examination of stack.c
showed that (const char **) is not really achieving what it is supposed
to when the callback is being invoked, what is needed is
(const char * const *). So the underlying stack.[ch] and the mapping
macros in safestack.h have all been altered to correct this.

What will follow are the vast quantities of "const" corrections required
in stack-dependant code that was being let "slip" through when
safestack.h was discarding "const"s. These now all come up as compiler
warnings.
</content>
</entry>
<entry>
<title>sk_value was also suffering from de-const-ification.</title>
<updated>2000-05-31T17:35:11Z</updated>
<author>
<name>Geoff Thorpe</name>
<email>geoff@openssl.org</email>
</author>
<published>2000-05-31T17:35:11Z</published>
<link rel='alternate' type='text/html' href='http://git.bktus.com/gpgfrontend/openssl/commit/?id=e20d7d71677c6d56f9e486148a0d33cb639765eb'/>
<id>urn:sha1:e20d7d71677c6d56f9e486148a0d33cb639765eb</id>
<content type='text'>
Also, add in a couple of missing declarations in pkcs7 code.
</content>
</entry>
<entry>
<title>All the little functions created by the IMPLEMENT_STACK_OF() macro will</title>
<updated>2000-05-31T15:28:01Z</updated>
<author>
<name>Geoff Thorpe</name>
<email>geoff@openssl.org</email>
</author>
<published>2000-05-31T15:28:01Z</published>
<link rel='alternate' type='text/html' href='http://git.bktus.com/gpgfrontend/openssl/commit/?id=01296a6de0f968d741d871efda8ab4ed5420442d'/>
<id>urn:sha1:01296a6de0f968d741d871efda8ab4ed5420442d</id>
<content type='text'>
cast their type-specific STACK into a real STACK and call the underlying
sk_*** function. The problem is that if the STACK_OF(..) parameter being
passed in has a "const *" qualifier, it is discarded by the cast.

I'm currently implementing a fix for this but in the mean-time, this is
one case I noticed (a few type-specific sk_**_num() functions pass in
const type-specific stacks). If there are other errors in the code where
consts are being discarded, we will similarly not notice them. yuck.
</content>
</entry>
<entry>
<title>Get rid of more non-ANSI declarations.</title>
<updated>2000-05-15T22:54:43Z</updated>
<author>
<name>Ulf Möller</name>
<email>ulf@openssl.org</email>
</author>
<published>2000-05-15T22:54:43Z</published>
<link rel='alternate' type='text/html' href='http://git.bktus.com/gpgfrontend/openssl/commit/?id=0e1c06128adbfd2d88dc304db2262140bad045fd'/>
<id>urn:sha1:0e1c06128adbfd2d88dc304db2262140bad045fd</id>
<content type='text'>
</content>
</entry>
<entry>
<title>ispell (and minor modifications)</title>
<updated>2000-02-03T23:23:24Z</updated>
<author>
<name>Ulf Möller</name>
<email>ulf@openssl.org</email>
</author>
<published>2000-02-03T23:23:24Z</published>
<link rel='alternate' type='text/html' href='http://git.bktus.com/gpgfrontend/openssl/commit/?id=657e60fa00ddde3618600d6306be913214d30457'/>
<id>urn:sha1:657e60fa00ddde3618600d6306be913214d30457</id>
<content type='text'>
</content>
</entry>
</feed>
