Merge pull request #1406 from Unity-Technologies/unity-master-tls-fix-missing-pinvokecallback-attribute

ashwinimurt · web-flow · commit e4bfd952a59a · 2021-03-01T15:05:10.000-08:00
Fix missing MonoPInvokeCallback on x509verify_callback
diff --git a/mcs/class/System/Mono.UnityTls/UnityTlsProvider.cs b/mcs/class/System/Mono.UnityTls/UnityTlsProvider.cs
@@ -16,6 +16,7 @@
 #else
 using Mono.Security.Interface;
 #endif
+using Mono.Util;
 
 using size_t = System.IntPtr;
 
@@ -48,6 +49,7 @@ internal override IMonoSslStream CreateSslStreamInternal (
 			return new UnityTlsStream (innerStream, leaveInnerStreamOpen, sslStream, settings, this);
 		}
 
+		[MonoPInvokeCallback (typeof (UnityTls.unitytls_x509verify_callback))]
 		static UnityTls.unitytls_x509verify_result x509verify_callback(void* userData, UnityTls.unitytls_x509_ref cert, UnityTls.unitytls_x509verify_result result, UnityTls.unitytls_errorstate* errorState)
 		{
 			if (userData != null)

Original file line number	Diff line number	Diff line change
`@@ -16,6 +16,7 @@`
`16`	`16`	`#else`
`17`	`17`	`using Mono.Security.Interface;`
`18`	`18`	`#endif`
	`19`	`+using Mono.Util;`
`19`	`20`
`20`	`21`	`using size_t = System.IntPtr;`
`21`	`22`
`@@ -48,6 +49,7 @@ internal override IMonoSslStream CreateSslStreamInternal (`
`48`	`49`	`return new UnityTlsStream (innerStream, leaveInnerStreamOpen, sslStream, settings, this);`
`49`	`50`	`}`
`50`	`51`
	`52`	`+ [MonoPInvokeCallback (typeof (UnityTls.unitytls_x509verify_callback))]`
`51`	`53`	`static UnityTls.unitytls_x509verify_result x509verify_callback(void* userData, UnityTls.unitytls_x509_ref cert, UnityTls.unitytls_x509verify_result result, UnityTls.unitytls_errorstate* errorState)`
`52`	`54`	`{`
`53`	`55`	`if (userData != null)`