Added comment for the frag->reassembly == NULL case as per feedback from Emilia
authorMatt Caswell <matt@openssl.org>
Thu, 24 Jul 2014 22:33:34 +0000 (23:33 +0100)
committerMatt Caswell <matt@openssl.org>
Wed, 6 Aug 2014 19:36:40 +0000 (20:36 +0100)
Reviewed-by: Emilia Käsper <emilia@openssl.org>
ssl/d1_both.c

index b720a54311b4858b3c62991c9bf7e3e172b164fc..d98ae9392f841168d8a7287b289f7cc9f793705a 100644 (file)
@@ -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)
                {