commit 0eb5a2df4a3028e7283189c2be2849493106a18b
author David Benjamin <davidben@chromium.org> Fri Jul 25 02:40:43 2014 -0400
committer Adam Langley <agl@google.com> Sat Jul 26 02:56:37 2014 +0000
tree 5088f79c20e4e5f957282cefa27d5d401b03408b
parent 0aa0767340baf925bda4804882aab0cb974b2d26

Fix tls1_get_curvelist default curves list size.

The length is the number of elements now, not the size in bytes. Caught by
ASan.

Change-Id: I4c5ccee61711e8d2e272b9bacd292dbff04b5133
Reviewed-on: https://boringssl-review.googlesource.com/1336
Reviewed-by: Adam Langley <agl@google.com>

ssl/t1_lib.c

diff --git a/ssl/t1_lib.c b/ssl/t1_lib.c
index 66add63..5bc0ae9 100644
--- a/ssl/t1_lib.c
+++ b/ssl/t1_lib.c
@@ -480,7 +480,7 @@
 		{
 	case SSL_CERT_FLAG_SUITEB_128_LOS:
 		*out_curve_ids = suiteb_curves;
-		*out_curve_ids_len = sizeof(suiteb_curves);
+		*out_curve_ids_len = sizeof(suiteb_curves) / sizeof(suiteb_curves[0]);
 		break;
 
 	case SSL_CERT_FLAG_SUITEB_128_LOS_ONLY:
@@ -499,7 +499,7 @@
 	if (!*out_curve_ids)
 		{
 		*out_curve_ids = eccurves_default;
-		*out_curve_ids_len = sizeof(eccurves_default);
+		*out_curve_ids_len = sizeof(eccurves_default) / sizeof(eccurves_default[0]);
 		}
 	}