commit 997e85c049c122357387e4c077d1045b8142e819
author Janos Follath <janos.follath@arm.com> Tue May 29 11:33:45 2018 +0100
committer Janos Follath <janos.follath@arm.com> Tue May 29 11:59:22 2018 +0100
tree fe0cd571692671571f1dc3be2d68970441614a23
parent 6b4bd3dcc4caa2d555eb087e97e3692e7ce59bfe

CCM*: Remove nested if

library/ccm.c

diff --git a/library/ccm.c b/library/ccm.c
index b195a71..32f8bb0 100644
--- a/library/ccm.c
+++ b/library/ccm.c
@@ -152,14 +152,12 @@
      * Check length requirements: SP800-38C A.1
      * Additional requirement: a < 2^16 - 2^8 to simplify the code.
      * 'length' checked later (when writing it to the first block)
+     *
+     * Also, loosen the requirements to enable support for CCM* (IEEE 802.15.4).
      */
-    if( tag_len < 4 || tag_len > 16 || tag_len % 2 != 0 )
+    if( tag_len == 2 || tag_len > 16 || tag_len % 2 != 0 )
     {
-        /*
-         * Loosen the requirements to enable support for CCM* (IEEE 802.15.4)
-         */
-        if( tag_len != 0 )
-            return( MBEDTLS_ERR_CCM_BAD_INPUT );
+        return( MBEDTLS_ERR_CCM_BAD_INPUT );
     }
 
     /* Also implies q is within bounds */