[Coop] Convert ves_icall_string_new_wrapper_impl. (mono/mono#15901)
authorJay Krell <jaykrell@microsoft.com>
Tue, 30 Jul 2019 21:37:59 +0000 (14:37 -0700)
committerAlexander Köplinger <alex.koeplinger@outlook.com>
Tue, 30 Jul 2019 21:37:58 +0000 (23:37 +0200)
Static analysis prefers not even these short lived raw pointers.
Extracted from mono/mono#15900.

Commit migrated from https://github.com/mono/mono/commit/ca8f155983139beaa306b77cabc2dde365021bd0

src/mono/mono/metadata/marshal.c

index 70192de..b9f4d4a 100644 (file)
@@ -186,12 +186,7 @@ ves_icall_mono_string_to_utf8_impl (MonoStringHandle str, MonoError *error)
 MonoStringHandle
 ves_icall_string_new_wrapper_impl (const char *text, MonoError *error)
 {
-       if (text) {
-               MonoString *s = mono_string_new_checked (mono_domain_get (), text, error);
-               return_val_if_nok (error, NULL_HANDLE_STRING);
-               return MONO_HANDLE_NEW (MonoString, s);
-       }
-       return NULL_HANDLE_STRING;
+       return text ? mono_string_new_handle (mono_domain_get (), text, error) : NULL_HANDLE_STRING;
 }
 
 void