api_string_export_wchar allocated one too less

Reini Urban · Reini Urban · commit df12f34e9465 · 2012-05-04T09:56:44.000-05:00
clang -faddress-sanitizer detected this heap-buffer-overflow in
t/src/embed/strings.t, allocated 173 bytes only, writing to cstr[173].
diff --git a/src/embed/strings.c b/src/embed/strings.c
@@ -102,7 +102,7 @@ Parrot_api_string_export_wchar(ARGIN(Parrot_PMC interp_pmc), ARGIN(Parrot_String
         cstr = Parrot_str_to_cstring(interp, string);
         len = strlen(cstr);
 
-        wstrout = (wchar_t *) malloc(sizeof (wchar_t) * len + 1);
+        wstrout = (wchar_t *) malloc(sizeof (wchar_t) * (len + 1));
         mbstowcs(wstrout, cstr, len);
         wstrout[len] = L'\0';
 
