[corlib] Fix potential RegisteredWaitHandle resource leak

If we construct RegisteredWaitHandle with executeOnlyOnce = true, and never call Unregister, we will keep _waitObject.SafeWaitHandle reference count unbalanced. This will lead to never releasing the waitObject handle.

As we only need to ensure that the waitObject handle is alive while in the Wait method, we use the SafeHandle mechanism only there. The 2 possibles scenarios in the Wait method are:
 - waitObject has not been closed: we increment its ref count, so we ensure that it's alive as long as we use it here
 - waitObject has been closed: a ObjectDisposedException is thrown, we catch it and keep on without trying to wait on the handle

Merge pull request #2627 from kasthack/sysweb-import

[System.Web] Reference source import

Merge pull request #2629 from gofman/fixtimezoneinfo

[System.TimeZoneInfo] Fix finding system time zone when /etc/localtim…

[System.TimeZoneInfo] Fix finding system time zone when /etc/localtime has a relative path.

[jit] Use mono_resolve_patch_target_checked () in more places.

[jit] Use mono_resolve_patch_target_checked () in more places.

[System.Web] Reference source import: System.Web.Util internal classes

[System.Web] Reference source import: System.Web.Routing internal classes

[System.Web] Reference source import

Fix the build.

[llvm] Remove some unfinished code committed by mistake.

Merge pull request #2625 from xmcclure/tinymemoryleak

Fix memory leak in mono_aot_plt_resolve when exiting with exception

We were not cleaning up our allocated memory on all code paths.

[mono-error] Replace mono_metadata_parse_type_full with mono_metadata_parse_type_checked.

[metadata] Introduce MonoError into mono_metadata_parse_type_internal.

[metadata] Remove mono_metadata_parse_array_full, nothing uses it.

[llvm] Fix a crash when a simd argument is returned from a method by reference.

[runtime] Fix the build.

[System.Web] Reference source import

[runtime] Replace hand-written type look up/caching with GENERATE_GET_CLASS_WITH_CACHE in method-to-ir.c and mini-exceptions.c.

The macro generates the right form of corlib type lookup & caching.

[runtime] Replace hand-written type look up/caching with GENERATE_GET_CLASS_WITH_CACHE in threads.c.

The macro generates the right form of corlib type lookup & caching.

[runtime] Replace hand-written type look up/caching with GENERATE_GET_CLASS_WITH_CACHE in verify.c and remoting.c.

The macro generates the right form of corlib type lookup & caching.

[runtime] Replace hand-written type look up/caching with GENERATE_GET_CLASS_WITH_CACHE in security-core-clr.c and security-manager.c.

The macro generates the right form of corlib type lookup & caching.

[runtime] Replace hand-written type look up/caching with GENERATE_GET_CLASS_WITH_CACHE in reflection.c.

The macro generates the right form of corlib type lookup & caching.

[runtime] Replace hand-written type look up/caching with GENERATE_GET_CLASS_WITH_CACHE in object.c.

The macro generates the right form of corlib type lookup & caching.

[runtime] Replace hand-written type look up/caching with GENERATE_GET_CLASS_WITH_CACHE in marshal.c.

The macro generates the right form of corlib type lookup & caching.

[runtime] Replace hand-written type look up/caching with GENERATE_GET_CLASS_WITH_CACHE in locale.c.

The macro generates the right form of corlib type lookup & caching.

[runtime] Replace hand-written type look up/caching with GENERATE_GET_CLASS_WITH_CACHE in icall.c.

The macro generates the right form of corlib type lookup & caching. Plus it's shorter.

Import System.Diagnostics.Process changes to mobile profile

Changes modeled on ludovic-henry commit 00b5517dd3. Fixes build.

Fix memory leak in mono_aot_plt_resolve when exiting with exception

[linker] do not set scope of non-marked typerefs

 - fixes #38818

 - setting the *unchanged* scope to non-marked types is harmful,
   because in case the type has declaring type, the scope of declaring
   type is set as well. see
   https://github.com/mono/cecil/blob/master/Mono.Cecil/TypeReference.cs#L132

   public virtual IMetadataScope Scope {
      get ...
      set {
          var declaring_type = this.DeclaringType;
          if (declaring_type != null) {
              declaring_type.Scope = value;
              return;
          }
          scope = value;
      }
   }

   so in our case, we first set correctly scope of
   System.Collections.Generic.Dictionary`2 to mscorlib, then (re)set
   scope of System.Collections.Generic.Dictionary`2/ValueCollection
   and
   System.Collections.Generic.Dictionary`2/ValueCollection/Enumerator
   to System.Collection (note that these types are not marked and are
   swept away by linker), which sets scope of its declaring type
   System.Collections.Generic.Dictionary`2 back to System.Collection

   and because the System.Collection is linked-out, the typeref is
   incorrectly written with the scope set to System later

[runtime] Bump corlib version

[Process] Add stress tests

That's to ensure we do not have regression in the Process implementation.

[System] Import System.Diagnostics.Process

[corlib] Fix race condition between RegisteredWaitHandle Wait and Unregister methods

In the following case we would have a race between Wait and Unregister methods:
 - T1 calls ThreadPool.RegisterWaitForSingleObject to wait on waitHandle
 - registeredWaitHandle is constructed to wait on waitHandle
 - registeredWaitHandle.Wait method is enqueued on the ThreadPool
 - T1 calls registeredWaitHandle.Unregister
 - T1 calls waitHandle.Close (happens in Process.StopWatchingForExit)
 - TTP (the threadpool thread) calls registeredWaitHandle.Wait
  - TTP calls WaitHandle.WaitAny (waitHandle, ...)

The last step try to access the waitHandle that has been previously closed, leading to a _wapi_handle_ref / _wapi_handle_unref_full error message.

The fix is to ensure that the RegisteredWaitHandle keeps the WaitHandle alive. To achieve this, we simply use the SafeWaitHandle inside the WaitHandle by: increasing its ref count when constructing the RegisteredWaitHandle, and decrementing it when unregistering it.
We also increase the count when using it in Wait, so if we execute Unregister and Wait concurrently, we guarantee that the WaitHandle is still valid in Wait, or get an exception if not.

[mcs] Add more known sdk paths redirections

Merge pull request #2603 from kumpera/fix_set_pending_exception

[runtime] Move pending_exception from InternalThread to Thread to avoid crashing on domain unload.

Merge pull request #2548 from esdrubal/discovery

[System.Web.Services] Fix DiscoveryClientProtocol

Merge pull request #2623 from esdrubal/fileinfo_moveto

[corlib] Fix FileInfo.MoveTo with same name.

Merge pull request #2615 from akoeplinger/fix-perfcounter-free-mem-calculation

[perfcounters] Fix calculation of free memory in mono_determine_physical_ram_available_size ()

[System.Web.Services] Test DiscoveryClientProtocol

Covers #36116

[corlib] Fix FileInfo.MoveTo with same name.

FileInfo.MoveTo was not moving a file from one directory to another when
the file name did not change.

Fixes #18361.

An ArgumentException when destFileName is an empty array was added,
otherwise FileInfoTest.MoveTo_DestFileName_Empty would fail. Both
Path.GetFullPath and File.Move throw similar exceptions but
MoveTo_DestFileName_Empty expects the one from File.Move so it was added
to this method.

[corlib] Test FileInfo.MoveTo with files with same name

Covers #18361

Bump reference source to fix the build

Merge pull request #2562 from BrzVlad/fix-drainstack-stats

[sgen] Drain the gray stack only in finish gray stack

[corlib] MethodInfo from reference source

[mcs] Update expression tree generator to use .net 4.5 api when available. Fixes #38703

[io-layer] Add Get/SetThreadPriority to wapi-remap.h

The new functions were added in d5768a7f141e2a579cbca26f76c791c215f4aabf but not remapped.

Merge pull request #2619 from directhex/dont_remap_npgsql

Don't remap all versions of Npgsql to 4.0.0.0

Don't remap all versions of Npgsql to 4.0.0.0

Mono has a list of assemblies considered "internal", whose versions are automatically remapped to the distro default version.

Since we no longer bundle Npgsql, it is up to individual distributions to package their own Npgsql - but Mono cannot load any version other than 4.0.0.0 from the GAC, due to this remapping:

Mono: Assembly Ref addref npghello[0x17e2fd0] -> mscorlib[0x1789c00]: 2
Mono: The request to load the assembly Npgsql v2.2.7.0 was remapped to v4.0.0.0
Mono: Assembly Loader probing location: '/usr/lib/mono/gac/Npgsql/4.0.0.0__5d8b90d52f46fda7/Npgsql.dll'.

Npgsql, Version=2.2.7.0, Culture=neutral, PublicKeyToken=5d8b90d52f46fda7

[msvc] Update order.xml and csproj files

[genproj] Fix bashism in library.make

This was added in 2687866e99a0bb6669dc61b84393bc3ec5de3fc6, but on systems where /bin/sh (which make uses by default) isn't bash this doesn't work.

Fix is to use a standard switch-case which works in all shells.

[msvc] Update .def files

Merge pull request #2400 from esdrubal/extrahead

[System] Fix extra head in config bug.

Merge pull request #2323 from esdrubal/servicemodel

[System.ServiceModel] Fixes missing xsi xsd attributes

Merge pull request #2613 from knocte/crypto-refsource

[Crypto] Include CngKeyBlobFormat from reference sources (rm old)

[mcs] Populate ProductVersion from all valid sources. Fixes #38553

[corlib] Add missing semicolon in StringTest

Merge pull request #2608 from ludovic-henry/monoerror-icall

[runtime] Use MonoError for icall.c functions

Merge pull request #2611 from akoeplinger/cleanup

[corlib] Remove FIXME_CORLIB_CMP section from Makefile

Merge pull request #2612 from xmcclure/largestring

Do not run 4GB string tests on low memory systems (fixes #38392)

"Low memory" here is defined as 32-bit, less than 6GB of physical RAM, or mobile (since mobile lacks the class for querying physical RAM).

Without this patch, one of the corlib tests allocates 4 GB of RAM, which causes OOM crashes on low-memory Jenkins instances and also mobile phones.

Merge pull request #2442 from schani/fix-larger-tlab-size

[automerge] Fix larger tlab size

[jit] Add a checked version to mono_resolve_patch_target () so exceptions are thrown in upper level code.

Revert "[Process] Add stress tests"

This reverts commit a7955a93f5d063feadb4198d8a6f142d81373b79.

The Process import commit was reverted in f74f7037195fe81396edde11c57cea3ec16f03fd,
this reverts the added tests as well since they fail on Jenkins now.

Merge pull request #2573 from BrzVlad/fix-conc-memusage

[sgen] Improve memory usage with the concurrent collector

Merge pull request #2589 from gofman/master

sgen: fix bit counting in mword

Fixes GC crashe for 64-bit architecture when using libmono.
http://lists.ximian.com/pipermail/mono-devel-list/2016-February/043507.html

[perfcounters] Fix calculation of free memory in mono_determine_physical_ram_available_size ()

The implementation added in b5d0ffe7ddba53a6b17587fc43750bf5c0ec2f81 forgot to multiply vmstat.free_count (== number of free pages)
by the page size in the __APPLE__ case, making the reported number much smaller than the reality.

Merge pull request #2585 from lambdageek/dev/bug-38222

[reflection] Signal pending loader error in mono_reflection_get_custom_attrs_data

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

While we're at it, convert `mono_reflection_get_custom_attrs_data` to use
`MonoError` and mark it external only.  Runtime should use
`mono_reflection_get_custom_attrs_data_checked`

Use Assert.Ignore for insufficient-memory test skips

[Crypto] Include CngKeyBlobFormat from reference sources (rm old)

CngKeyBlobFormat was never included in the build, but anyway
we can include the one from referencesource now.

Detected by a user of Npgsql 3.0.5 which reported the problem
in stackoverflow:
http://stackoverflow.com/questions/35016066/failure-to-resolve-system-security-cryptography-cngkeyblobformat-when-using-npgs

Do not run 4GB string tests on low memory systems (fixes #38392)

"Low memory" here is defined as 32-bit, less than 6GB of physical RAM,
or mobile (since mobile lacks the class for querying physical RAM).

[corlib] Remove FIXME_CORLIB_CMP section from Makefile

Looks like this isn't needed anymore.

Merge pull request #2605 from lambdageek/dev/bug-38408

[io] Release Pipe and Console shared handles

Revert "[System] Import System.Diagnostics.Process"

This reverts commit d30e99d172ec5074ce6deb44bc37f9a9ecaef7fe.

[sgen] Allow TLABs larger than MAX_SMALL_OBJ_SIZE.

`sgen_ensure_free_space()` assumed that if `size` was larger than
`SGEN_MAX_SMALL_OBJ_SIZE`, it should find free space on the major
heap, so it wouldn't consider triggering a nursery allocation.  Making
TLABs larger than that would thus never trigger nursery collections
anymore.

Now we pass in the generation that says where we want space.

[sgen] Add necessary type cast.

[runtime] Use MonoError for mono_module_get_types

[runtime] Use MonoError for fill_reflection_assembly_name

[runtime] Use MonoError for create_version

[runtime] Use MonoError for type_from_parsed_name

Merge pull request #2576 from ludovic-henry/referencesource-process

[System] Import System.Diagnostics.Process

Use X509 serial number in Mono cert store filename

The Mono cert store traditionally uses the SKI for the filename of a certificate, but this does not handle cases where a cert has been reissued with the same SKI but different serial (seen in the wild with StartCom certs). This patch appends the serial to all new imported certs, and attempts to interoperate cleanly with existing serial-less certificate stores to avoid duplication.

Closes: #30902

Merge pull request #2604 from mattleibow/pcl-mono.dataconverter

[corlib] Ensure PCL compatibility with Mono.DataConverter

[io] Don't leak Pipe filename

Named FIFOs do indeed have names, but pipe_close doesn't free
it.  (Because CreatePipe doesn't set a filename).

[io] Release Pipe and Console shared handles

Allows opening a character device or named fifo more than once with code like:

  using (var f = new FileStream ("/dev/zero", FileMode.Open))
  {
    ...
  }
  using (var f = new FileStream ("/dev/zero", FileMode.Open))
  {
    ...
  }

[io] Test repeatedly opening a character device

Regression test for https://bugzilla.xamarin.com/show_bug.cgi?id=38408

Merge pull request #2606 from akoeplinger/fix-hardcoded-testports

[System] Fix a few more hardcoded ports in tests

They'd cause "address already in use" errors when two builds are running concurrently on the same machine on Jenkins.

@monojenkins merge

Add link to the updated location for the utf8 test file

[Process] Add stress tests

That's to ensure we do not have regression in the Process implementation.

[System] Import System.Diagnostics.Process

[System] Fix a few more hardcoded ports in tests

They'd cause "address already in use" errors when two builds are running concurrently on the same machine on Jenkins.

[jit] Fix the bblock linking in c5512c0d3e2b102ed16158ce8c599c8203ccd510.

[Mono.Security]: Make 'Mono.Security.Protocol.Ntlm' public and include in System.dll on mobile.

(cherry picked from commit 2026a0b408d122cf3eb2f25cb7355215aa100291)

Remove old, unused code

[corlib] Ensure PCL compatibility with Mono.DataConverter

[runtime] Move pending_exception from InternalThread to Thread to avoid crashing on domain unload.

InternalThread is a fundamentally different type WRT to appdomain. All instances of Thread bound to the same
os thread have a reference to a shared InternalThread.

InternalThread is always allocated in the root domain, so it MUST NOT have references to other domains as
it will lead to crashes on unload.

This was the case with pending_exception. It could point to an exception meant to the domain unloading.

This bug became more and more frequent as we move to use mono_set_pending_exception in more places. I could
reproduce it in 10 minutes of looping appdomain-unload.exe.

Merge pull request #2560 from ludovic-henry/coop-threads-cleanup

[threads] Split abort/suspend into self and async cases

Merge pull request #2587 from lambdageek/monoerror-api-cleanup-mistakes

[runtime] Better error handling in mono_{assembly,module_file}_get_object

Merge pull request #2590 from esdrubal/wcf_req

BCL changes required by dotnet/wcf

Merge pull request #2595 from ludovic-henry/tests-sgen

[tests] Remove manual for loop to run sgen tests and use different make targets

[reflection] Better error behavior for API functions

mono_assembly_get_object and mono_module_file_get_object should return
NULL on failure without raising an exception.