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

index f35861de62bb11c33df9e01905ac7ba5f1f4c254..48dcfe290155617ebacfb758ac830e841760a42c 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)
                {