On Sat, Jun 13, 2009, Guenter wrote:

> Now due to a couple of recent changes I see that currently all 3
> branches HEAD, 1_0_0 and 0_9_8 are broken for NetWare, and one of these
> breaks is that compilation for CLIB does no longer work without having
> BN_LLONG defined with 0_9_8 branch:
> 
> d1_pkt.c: In function `dtls1_read_bytes':
> d1_pkt.c:760: incompatible type for argument 3 of `dtls1_buffer_record'
> make[1]: *** [d1_pkt.o] Error 1
> 
> though branches 1_0_0 and HEAD compile fine; I think because the
> prototype is different:
> 
> static int dtls1_buffer_record(SSL *s, record_pqueue *q,
>                               unsigned char *priority);
> 
> while with 0_9_8 we have:
> 
> static int dtls1_buffer_record(SSL *s, record_pqueue *q,
>                               PQ_64BIT priority);
> 
> line 760 / 0_9_8 and line 803 / 1_0_0 and HEAD is:
> 
> dtls1_buffer_record(s, &(s->d1->buffered_app_data), 0);
> 
> maybe the prototype from 1_0_0/HEAD needs backported to 0_9_8 branch?
> 
> Then I would also like to get some clarification about the BN_LLONG
> define, and if its valid to compile without this define for a 32-bit
> platform?
> 

Yes it should be possible to compile without BN_LLONG and this shows the above
code is broken.

Steve.
--
Dr Stephen N. Henson. Email, S/MIME and PGP keys: see homepage
OpenSSL project core developer and freelance consultant.
Homepage: http://www.drh-consultancy.demon.co.uk
______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
Development Mailing List                       [email protected]
Automated List Manager                           [email protected]

Reply via email to