Merge pull request #1345 from mattleibow/websocket-continuation-frame-fix

[system] WebSocket continuation frame incorrectly handles the message type

Merge pull request #2947 from mfilippov/master

Socket.AcceptAsync should use SocketAsyncEventArgs.AcceptSocket

Socket.AcceptAsync shouldn't create new socket if SocketAsyncEventArgs.AcceptSocket isn't null.

[runtime] Moved MONO_ZERO_LEN_ARRAY to mono-publib.h.

Merge pull request #2960 from directhex/switchable-configure-in-jenkins

Switchable configure in jenkins

[jenkins] introduce CI_TAGS env var, to alter flow of jenkins script

Refactoring SocketAcceptAsyncTest

Fix SocketAcceptAsyncTest

[jenkins] Don't overwrite EXTRA_CONF_FLAGS inside run-jenkins.sh

This means we can pass EXTRA_CONF_FLAGS to the script, to more easily change the script behaviour from Jenkins

Merge pull request #2949 from Unity-Technologies/xbuild-windows-mcs

Make xbuild on Windows use Mono instead of .NET

Merge pull request #2957 from akoeplinger/fix-corlib-mobile-tests

Fix two corlib test issues on mobile

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

@monojenkins merge

Merge pull request #2953 from mono/fix-fileinfo-delete

[corlib] Bring the FileInfo.Delete fix

[corlib] Fix FileInfoTest.MoveTo_SameName test that failed on mobile

The MoveTo target is in the current working directory instead of in the temp dir, so it runs into permission issues.
Use the temp dir instead. Fixes part 2 of #40624

[corlib] Ignore a TimeZoneInfo test on mobile that doesn't make sense there

It failed on maccore because it uses reflection on code that's excluded on mobile.
Exclude the test too, fixes part 1 of #40624

[runtime] Remove dead mono_thread_get_and_clear_pending_exception () function.

Merge pull request #2946 from ludovic-henry/fix-process-40643

[Process] Fix Process.Start handle leak

Merge pull request #2924 from ludovic-henry/fix-missing-gsharedvt-wb

[mini] Add missing gsharedvt write barrier

[corlib] Bring the FileInfo.Delete fix

See https://github.com/mono/referencesource/commit/fdebd8a4d709002bb499555c35e1dfd3834e2f4f for details.

Also add a test for checking if deleting a non-existent file works (we were missing coverage there before).

[io-layer] Remove dead code

Merge pull request #2952 from mono/revert-2721-fix-mono_ms_ticks

Revert "[runtime] Fix potential overflow when using mono_msec_ticks"

Revert "[runtime] Fix potential overflow when using mono_msec_ticks"

Merge pull request #2721 from ludovic-henry/fix-mono_ms_ticks

[runtime] Fix potential overflow when using mono_msec_ticks

Make xbuild on Windows use Mono instead of .NET

We don't have .NEt 4.6 installed on all of our build machines, to
running mcs.exe on Windows fails, as it was built with .NET 4.6.
Instead, we will run mcs.exe via Mono on Windows (which is what happens
on OSX, although there the low-level Process class handles it).

To accomplish this, we run mcs.bat instead of mcs.exe.

Socket.AcceptAsync shouldn't create new socket if SocketAsyncEventArgs.AcceptSocket isn't null

[Process] Fix handle leak on child process Creation

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

[Process] Improve debugging info and slight refactor of CreateProcess

[Process] Fix System.Diagnostics.ProcessTest.StandardInputWrite

The child process would simpky execute ls, then exiting right away. Writing to its stdin would then crash with "EPIPE: An attempt is made to write to a pipe that is not open for reading by any process." as the child process exited.

Merge pull request #2941 from jogibear9988/patch-1

Bugfix DateTime in Sqlite: https://bugzilla.xamarin.com/show_bug.cgi?id=40603

Revert "[verifier] Do method visibility checks for virtual final methods. Enable type visibility for all methods. Fixes #5644."

This reverts commit 24ee5b8db0ab3115f88dcef72432568e3c1b8e83.

Revert "WIP - don't merge"

This reverts commit 359bb511d3cdb2b320fba35d99684152615d5242.

Revert "WIP fix the previous issue."

This reverts commit a26a53369857ed120522c8625b1bebc653f25baf.

[corlib] Reenable IsReadOnly test

[xbuild] Fix ApplicationIcon incremental build dependency

Merge pull request #2942 from jogibear9988/patch-2

bugfix numInfo of Cultureinfo NonSerialized on Windows

Merge pull request #2945 from mono/xi-40574

[linker] Method decorated with System.Runtime.Serialization.*Attribut…

[linker] Method decorated with System.Runtime.Serialization.*Attribute must be marked, Fixes #40574

Some serialization attributes are unlikely to have direct reference,
from IL, but will be needed at runtime to correctly serialize (or
deserialize) objects.

* OnDeserializedAttribute
* OnDeserializingAttribute
* OnSerializedAttribute
* OnSerializingAttribute

This, along with aad627c5e8ddf15397c87fa2d3e18b54f929c927, fixed bug
40574. The later part will have new unit tests in XI.

references:
https://bugzilla.xamarin.com/show_bug.cgi?id=40574

[corlib] Disable core security icalls on mobile

Microsoft relicensed this fork of the Ajax library under the MIT license

Microsoft relicensed this fork of the DLR under the MIT license

Merge pull request #2944 from lambdageek/dev/monoerror-thread_create

[runtime] MonoError-ize mono_thread_create{,_internal}

[runtime] MonoError-ize mono_thread_create{,_internal}

Mark mono_thread_create external only.  Runtime should use
mono_thread_create_checked.

[corlib] FileInfo from referencesource

[corlib] Remove System.Core security types

[corlib] Remove duplicated sources

[System.Core] Add CommonCrypto dependencies

[corlib] FileSystemInfo from reference source

[acceptance-tests] Move roslyn build dependencies to download server

storage.bos.internalx.com is only available via VPN now.

Also bump coreclr to get a recent minor fix.

Merge pull request #2943 from mono/xi-40570

[System.Core] Add back AesManaged to tvOS and watchOS profiles. Fixes #40570

[WindowsBase] Serialization fix of Int32Rect, Point, Rect, Size, Vector, Matrix (#2873)

The problem is that private fields in described types doesn't start from _ prefix, but it should be so according to reference source.
That's way MONO can't deserialize that types correctly if source stream was created by MS .NET binary serializer.

This change is released under the MIT license.

[System.Core] Add back AesManaged to tvOS and watchOS profiles

Switching maccore to use mono-4.5.0-branch introduced this bug where
AesManaged is not available in System.Core for both tvOS and watchOS
profiles (iOS is fine).

This results in unit tests failures on bots building maccore/master.

This fix the problem by importing the "main" monotouch_System.Core.
dll.sources into the other profiles - since they all share the same
feature set (in this case) and make it less likely to forget something
else in the future.

reference:
https://bugzilla.xamarin.com/show_bug.cgi?id=40570

[runtime] Allow enums with base type of char in Enum.ToObject (), this is valid in f#. Fixes #40568.

Merge pull request #2940 from xmcclure/disable-jit-gsharedvt

Fix gsharedvt compile on DISABLE_JIT platforms

bugfix numInfo of Cultureinfo NonSerialized on Windows, https://bugzilla.xamarin.com/show_bug.cgi?id=35872

Cultureinfo numInfo is not NonSerialized on DotNet for windows, so it also should not on mono.

http://referencesource.microsoft.com/#mscorlib/system/globalization/cultureinfo.cs,76

Bugfix https://bugzilla.xamarin.com/show_bug.cgi?id=40603

[I18N/CJK] iso-2022-jp GetChars() had some bogus conversion ranges.

Fix gsharedvt compile on DISABLE_JIT platforms

Merge pull request #2939 from discofire/patch-1

Update OracleConnection.cs

Update OracleConnection.cs

Fix SQL syntax for GetSchemaProcedures function. (Should have been 'SELECT' instead of 'SSELECT')

Merge pull request #2918 from lateralusX/jlorenss/aot-x86-symbol-fix

Windows 32-bit x86 AOT assembler didn't compile due to incorrect symbol mangling.

Merge pull request #2869 from BrzVlad/feature-mod-union-opt

[sgen] Optimize mod union usage

Merge pull request #2935 from kumpera/new-gc-test

[sgen] Add regression test for the interaction between the bridge and handles.

The bridge code was triggering handles to be considered deallocated thus leading
them to be overwritten.

Merge pull request #2932 from ludovic-henry/fix-waithandle-remoting

[corlib] Ifdef out calls to SynchronizationAttribute on monotouch

When WaitHandle was imported from referencesource in f6fa8f0, the code was refactored and basically all WaitHandle.Wait* calls end up in WaitOneNative or WaitMultiple now.

On monotouch those methods threw a NotSupportedException when exitContext == true even before the import, so this behavior got inherited to some other methods that didn't throw after the exception was removed from them in 93e31cb. This broke tests in maccore and other code that passed exitContext=true.

To fix this, we just ifdef out the calls to SynchronizationAttribute on monotouch since they don't make sense there anyway since remoting isn't available. The commit that initially added the exception in 2ce9fc1 only did that to make the behavior consistent when the linker is enabled. By removing the calls we get the same result.

Do the same for Monitor so we're consistent.

commit message from @akoeplinger

Merge pull request #2880 from kumpera/fix_5644

[verifier] Do method visibility checks for virtual final methods. Enable type visibility for all methods. Fixes #5644.

I cannot repro the issue mentioned in the comments of mono_method_can_access_method.

* FIXME:
* with generics calls to explicit interface implementations can be expressed
* directly: the method is private, but we must allow it. This may be opening
* a hole or the generics code should handle this differently.
* Maybe just ensure the interface type is public.

@monojenkins merge

[sgen] Add regression test for the interaction between the bridge and handles.

The bridge code was triggering handles to be considered deallocated thus leading
them to be overwritten.

[docs] Added index.xml entry for CFNetworkHandler. (#2930)

This particular piece of documentation is added manually, because it's not in the regular BCL library, but is provided by Xamarin on the iOS platform. Because it was missing an entry in the index file, it was excluded from the published documentation when doing mdoc assemble.

Merge pull request #2934 from vargaz/fix-36852

[sgen] Avoid returning NULL from null_link_if (), that means an empty…

[corlib] Ifdef out calls to SynchronizationAttribute on monotouch

When WaitHandle was imported from referencesource in f6fa8f0, the code was refactored and basically all WaitHandle.Wait* calls end up in WaitOneNative or WaitMultiple now.

On monotouch those methods threw a NotSupportedException when exitContext == true even before the import, so this behavior got inherited to some other methods that didn't throw after the exception was removed from them in 93e31cb. This broke tests in maccore and other code that passed exitContext=true.

To fix this, we just ifdef out the calls to SynchronizationAttribute on monotouch since they don't make sense there anyway since remoting isn't available. The commit that initially added the exception in 2ce9fc1 only did that to make the behavior consistent when the linker is enabled. By removing the calls we get the same result.

Do the same for Monitor so we're consistent.

[sgen] Avoid returning NULL from null_link_if (), that means an empty slot, not a handle pointing to null. Fixes #36852.

WIP fix the previous issue.

WIP - don't merge

[verifier] Do method visibility checks for virtual final methods. Enable type visibility for all methods. Fixes #5644.

I cannot repro the issue mentioned in the comments of mono_method_can_access_method.

* FIXME:
* with generics calls to explicit interface implementations can be expressed
* directly: the method is private, but we must allow it. This may be opening
* a hole or the generics code should handle this differently.
* Maybe just ensure the interface type is public.

[build] Add missing empty assemblies needed by default csc config

[profile] Fix a regression from last commit.

Merge pull request #2859 from lambdageek/dev/monoerror-object-c

[runtime] Grab bag of MonoError fixes in object.c

[mcs] Fixes crash during conversion error reporting. Fixes #40536

[mcs] null operator receiver type needs to be convertible to null. Fixes #40493

[tools] Fixes compiler tester error messages verification

Merge pull request #2891 from xmcclure/bug-39669

Allow xml configuration sections to reference System.dll (bug#39669)

[profiler] Remove unneeded call to mono_lock_free_queue_node_init.

Remove calls to mono_lock_free_queue_node_init for nodes that just came out of mono_lock_free_queue_dequeue.

That call is superfluous as dequeue will set the returned node 'next' field to INVALID_NEXT, which is the
same thing the mono_lock_free_queue_node_init call did.

Merge pull request #2922 from akoeplinger/fix-tvos

[utils] Fix MONO_SIGNAL_USE_UCONTEXT_T not being defined on tvOS simulator

In the maccore tvossimulator tests we were seeing the following assertion with the Mono 4.5 preview:

```
error: * Assertion: should not be reached at /Users/builder/data/lanes/1381/0d6923b4/source/maccore/external/mono/mono/utils/mono-context.c:194

0   dont link                           0x000000010175702d mono_handle_native_sigsegv + 253
1   libsystem_platform.dylib            0x000000010a0cfeaa _sigtramp + 26
2   ???                                 0x0000000001bfc4db 0x0 + 29344987
3   libsystem_c.dylib                   0x0000000109e56cbc abort + 129
4   dont link                           0x00000001018d1503 _ZL12log_callbackPKcS0_S0_iPv + 67
5   dont link                           0x00000001018c26d0 monoeg_assertion_message + 192
6   dont link                           0x00000001018af6de mono_sigctx_to_monoctx + 30
7   dont link                           0x000000010183f473 sgen_suspend_thread + 323
8   dont link                           0x000000010183f58e sgen_thread_handshake + 126
9   dont link                           0x000000010183fb20 sgen_client_stop_world + 1296
10  dont link                           0x0000000101871283 sgen_stop_world + 67
11  dont link                           0x0000000101870d43 sgen_perform_collection + 99
12  dont link                           0x0000000101872576 sgen_gc_collect + 102
13  dont link                           0x00000001018d1886 _ZL7pump_gcPv + 54
14  libsystem_pthread.dylib             0x000000010a0b7c13 _pthread_body + 131
15  libsystem_pthread.dylib             0x000000010a0b7b90 _pthread_body + 0
16  libsystem_pthread.dylib             0x000000010a0b5375 thread_start + 13
```

This was caused by a4e257a07f511062a8feb3e06e1566fd980110ce, the issue is that the check was changed from UCONTEXT_REG_RAX to MONO_SIGNAL_USE_UCONTEXT_T, but that is only available when HAVE_SIGACTION is defined which it is not in this case.

Talked with Zoltan, fixing this by enabling MONO_SIGNAL_USE_UCONTEXT_T on all apple platforms for now.

/cc @vargaz @xmcclure

@monojenkins merge

Another round of profiler fixes/improvements. PR #2882 (Manually Merged)

https://github.com/mono/mono/pull/2882

[runtime] MonoError-ize mono_ldstr_metadata_sig

[runtime] MonoError-ize mono_string_new_len

Mark it external only.  Runtime should use mono_string_new_len_checked.

[runtime] MonoError-ize mono_wait_handle_new

[runtime] MonoError-ize mono_array_new_cached

[runtime] MonoError-ize mono_array_new

Mark it external only.

Runtime should use mono_array_new_checked.
The icall is ves_icall_array_new.

[runtime] object.c don't raise in external only functions

- Variants of mono_object_new
- mono_object_clone
- mono_array_new_full

All these functions have _checked variants that are used by the runtime.

[runtime] MonoError-ize mono_array_clone

1. Mark mono_array_clone external only.  Runtime should use
  mono_array_clone_checked.
2. Add MonoError* arg to mono_array_clone_in_domain.
3. Separate ves_icall_System_Array_Clone

[runtime] Mark mono_object_new_from_token external only.

Does not appear to be used in the runtime.
Changed it not to raise exceptions.

[runtime] Set pending, don't raise, in object.c icalls

[runtime] MonoError-ize mono_object_xdomain_representation

[runtime] MonoError-ize mono_property_get_value

Mark it external only.  Runtime should use mono_property_get_value_checked.

[runtime] MonoError-ize mono_property_set_value

Mark it external only.  Runtime should use
mono_property_set_value_checked.

Note: mono_property_set_value_checked always catches exceptions from
managed code and threads them out via MonoError.

[runtime] Don't raise exns, set pending exception

[runtime] MonoError-ize mono_runtime_create_jump_trampoline

[attach] Don't raise in mono_attach_load_agent

Merge pull request #2806 from lambdageek/dev/monoerror-marshal_remote

[runtime] MonoError-ize marshal and remoting code

Get rid of calls to `mono_raise_exception` in marshaling and remoting code.

[profiler] Don't link against libmono.

This would result in duplicate runtimes being loaded (as evidenced by
/proc/$pid/maps), which could lead to all sorts of breakage.

[profiler] Load the profiler with MONO_DL_EAGER.

We need to make sure all external references have been resolved so that we
don't invoke the dynamic linker in the mono_sample_hit () callback which is
called in async-signal context.

[utils/dl] Add a MONO_DL_EAGER enum value for clarity.

[profiler] Don't use a real-time signal on Android.