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:27:51 +0000 (20:27 +0100)
Reviewed-by: Emilia Käsper <emilia@openssl.org>
ssl/d1_both.c

index cdb83b6558be61b56ae391f2f22fa85d73e81500..6559dfcd01b4f4d59169593b88e29238bfcea2a1 100644 (file)
@@ -639,7 +639,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)
                {