Hi there,

Um, actually this isn't right ...

On Tue, 20 Feb 2001 [EMAIL PROTECTED] wrote:

[snip]

>   Modified:    crypto/objects o_names.c

[snip]

>   @@ -67,8 +67,8 @@
>               {
>               MemCheck_off();
>               name_funcs = OPENSSL_malloc(sizeof(NAME_FUNCS));
>   -           name_funcs->hash_func = lh_strhash;
>   -           name_funcs->cmp_func = strcmp;
>   +           name_funcs->hash_func = (LHASH_HASH_FN_TYPE)lh_strhash;
>   +           name_funcs->cmp_func = (LHASH_COMP_FN_TYPE)strcmp;
>               name_funcs->free_func = 0; /* NULL is often declared to
>                                               * ((void *)0), which according
>                                               * to Compaq C is not really

the cmp_func and hash_func entries of the NAME_FUNCS structure are the same
prototypes as strcmp and lh_strhash. So these casts are currently unecessary and
have in fact introduce an "incompatible" pointer warning. However, what's
actually getting used in the lhash code *should* be of the
LHASH_[HASH|COMP]_FN_TYPE types so I'll have to figure out where that's at.
Working on it ...

Cheers,
Geoff


______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
Development Mailing List                       [EMAIL PROTECTED]
Automated List Manager                           [EMAIL PROTECTED]

Reply via email to