commit 6c05208f9663c85bae3fb76204fd74432e1e5948
author Andres AG <andres.amayagarcia@arm.com> Mon Sep 26 10:09:30 2016 +0100
committer Simon Butcher <simon.butcher@arm.com> Wed Oct 12 19:54:07 2016 +0100
tree d2f9938a5e7233cae85e5859f251e57d5a1650e2
parent d5e33f14df4a4c0bcc075c0e77a647c9a7009c24

Fix documentation for mbedtls_gcm_finish()

Fix implementation and documentation missmatch for the function
arguments to mbedtls_gcm_finish(). Also, removed redundant if condition
that always evaluates to true.

library/gcm.c

diff --git a/library/gcm.c b/library/gcm.c
index aaacf97..f1210c5 100644
--- a/library/gcm.c
+++ b/library/gcm.c
@@ -415,8 +415,7 @@
     if( tag_len > 16 || tag_len < 4 )
         return( MBEDTLS_ERR_GCM_BAD_INPUT );
 
-    if( tag_len != 0 )
-        memcpy( tag, ctx->base_ectr, tag_len );
+    memcpy( tag, ctx->base_ectr, tag_len );
 
     if( orig_len || orig_add_len )
     {