Moin Goetz,
Goetz Babin-Ebell wrote:
..
Index: openssl/crypto/bio/bio.h
============================================================================
$ cvs diff -u -r1.69 -r1.70 bio.h
--- openssl/crypto/bio/bio.h 18 Dec 2005 19:11:36 -0000 1.69
+++ openssl/crypto/bio/bio.h 29 Nov 2006 20:54:55 -0000 1.70
[...]
@@ -246,14 +250,14 @@
#define BIO_cb_pre(a) (!((a)&BIO_CB_RETURN))
#define BIO_cb_post(a) ((a)&BIO_CB_RETURN)
-#define BIO_set_callback(b,cb) ((b)->callback=(cb))
-#define BIO_set_callback_arg(b,arg) ((b)->cb_arg=(char *)(arg))
-#define BIO_get_callback_arg(b) ((b)->cb_arg)
-#define BIO_get_callback(b) ((b)->callback)
-#define BIO_method_name(b) ((b)->method->name)
-#define BIO_method_type(b) ((b)->method->type)
+long (*BIO_get_callback(const BIO *b)) (struct bio_st *,int,const char
*,int, long,long);
+void BIO_set_callback(BIO *b,
+ long (*callback)(struct bio_st *,int,const char *,int, long,long));
^^^^^^^^^^^^
+char *BIO_get_callback_arg(const BIO *b);
^^^^^^
+void BIO_set_callback_arg(BIO *b, char *arg);
^^^^^^^^^
I think these should be void *
I agree that a void pointer would be more appropriate here but IMHO
the type used to set a value should be the same type used in the
structure which is currently a char pointer (which is the same type
as the second parameter of the bio callback ...).
Cheers,
Nils
______________________________________________________________________
OpenSSL Project http://www.openssl.org
Development Mailing List openssl-dev@openssl.org
Automated List Manager [EMAIL PROTECTED]
