[reflection] Use MonoError in mono_reflection_get_type_with_rootimage

Merge pull request #2684 from ludovic-henry/monoerror-exception

[runtime] Use MonoError for mono_get_exception_type_initialization, mono_get_exception_reflection_type_load and mono_get_exception_runtime_wrapped

Merge pull request #2696 from lambdageek/dev/unlock-mono_class_create_runtime_vtable

[runtime] Don't raise exception while holding locks

[msvc] Update .def files to fix Windows build

[runtime] Use MonoError for mono_get_exception_type_initialization, mono_get_exception_reflection_type_load and mono_get_exception_runtime_wrapped

[runtime] Move some mono_error_raise_exception () calls to icalls from utility functions. NFC.

[runtime] Rename some icalls to match the icall naming convention. NFC.

Merge pull request #2705 from kumpera/loader-error-cleanup7

[runtime] Remove the loader error from the remaining of the loader.

[misc] Multiple small fixes and cleanups.

[utils] Add mono_error_move.

[runtime] Ban mono_method_get_header and promote mono_method_get_header_check to the API as the profiler needs it.

Merge pull request #2662 from mattleibow/pcl-mono.options

[Mono.Options] Changed the code a bit so that it works with PCL

[runtime] Remove all usages of mono_method_get_header from the runtime.

[runtime] Replace mono_method_get_header with mono_method_get_header_checked.

[runtime] Use mono_method_get_header_checked in debug-helpers.c

[gc] Use the new MonoMethodBuilder:init_locals field.

[runtime] Add init_locals field to MonoMethodBuilder to make it easy to get it set to false.

[runtime] Add mono_method_get_header_checked and implement mono_method_get_header using it.

[runtime] Ban mono_method_get_signature_full. It has no runtime usage left.

[runtime] Ban mono_method_get_signature. Nothing else uses it.

[runtime] Ban mono_class_inflate_generic_type from usage by the runtime. Move code to use mono_class_inflate_generic_type_checked.

[runtime] Remove mono_class_inflate_generic_class, replace usage with mono_class_inflate_generic_class_checked.

[runtime] Add mono error to inflate_generic_header. Fix buglet in mono_method_get_signature_checked.

[runtime] Ban mono_metadata_parse_mh from the runtime.

[runtime] Introduce MonoError in mono_metadata_parse_mh_full.

[runtime] Remove dead function mono_metadata_parse_type_full.

[runtime] Disallow usage of mono_metadata_parse_type, mono_metadata_parse_param and mono_metadata_parse_field_type.

[mono-error] mono_metadata_parse_array is not longer used by the runtime, ban it.

[mono-error] mono_class_from_name and mono_class_from_name_case now can cleanly return NULL on failure.

Merge pull request #2686 from ludovic-henry/monoerror-mono_remoting_invoke

[runtime] Use MonoError for mono_remoting_invoke

Merge pull request #2701 from akoeplinger/fix-async-read-test

Fix unreliability in async_read.exe test

Merge pull request #2702 from akoeplinger/bug-38933

[System.Security] Fix CryptographicException from ProtectedData.Protect with multiple threads

Merge pull request #2703 from akoeplinger/pupitetris-Bug-35635

Don't trim TimeZone names ending in parenthesys

Replaces/Closes #2206

@monojenkins merge

[corlib] Add test for #35635

Don't trim TimeZone names ending in parenthesys.
Such as "Central Standard Time (Mexico)"

[System.Security] Fix CryptographicException from ProtectedData.Protect with multiple threads

The code in ManagedProtection.GetKey() instantiated a new RSACryptoServiceProvider singleton without proper
locking to avoid multiple threads running the initialization concurrently.
Since RSACryptoServiceProvider (or rather KeyPairPersistence underneath) uses an XML file with a name derived
from the parameters to store the data there could be a case where one thread writes while another tries to read
and gets garbage. This results in a CryptographicException later on.

Added locking to prevent this.

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

Fix unreliability in async_read.exe test

It failed on Jenkins today (https://jenkins.mono-project.com/job/test-mono-mainline/label=debian-amd64/3611/parsed_console/log_content.html#WARNING1)
and I could repro it after running the test in a loop.

While looking at the test, the "sum -= buf [0]" seemed suspicious to me. The final callback invocation after the last byte was read results
in buf [0] being 0 most of the time (because no bytes were read). However, due to the async nature of the test that final callback might happen
before another callback and buf then has a value, resulting in a wrong checksum if that value is substracted.

The fix is to remove the substraction. I have no idea why it was done in the first place, but it's been there since the test was added in 2003...

While at it, I've added another check for the number of bytes read and used proper Interlocked.* operations to ensure atomicity.

Merge pull request #2695 from vargaz/process-set-pending

[runtime] Use pending exceptions in process.c.

[runtime] Use the return_if_nok () macros in socket-io.c.

Merge pull request #2691 from vargaz/socket-io-pending-exception

[runtime] Use pending exceptions in socket-io.c.

[runtime] Use return_if_nok/return_val_if_nok to simplify error handling code in process.c.

Merge pull request #2694 from ludovic-henry/fix-sleep-overflow

[threads] Fix mono_thread_info_sleep overflow

This overflow would happen of the machine was up for more than 29 days, leading to an overflow in the value returned by mono_100ms_ticks.

The simplest solution is simply to resort to mono_100ns_ticks which would only overflow after more than 29247 years. So even the platform implement this value as the time since 0000-00-00 00:00:00, we still have more than 27000 years if front of us. It's also highly unlikely that we would overflow the `end' value when adding `ms', as that would mean we are in the last 29 days of that 29247 years.

@monojenkins merge

Merge pull request #2680 from lambdageek/dev/reflection-no-loadererror

[reflection] Get rid of loader error in reflection.c

Get rid of (direct and indirect) calls to `mono_loader_set_error_from_mono_error` in reflection.c:

- `mono_custom_attrs_from_index`
- `mono_custom_attrs_from_method`
- `mono_custom_attrs_from_class`
- `mono_custom_attrs_from_assembly`
- `mono_custom_attrs_from_module`
- `mono_custom_attrs_from_property`
- `mono_custom_attrs_from_event`
- `mono_custom_attrs_from_field`
- `mono_custom_attrs_from_param`

In all the functions, add a `_checked` variant that has a `MonoError*` out argument and for the  MONO_API functions.  The public API versions just clean up the error and return NULL.  Loader error isn't touched anymore.

Merge pull request #2675 from lambdageek/dev/monoerror-mono_string_intern

Use MonoError in mono_string_{is_interned,intern}

Also mark mono_string_intern external only.  Runtime should use
mono_string_intern_checked.

[metadata] Don't abort if trying to fetch the header summary of a method with no body. Fixes #39200.

This assert made it impossible to fail gracefully when inlining broken methods.

[corlib] Fix tests to compile when Thread.Suspend/Resume is not supported.

[runtime] Don't raise exception while holding locks

[reflection] Expect no loader error in mono_reflection_get_custom_attrs_by_type anymore

[reflection] Use MonoError for mono_custom_attrs_from_param

Mark it external only.  Runtime should use mono_custom_attrs_from_param_checked

[reflection] Use MonoError for mono_custom_attrs_from_field

Mark it external only.  Runtime should use mono_custom_attrs_from_field_checked.

[reflection] Use MonoError for mono_custom_attrs_from_event

Mark it external only.  Runtime should use mono_custom_attrs_from_event_checked.

[reflection] Use MonoError for mono_custom_attrs_from_property

Mark it external only.  Runtime should use mono_custom_attrs_from_property_checked.

[reflection] Use MonoError for mono_custom_attrs_from_module

[reflection] Use MonoError for mono_custom_attrs_from_assembly

Mark it external only.  Runtime should use mono_custom_attrs_from_assembly_checked.

[reflection] Use MonoError for mono_custom_attrs_from_class instead of loader error.

Also mark mono_custom_attrs_from_class external only.  Runtime should
use mono_custom_attrs_from_class_checked.

[reflection] Use MonoError in mono_custom_attrs_from_{index, method} instead of loader error.

Also mark both as external only.  Runtime should use
mono_custom_attrs_from_index_checked and mono_custom_attrs_from_method_checked.

[runtime] Fix compilation warning

[runtime] Use MonoError for mono_remoting_invoke

Use MonoError in mono_string_{is_interned,intern}

Also mark mono_string_intern external only.  Runtime should use
mono_string_intern_checked.

[runtime] Use pending exceptions in process.c.

[threads] Fix mono_thread_info_sleep overflow

This overflow would happen of the machine was up for more than 29 days, leading to an overflow in the value returned by mono_100ms_ticks.

The simplest solution is simply to resort to mono_100ns_ticks which would only overflow after more than 29247 years. So even the platform implement this value as the time since 0000-00-00 00:00:00, we still have more than 27000 years if front of us. It's also highly unlikely that we would overflow the `end' value when adding `ms', as that would mean we are in the last 29 days of that 29247 years.

Merge pull request #2693 from dellis1972/master

[msbuild] Getting build error "Error initializing task XmlPeek: Not registered task XmlPeek".

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

Removes the uses of LogXXXFromResources from XmlPeek because
we do not use string resources in xbuild.

[msbuild] Getting build error "Error initializing task XmlPeek: Not registered task XmlPeek".

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

Removes the uses of LogXXXFromResources from XmlPeek because
we do not use string resources in xbuild.

Merge pull request #2692 from vargaz/process-coding-conv

[runtime] Use mono coding conventions in process.c.

[runtime] Use mono coding conventions in process.c.

[runtime] Use pending exceptions in socket-io.c.

Merge pull request #2690 from steffen-kiess/json-operator-uint

Fix typo in JsonValue which broke the conversion to uint and add test case

Fix typo in JsonValue which broke the conversion to uint and add test case

Bring new IKVM

Merge pull request #2580 from ItsVeryWindy/master

Updates to WebRequestHandler

Merge pull request #2672 from Numpsy/sgenexception

[sgen] catch NotSupportedException from system.xml.serialization

Merge pull request #2689 from vargaz/socket-io-cleanup

[runtime] Clean up the socket-io.c file to use the mono coding conven…

[runtime] Clean up the socket-io.c file to use the mono coding conventions. Rename 'error' to 'werror' so 'error' can be used to refer to MonoError as everywhere else. NFC.

Merge pull request #2688 from vargaz/marshal-set-pending

[runtime] Use pending exceptions instead of mono_raise_exception () i…

…n Marshal icalls.

[runtime] Use pending exceptions instead of mono_raise_exception () in Marshal icalls.

[aot] Fix the usage of MonoError in init-llvm_method ().

Merge pull request #2678 from esdrubal/filestream

[corlib] Fixed FileStream Write after Read issue

Merge pull request #2685 from ludovic-henry/monoerror-mono_object_register_finalizer

[runtime] Use MonoError for mono_object_register_finalizer

Merge pull request #2671 from lambdageek/dev/monoerror-mono_runtime_class_init

[runtime] Use MonoError for mono_class_vtable_full and mono_runtime_class_init_full

Switch from a boolean `raise_on_error` arg to a MonoError outarg and let
callers decide what to do with the error.

Also mark mono_runtime_class_init external only.  Runtime should use
mono_runtime_class_init_full.

[runtime] Use MonoError for mono_class_vtable_full and mono_runtime_class_init_full.

Switch from a boolean raise_on_error arg to a MonoError outarg and let
callers decide what to do with the error.

Also mark mono_runtime_class_init external only.  Runtime should use
mono_runtime_class_init_full.

[scripts] Point scripts like nunit-console2/mono-service2 to the 4.5 directory

The 2.0 profile doesn't ship with Mono anymore. To avoid breaking existing scripts we just point to the 4.5 version instead.

[System.ServiceModel] Fix a hardcoded port in a test

This failed with an "address already in use" error on Jenkins yesterday.

[runtime] Use MonoError for mono_object_register_finalizer

Merge pull request #2679 from lambdageek/dev/monoerror-reflection

[reflection] MonoError cleanups

[runtime] Remove some declarations from marshal.h which are not used outside of marshal.c.

Merge pull request #2683 from mfilippov/master

Add generation WepApplication.targets for v12.0 and v14.0 xbuild.

Add generation WepApplication.targets for v12.0 and v14.0 xbuild.

Merge pull request #2681 from kumpera/pedump_fix

[pedump] Fix init crash on pedump when using --verify.

@marek-safar  here's your fix

@monojenkins merge

[aot] Move a mono_error_raise_exception () call higher up the call chain.

[pedump] Fix init crash on pedump when using --verify.

[sdb] Remove some mono_error_raise_exception () calls.

[jit]: Correctly restore callee saved registers in mono_arch_get_call_filter() on amd64.

(cherry picked from commit 8c1ffe78ccf68fab9acad53eebc01b5c84fc81ec)

[corlib] Fixed MonoIO.Write loop.

[corlib] Fixed FileStream Write after Read issue

FileStream was writing to the wrong position When a value longer than
the FileStream buffer was written after a read.

This was fixed by doing a seek before doing the problematic write.

Fixes #11699

[corlib] Test FileStream with Read and Writes

Covers #11699

Fixed the build.

[support/zlib] Added API exports to exported calls from zlib-helper.c.

This is so we can re-use this code from libmonodroid, which currently has its own duplicated copy.

[support/zlib] Remove eglib link dependencies from zlib-helper.c.

This is so we can re-use this code from libmonodroid, which currently has its own duplicated copy.

[reflection] MonoError in mono_reflection_get_custom_attrs_info

Also mark that function external only.  Runtime should use
mono_reflection_get_custom_attrs_info_checked

[reflection] MonoError for mono_reflection_type_get_underlying_system_type