Merge branch 'bugfix'

* bugfix:
  [asp.net] Part of fix for bug #579837. AuthorizationRule.SerializeElement will not process unmodified elements.
  [configuration] Part of fix for bug #579837. CommaDelimitedStringCollection must properly note that it was modified.

[asp.net] Part of fix for bug #579837. AuthorizationRule.SerializeElement will not process unmodified elements.

* AuthorizationRule.cs: if the configuration save is not Full, SerializeElement checks whether the rule being
  serialized is modified. If it isn't, validation and serialization are not performed.
* AuthorizationRuleCollection.cs: there is no element name for the collection.

[configuration] Part of fix for bug #579837. CommaDelimitedStringCollection must properly note that it was modified.

When one manipulates the collection as an instance of StringCollection, the 'modified' field will not
be set (Add, Remove etc. from the underlying StringCollection type will be used). To work around it
a check has to be performed in IsModified whether the contents of the collection changed. It is done
by comparing string hash codes of the original vs current string.
CommaDelimitedStringCollectionConverter forces string hash code update on the new collection, so that IsModified
works correctly on collections converted from comma-delimited strings.

2010-10-01  Veerapuram Varadhan  <v.varadhan@gmail.com>

** Fixes #582732 - Patch by Jeffrey Alvarez <kuritsu@gmail.com>
* DataColumn.cs (Clone): Deep copy the ExtendedProperties
collection.
* XmlSchemaDataImporter.cs: Support import of extended
properties.
* XmlSchemaWriter.cs: Fix exception by writing the extended
properties before simpletype element.

Merge branch 'master' of github.com:mono/mono into masterwork

2010-10-01  Veerapuram Varadhan  <vvaradhan@novell.com>

** Fixes #561667
* SqlCommand.cs (Dispose): While Disposing, set connection's
datareader to null so the connection can be reused with another datareader.

Handle Initialization correctly.

Set have_response

Set have_response to true when there's a protocol error or a previous
exception for POST/PUT.
Patch by Bassam Tabbara.

Check no longer needed.

The is_* function takes care of this check now.

Get constructor arguments (x:Arguments) working on XamlXmlWriter and XamlObjectReader.

Fix line endings.

Execute some specialized tests using the normal test infrastructure.

Fix the handling of varargs in g_set_error ().

Fixed conding style and added license/copyright header to new files

[FIX] Fixes https://bugzilla.novell.com/show_bug.cgi?id=643368 (Transaction.TransactionCompleted not firing

[API] Implemented System.Transactions (tx preparation) timeout handling

Check class names in a better way

Verify image, nesting status, namespace and class for the types
used in threadpool.c

Retval is not a reserved keyword, fixes 643345

remove extraneous reference to sys.configuration.

A couple of fixes around XamlSchemaContext.GetXamlType(XamlTypeName) and get XamlXmlReader pass with unknown type names.

The EndPoint cannot be null

The EndPoint argument in Begin/EndSendTo cannot be null.

Create only 1 callback instance for read/write

Read/Write were creating one callback instance per call. Now we keep
the same delegate around for all the calls.

Handle error in BeginAcceptReceive

We were ignore possible errors when reading after a socket has been
accepted.

Schedule the user callback from unmanaged code

When possible, schedule the user callback from unmanaged code. We pass
it from the IO pool to the regular pool after our internal callback is
finished.

Check the debugger_thread_exited condition in a loop to guard against spurious wakeups. Fixes #642780.

ref/out arguments are always of static type

Keep imported indexer parameters (they can contain non-inflated type parameters)

Implement compound assignment over null value

Work around Android's pthread/kernel_id/fork bug to fix mono multithreading and GC_stop_world in processes that have been forked.

Fix a crash when using the embedding api

error_msg is null when passed into this function from a few places, so
lets not do a null deref here.

Fix code buffer reallocation check for OP_SWITCH on arm.

2010-09-29  Veerapuram Varadhan  <vvaradhan@novell.com>

** Fixes failed test for smallmoney
* Tds70.cs (WriteParameterInfo): Round money values to 4
decimals, for smallmoney too.

New test

Dynamic compound assignment breaks standard C# result conversion rules

void is never of type object

Null constant cannot be used for ref/out variables

Initialize both disposable and local variable for dynamic expressions.

rename source file.

Remove old code.

Fix a deadlock introduced by 5c2e3bf7 by using a separate low level lock to procect access to the file_share_hash hash table. Fixes #642985.

[wcf] serialization member namespace was wrong when root namesp ace is given. Fixed possible infinite loop during DefaultTypeMap initialization.

MS Basic/Contract/Data/POCO sample got working fine.

Remove extra call to HttpResponse.Flush() which would crash.

[asp.net] Handle closed responses gracefully.

If ReleaseResources or Flush (true) were called, a flag is set that causes
any subsequent Flush () calls to fail with an exception.
At the same time, ReleaseResources does not destroy the output stream completely -
it has to remain around to match .NET behavior.

Revert "[asp.net] Make sure output_stream is not null before using it"

This reverts commit a4ed8be6973dcfede7131f080340220b3ce4830b since what we
do in ReleaseResources is not what .NET does - a better fix will follow.

[Json] Fix uint type conversion when reading, should be mapped to long, not int

Avoid a few warnings

Add DataContractResolver stuff (dummy), and fix Id reference from inside the referenced object.

More awkward nullable binary expressions which are not lifted even though they should be

Implement binary user operators over nullable not lifted expressions.

Dynamic compound assignment can be used as an expression as well.

Invoke the callback in separate threads

The callbacks need to be invoked in different threads to avoid clogging
the IO pool.

Fixes bug #639656.

Disable the freeing of monitors again, it still crashes with sgen when running tests/async_read.exe.

Fix AOT support for delegates

Fix buffer checks on Socket.[Receive|Send]Async

* System.Net.Sockets/Socket_2_1.cs: Fix, by explicitely throwing an NRE,
the checks in [Receive|Send]Async methods - otherwise an exception would
occur later, on another thread and close the socket (leading to other,
unexpected, exceptions)

* Test/System.Net.Sockets/SocketTest.cs: Add parameter validation test
cases for both ReceiveAsync and SendAsync

[moon] Move policy restriction / elevated trust check outside of loop

* System.Net.Sockets/SocketAsyncEventArgs.cs: Move policy restriction
and elevated trust check outside of loop since its value won't change
based on the ipaddress

Fix "4762b18c: Fix the moonlight build"

* System.Net.Sockets/Socket_2_1.cs: Do not #id-def out [Receive|Send]
Generic from moonlight build

Fix arm and ppc builds

Handle error creating xdb.il

If creating the xdb.il file fails, disable the mono support for
debugging in gdb. Managed symbol names won't be displayed in stack
traces.

Fixes bug #642625.

Remove "NotWorking" from a test that now works.

ndle static methods with bound first arg.

* marshal.c (mono_marshal_get_delegate_invoke): Properly
handle a static method that has it's first argument bound.

* marshal.c (mono_marshal_free_inflated_wrappers): Free
from the new hashtable too.

* image.c (mono_image_close_except_pools): Free the new
hash table.

* metadata-internals.h (MonoImage): Add new hashtable
to store the new kind of invoke wrappers.

Add missing members.

[ReaderWriterLockSlim] Fix state corruption and race in a special interleaving.

Fix possible state corruption in the case where Upgradeable wasn't reactivating its read bit after a downgrade and set events accordingly. Fix possible race where a write process wannabe could step in the middle of Upgradeable actually upgrading to write mode.

Add some missing StructLayoutAttributes.

[wcf-discovery] ExclusiveAddressUse won't work. AcceptChannel() should nreturn null when it is being closed.

[wcf-discovery] Fix MessageEncoder retrieval, failure on EndReceive and ExclusiveAddressUse.

[wcf discovery] Use appropriate MessageVersion defined in DiscoveryVersion.

[wcf] Make sure context exists in the dictionary getting its associated wait handle.

SvcHttpHandler maintains a mapping of HttpContext to ManualResetEvent. Entries are
removed from the cache in EndHttpRequest. The code makes sure the context exists in
the cache before attempting to access it.

Merge branch 'bugfix'

* bugfix:
  [asp.net] Make sure output_stream is not null before using it

[asp.net] Make sure output_stream is not null before using it

It is possible that some code will still attempt to access the HttpResponse instance
after ReleaseResources got called. This patch prevents NREX in such cases.

[System.ServiceModel] Fix a few potential exceptions in background threads

Pass the listener as it's own async state so that we avoid potential null derefs in the callback. Also wrap the BeginAccept and EndAccept calls in their own try/catches in case they throw an exception.

2010-09-20  Veerapuram Varadhan  <vvaradhan@novell.com>

* SqlDataReaderTest.cs: Add test for bug#613087

test fixes

New test.

[642437] Nested types have to check type arguments only and not full class

New test.

Skip inaccessible nested types for member access expressions.

Inflate custom site container delegates correctly.

Optional parameters can be of a user value type

Ignore imported not visible internal types

Fix the issue that callback contract often gets confused by picking unexpected contract type.

The problem is that DispatchRuntime.Type is a service implementation type
that could implement two or more contract interfaces, and it was in random
order how ServiceProxyGenerator picks up the contract type. Now it looks
for correct contract type instead of the service type by exactly looking
for the target contract in the endpoint (dispatcher).

Allow mono_install_assembly_preload_hook() to provide mscorlib.dll

In some embedding scenarios, mscorlib.dll may be present in a non-standard
location (outside of MONO_PATH) or with an alternate filename.

Use the installed assembly preload hooks to search for mscorlib.dll before
falling back to the default lookup strategy.

Fix 'thread may have been prematurely finalized' warnings when a GC happens during thread attach in sgen.

Remap more eglib symbols

[asp.net] Remove lock recursion in Cache.ExpireItems

TCP channel listener: remove extra complication (threads) in the previous change.

2010-09-16  Veerapuram Varadhan  <vvaradhan@novell.com>

* SqlCommandTest.cs: Add test for bug#569543

2010-09-16  Veerapuram Varadhan  <vvaradhan@novell.com>

* SqlCommandTest.cs: Add test for bug#584833

Rework TCP listening model, and fix AcceptTcpClient() to not drop timed out requests that actually steals next requests in the air.

WebResponse.Headers is not always available (e.g. moonlight browser stack)

* System.ServiceModel.Channels/HttpRequestChannel.cs: For Moonlight use
the WebResponse.SupportsHeaders before accessing the Headers collection.
This is needed because (the default) browser stack does not expose the
headers from its responses. Fix bug #642193

Turn Is*LockHeld properties into thread-specific values in ReaderWriterLockSlim. Add corresponding unit tests.

[asp.net] Get rid of a possible (although unlikely) race condition when acquiring locks

The patch removes a small race condition where a boolean flag is set after acquiring a lock to
indicate to code executing in the finally {} block that it should release the lock. The boolean
variable is now removed and the lock is released unconditionally. It carries a potential to
throw an exception when the lock is not held, but it's better than to fail to release it and
lead the application to a deadlock.

Avoid a culture-specific check for the end of an URI

* System/Uri.cs: Avoid the culture-specific String.EndsWith when
reducing an URI

Use same socket family as the endpoint when retrieving the policy

* System.Net.Policy/CrossDomainPolicyManager.cs: Use same socket family
as the endpoint when retrieving the policy. Also do not try to create a
policy out of an empty stream (avoid misleading exceptions in logs)

Fix Socket.RemoteEndPoint after ConnectAsync on Moonlight

* System.Net.Sockets/SocketAsyncEventArgs.cs: Set the Socket
RemoteEndPoint to our RemoteEndPoint on Connect operation.
* System.Net.Sockets/Socket_2_1.cs: Return the 'candidate' EndPoint
provided by SocketAsyncEventArgs in ConnectAsync (intead of null)
until the connection is completed.

[wcf] Make simplified configuration for behaviors work (4.0)

* NamedServiceModelExtensionCollectionElement.cs: minimum length of the Name property value is 0 for 4.0
* ServiceBehaviorElementCollection.cs: added an override of BaseAdd which makes sure that the element being
  added has its Name set to String.Empty if the element's name is null
* ServiceHostBase.cs: support for simplified behavior configuration. Refactored behavior addition code to a
  separate method. Check whether behavior is not in the collection before adding it.

Add runtime test for delegates with null target.

Add comment on the reason of a non obvious check.

Enable test that has been fixed in the runtime.

Raise ArgumentException for delegate on closed methods.

* method-to-ir.c (mono_method_to_ir): Check for null
on delegate construction so closed-on-null is not allowed
when using the delegate .ctor.

If the delegate been constructed is open-on-target then
the null check must not be applied as the IL pattern is:

ldnull
ldftn void Bla::Foo(int)
newobj Bar (object, native int) // delegate void Bar(Bla,int)

Mov the jit map generation code from tramp-x86.c to mini-trampolines.c, add amd64 support.