From: Matt Caswell <matt@openssl.org>
Date: Thu, 24 Jul 2014 22:33:34 +0000 (+0100)
Subject: Added comment for the frag->reassembly == NULL case as per feedback from Emilia
X-Git-Tag: master-pre-reformat~498
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=11e7982a7cc6cbbcbb13046ceed37639c26a0bc1;p=openssl

Added comment for the frag->reassembly == NULL case as per feedback from Emilia

Reviewed-by: Emilia Käsper <emilia@openssl.org>
---

diff --git a/ssl/d1_both.c b/ssl/d1_both.c
index b720a54311..d98ae9392f 100644
--- a/ssl/d1_both.c
+++ b/ssl/d1_both.c
@@ -644,7 +644,8 @@ dtls1_reassemble_fragment(SSL *s, struct hm_header_st* msg_hdr, int *ok)
 
 
 	/* If message is already reassembled, this must be a
-	 * retransmit and can be dropped.
+	 * retransmit and can be dropped. In this case item != NULL and so frag
+	 * does not need to be freed.
 	 */
 	if (frag->reassembly == NULL)
 		{