Merge pull request #3591 from directhex/mono_libdir_fallback

Handle relocatable libMonoPosixHelper.so when --libdir= isn't lib/

[System]: Move MonoBtlsProvider.IsSupported() -> MonoTlsProviderFactory.IsBtlsSupported() (#3760)

* [System]: Move MonoBtlsProvider.IsSupported() -> MonoTlsProviderFactory.IsBtlsSupported().

Move the intern-call which is used to determine whether we're using BTLS out of MonoBtlsProvider
to allow the managed linker to eliminate the Mono.Btls.* classes when it's not supported.

* Fix Android build.

(cherry picked from commit 90699022b6fed7fd273ba3379a2e3696bbd8956f)

Merge pull request #3762 from akoeplinger/fix-ftp-ipv6

[System] Add IPv6 support to FtpWebRequest

Merge pull request #3738 from lateralusX/jlorenss/win-api-family-support-libmonoruntime

Build libmonoruntime under none desktop Windows API family.

Build libmonoruntime under none desktop Windows API family.

Initial work to build libmonoruntime under none desktop Windows API families.

Merge pull request #3755 from kumpera/async-reader-hardering-2

[System] Another round of hardening around disposal of Process while doing async reads.

[System] Add tests for IPv6-enabled FtpWebRequest

[System] Add IPv6 support to FtpWebRequest

When connecting to an FTP server we weren't handling the RFC2428
extension to the FTP protocol which allows connections over IPv6
for the data channel.

We'd try to connect via IPv4 and the server would return an error
saying that this protocol is not supported.

The fix is to add handling for the protocol extension, namely
to use EPSV (for passive FTP) or EPRT (for active FTP) when opening
the data channel and send/parse the IPv6 addresses.

Fixes https://bugzilla.xamarin.com/show_bug.cgi?id=44025

Note: we're now using the IP address of the the control channel we connected to for the
data channel instead of the address that is returned by the server in the FTP response message.
The reason is that for EPSV there's no IP returned by the server, and it should be
more reliable for normal PASV as well since the server might not know the
correct external address. This is also what MS does in referencesource.

[mcs] Handle interpolated strings with escaped quotes. Fixes #45286

[System] Make sure FtpWebRequest processing happens in a background thread

We don't want it to block app shutdown.

Bump ikvm for mcs fixes

Merge pull request #3754 from marek-safar/rs-sockets

[System] More Socket types from referencesource

[Mono.Data.Sqlite] Regression introduced by 04162ffbbdfed88ba298341f0384a35949b3c0f9 which changed _datetimeFormats order. Fixes #40603

[Mono.Data.Sqlite] Enable more tests

Merge pull request #3749 from BrzVlad/fix-mips-fix

[mips] Fix mips

Merge pull request #3753 from henricm/enable-windowsbase-tests-on-windows

Enable WinowsBase tests on Windows

[System] Warnings cleanup

[System] More Socket types from referencesource

Merge pull request #3750 from marek-safar/socket

[System] More Socket pieces from referencesource

[mips] Disable div with mul on 32bit mips

[llvm] Avoid generating invalid IL for valuetypes+vphi+volatile variables in gsharedvt methods. Fixes part of #45270.

[corlib] Fix UdpClient tests on watchos

They were reenabled in 1fa85ad4afb4139e607876121a080a3be122a7a9,
but UdpClient throws PlatformNotSupportedException on watchos.

Merge pull request #3715 from kumpera/fix-44707

[corlib] Don't load the machine config file if it's missing. Fixes #44707

[System] Another round of hardening around disposal of Process while doing async reads.

This round ensures stream and decoder are protected from dispose and that we properly
bail out to the EOS case.

[cert-sync]: Change back to using the '--btls' argument.

Without arguments, 'cert-sync' now operates on the old certificate store
again, like it was before.  To test BTLS, you need to explicitly run it
with '--btls'.

(cherry picked from commit 46379bda6d13e6e216ca3fc35654eb910c891c41)

Merge pull request #3745 from akoeplinger/fix-bug44937

[io-layer] Correctly detect a PE32+ assembly as managed when starting process

[System.ServiceModel] Split unreliable tests

[System] Set more error details on failed https proxy connect. Fixes #45108

[mono] Fixes hardcoded stack limit in EnsureSufficientExecutionStack check

[System] More Socket pieces from referencesource

Enable WinowsBase tests on Windows

Merge pull request #3662 from henricm/fix-windowsbase-on-win

WindowsBase zlib, ZipSharp and Package fixes for Windows

Making sure stream is closed when exception is thrown

Stream will never be closed if we throw exception in Open
which leads to file being locked in Windows and that we
leak file handles.

Setting Culture to en-us for ToStringTest

Make ZipSharp work on Windows

This patch adds the zlib functions needed by ZipSharp to the VS
MonoPosixHelper.def export file.

This patch also changes the ZipSharp code to use different code paths
depending on the size of the C long type on the underlying platform. On
gcc/clang the C long type follows the bitness of the targeted architecture,
it's 32-bit on 32-bit systems and 64-bit on 64-bit systems. With the VS
compiler however, the C long type is always 32-bit regardless of the target
architecture. zlib and minizip uses C long in a number of different function
signatures and structs.

Merge pull request #3746 from ntherning/fix-SocketResponder-on-windows

Fix SocketResponder on Windows

Merge pull request #3747 from marek-safar/updclient

UpdClient from referencesource

[mips] Add missing define

[mips] Don't assert when decomposing longs

There is no point in asserting if we encounter an unhandled long op. These ops may still get removed, if emulated later on, or maybe from other passes. Asserting happens as a last resort when emitting the native code for the method.

[corlib] Add missing constructors to RNGCryptoServiceProvider on monotouch

Those were missed in cb46f78eef5263ce24ba12c583e4c72ac64734c1 because there's a separate
implementation for monotouch in an unrelated file.

[System]: Fix a typo in X500DistinguishedName.AreEqual().

(cherry picked from commit 1ac809aad7a2ca1d69c6d26e7ac18b36d5c90610)

[btls] Use UTC for the Unix epoch in the managed BTLS code (#3702)

The epoch is specified as being in UTC, but the DateTime(int, int, int) constructor
creates a DateTime with an 'Unspecified' timezone, which could lead to problems later on.

[BTLS]: Correctly handle certificates with negative serial numbers. (#3719)

In mono_btls_x509_get_serial_number(), first call i2c_ASN1_INTEGER()
which takes care of padding and computing the 2s complement for negative
numbers, then reverse the buffer.
(cherry picked from commit 1a55680bfaffa3fa00881889133e5313a6cd9ab4)

[System.Core] Remove unused files

Fix SocketResponder on Windows

There seems to be a race in some tests in the System test suite that use
SocketResponder on Windows. Some tests fail with an exception that the socket
has been closed before they get a chance to see the response sent by the
SocketResponder. The same happens occasionally when using SocketResponder on
.NET. This patch changes the order of the Socket.Shutdown() calls and adds a
Socket.Receive() before Shutdown() is called to give the sender a chance to
actually send any data before the responder shuts down and closes its side of
the connection.

[System] Enable more tests

[System] UdpClient from referencesource

[System.Data] Use same socket family for data receiver as it has its client. Fixes #44624

Merge pull request #3740 from Unity-Technologies/gc-options-command-line

GC debug and params command line options

[io-layer] Correctly detect a PE32+ assembly as managed when starting process

We weren't checking whether a PE file is in PE32 or PE32+ format,
causing us to use the PE32 offsets on the newer format as well.

This means that using Process.Start() on a managed assembly didn't
always work when a PE32+ file was in play. Such a file is created
when targeting x64 e.g. via the -platform:x64 csc.exe/mcs option.

The reason why we probably didn't notice until now is that in an
assembly produced by mcs there happens to be some data at the PE32
CLR header offset even in a PE32+ file, causing the check to "work".

This doesn't apply to csc.exe/roslyn though and so we couldn't
execute those assemblies via Process.Start() even though they're
perfectly managed. Since the .NET Core project.json toolchain
produces x64-targeted assemblies by default more people ran into
the issue trying to run those on Mono: https://github.com/cake-build/cake/issues/1247

Fixes https://bugzilla.xamarin.com/show_bug.cgi?id=44937

PE/COFF spec at https://msdn.microsoft.com/en-us/library/windows/desktop/ms680547(v=vs.85).aspx

Merge pull request #3739 from akoeplinger/add-more-netstandard-api

[bcl] Expose a few more APIs on mobile to match with netstandard

[bcl] Expose a few more APIs on mobile to match with netstandard

See https://github.com/dotnet/standard/issues/47.

Merge pull request #3653 from ntherning/alertable-socket-wait

Make blocking socket calls on Windows interruptible

Merge pull request #3744 from ntherning/enable-system-security-tests-in-pr-builds-on-windows

Enable System.Security tests in PR builds on Windows

Enable System.Security tests in PR builds on Windows

Merge pull request #3742 from alexrp/profiler-c9-fixes

[profiler] Start the helper thread before the writer thread.

[profiler] Start the helper thread before the writer thread.

This fixes a regression introduced in 0db428169a42373a73cd5c502bd40033629f3f8d
which caused the profiler to not report the correct command port in the file
header on startup.

[SRE] Set the wastypebuilder flag on completed generic param builder … (#3712)

* [SRE] Set the wastypebuilder flag on completed generic param builder classes to remove a workaround from mono_type_get_object_checked ().

* [SRE] Remove unused verify_safe_for_managed_space () function.

[llvm] Make sure the initlocals_bb is not treated as part of a try block if one starts at IL offset 0, since it could mess up the control flow if it contains calls. (#3718)

Merge pull request #3741 from akoeplinger/fix-aot-DoAssemblyResolve

[aot] Fix AppDomain.DoAssemblyResolve runtime invoke wrapper

[aot] Fix AppDomain.DoAssemblyResolve runtime invoke wrapper

In https://github.com/mono/mono/pull/1528 the DoAssemblyResolve method was changed
from two parameters to three, but the code in aot-compiler.c was not updated to match.

Document --gc-debug and --gc-params command line options

Add --gc-params and --gc-debug command line options

Merge pull request #3731 from lambdageek/dev-handles-gbmc

[runtime] System.Configuration.InternalConfigurationHost:get_bundled_machine_config  needs HANDLES

Merge pull request #3729 from ntherning/fix-failures-in-msbuild-tests-on-windows

Fix failures in MSBuild tests on Windows

[runtime] Fix the blittable flag and the instance size for pointer types. Fixes #45131. (#3735)

Merge pull request #3737 from akoeplinger/fix-babysitter

[ci] Set the env var in a new subshell to fix babysitter processing

Fix failures in MSBuild tests on Windows

Merge pull request #3727 from henricm/fix-system-web-on-windows

Fixes for System.Web to work on windows

Enabling System.Web tests on Windows

Make blocking socket calls on Windows interruptible

This second commit adds functions to socket-io.c which uses
WSAEventSelect()/WSAWaitForMultipleEvents() or overlapped I/O to make
potentielly blocking socket calls interruptible (alertable).

Calling directly into Win API to get current process exe

On Windows the MonoToolsLocator tries to determine which mono.exe
to use by looking at the current process. Due to problems with
AppDomains and shadow copying (which happens in System.Web) we
could not use this approach. Instead we now call GetModuleFileName
directly using p-invoke to get the exe file of the running process.
This is only used when running on Windows.

[System] Use default parsing rules for Uri.IsWellFormedUriString. Fixes #45129

Make blocking socket calls on Windows interruptible

This first commit changes the signatures of the icalls for blocking socket
calls in Socket to propagate whether the socket call should block or not. This
is required since it's not possible on Windows through any native API to
determine whether a socket is in blocking or non-blocking mode.

This commit also adds the necessary RegisterForBlockingSyscall() guards to the
Socket.SendFile_internal() method which takes a WSABUF array. Without this
calls to this method won't be signaled when the Socket is closed.

This commit also fixes the error handling in the Socket.SendFile() method to
not rely on WSAGetLastError(). It's likely to have been overwritten between
when the error is generated and the WSAGetLastError() call from managed code.

[ci] Set the env var in a new subshell to fix babysitter processing

Setting the MONO_TLS_PROVIDER env var in the testing script makes it opaque to the
babysitter and other processing scripts. By setting it in a new subshell we can
ensure that the scripts see the whole invocation.

Merge pull request #3730 from kumpera/async-reader-dispose-hardening

[System] Add some hardening when disposing a AsyncStreamReader as it will have things happening concurrently.

[corlib] RemotingConfiguration now try to use the bundled machine.config first.

Merge pull request #3660 from lambdageek/bug-42271

[coop] Switch to GC SAFE for backtrace() and backtrace_symbols()

[runtime] give get_bundled_machine_config ves_icall_Class_Method-style names

[runtime] System.Configuration.InternalConfigurationHost:get_bundled_machine_config
needs HANDLES

The method (which also goes by
System.Configuration.DefaultConfig:get_bundled_machine_config) was
converted to use coop handles earlier, but we missed this declaration.

[System] Add some hardening when disposing a AsyncStreamReader as it will have things happening concurrently.

[SRE] Fixup the tokens in subtypes too. (#3721)

Merge pull request #3725 from ntherning/fix-symbolicate-tests-stack-trace-diff-on-windows

Fix symbolicate tests stack trace diff on Windows

[mempool] Fix indentation in mono_backtrace ()

[coop] Switch to GC SAFE for backtrace() and backtrace_symbols()

Fixes [#42271](https://bugzilla.xamarin.com/show_bug.cgi?id=42271)

[corlib] Add ValueTuple

Fix symbolicate tests stack trace diff on Windows

The symbolicate tests compare stack traces generated by the mono-symbolicate
tool against expected stack traces. The expected stack trace files have been
preprocessed to remove absolute paths and the Makefile does the same with the
output from the tool. However, the preprocessing doesn't take different
directory separator characters into account. This patch adds a 'tr' command to
the Makefile which converts \ -> / before the absolute paths are filtered out
from the actual output from the tool to make the comparison succeed on Windows
as well.

Ignoring CR in WebControls tests

Merge pull request #3707 from lateralusX/jlorenss/win-api-family-support-libmonoutils

Build libmonoutils under none desktop Windows API family.

Merge pull request #3720 from ntherning/fix-monodoc-tests-on-windows

Fix monodoc tests on Windows

Fix a typo.

Merge pull request #3714 from kumpera/fix_process_stream_dispose

[System] Fix Process::Dispose resource leak due to RS import.

[System] Dispose all streams in Process::Close if they were not exposed to the user. Fixes a RS import regression.

RS takes the stance that redirected streams should not be closed by the Process class because they might
be referenced from the outside.

That would be ok if it was true. If you use Process::BeginOutputReadLine, the output stream becomes inaccessible
with no way to dispose them by any mean but GC.Collect. You get an exception when accessing Process::StandardOutput
once async reading starts.

This change addresses the case where the streams are not accessible. This is tracked by the *StreamReadMode variables
that are set on stream getters or Begin*ReadLine.

[System.Net.Http] Wrap certain exceptions into HttpRequestException. Fixes #44978

Merge pull request #3684 from lambdageek/dev-monoerror-poolbox

[runtime] Use boxed MonoError for class failure.

Fix monodoc tests on Windows

These tests are sensitive to differences in line endings between Windows and
other platforms. This patch changes the diff commands used on Windows to
ignore changes in whitespaces at end of lines.

Merge pull request #3692 from lateralusX/jlorenss/win-api-family-support-libmono

Build libmono under none desktop Windows API family.

[mcs] Don't report version mismatch error for FX assemblies

Merge pull request #3709 from rolfbjarne/linker-mark-nested-types

[linker] We need to mark nested types even if the declaring type isn't marked.

[ci] Test System.dll with both the legacy and btls TLS provider