This suggested patch is not conform to ANSI C. The declaration of  
variables always has to be done at the beginning of their scope:


--- ssl/d1_both.c       2009-04-19 20:03:11.000000000 +0200
+++ ssl/d1_both.c       2009-05-12 09:23:30.000000000 +0200
@@ -519,6 +519,8 @@

        if ( s->d1->handshake_read_seq == frag->msg_header.seq)
                {
+               unsigned long frag_len = frag->msg_header.frag_len;
+
                pqueue_pop(s->d1->buffered_messages);

                al=dtls1_preprocess_fragment(s,&frag->msg_header,max);
@@ -536,7 +538,7 @@
                if (al==0)
                        {
                        *ok = 1;
-                       return frag->msg_header.frag_len;
+                       return frag_len;
                        }

                ssl3_send_alert(s,SSL3_AL_FATAL,al);






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

Reply via email to