Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
PR: 1923
Submitted by: Daniel Mentz <daniel.m@sent.com>, Robin Seggelmann <seggelmann@fh-muenster.de>
Approved by: steve@openssl.org

Don't access freed data structure.
  • Loading branch information
snhenson committed May 13, 2009
1 parent 756d207 commit 561cbe5
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion ssl/d1_both.c
Expand Up @@ -519,6 +519,7 @@ dtls1_retrieve_buffered_fragment(SSL *s, long max, int *ok)

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);
Expand All @@ -536,7 +537,7 @@ dtls1_retrieve_buffered_fragment(SSL *s, long max, int *ok)
if (al==0)
{
*ok = 1;
return frag->msg_header.frag_len;
return frag_len;
}

ssl3_send_alert(s,SSL3_AL_FATAL,al);
Expand Down

0 comments on commit 561cbe5

Please sign in to comment.