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

index b808f044964df6a7f59f14996f5ecf86e6370736..9bc416c1c10355832572c849203b5660e177b4e5 100644 (file)
@@ -633,7 +633,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)
                {