In System:
authorRobert Jordan <robertj@gmx.net>
Wed, 7 May 2008 14:00:30 +0000 (14:00 -0000)
committerRobert Jordan <robertj@gmx.net>
Wed, 7 May 2008 14:00:30 +0000 (14:00 -0000)
2008-05-07  Robert Jordan  <robertj@gmx.net>

* RuntimeFieldHandle.cs, RuntimeTypeHandle.cs, RuntimeMethodHandle.cs:
Don't try to serialize uninitialized handles. Fixes #386641.

In .:
2008-05-07  Robert Jordan  <robertj@gmx.net>

* corlib_test.dll.sources: Add System/Runtime*HandleTest.cs

In Test/System:
2008-05-07  Robert Jordan  <robertj@gmx.net>

* Runtime*Handle.cs: Add serialization tests.

svn path=/trunk/mcs/; revision=102727

mcs/class/corlib/ChangeLog
mcs/class/corlib/System/ChangeLog
mcs/class/corlib/System/RuntimeFieldHandle.cs
mcs/class/corlib/System/RuntimeMethodHandle.cs
mcs/class/corlib/System/RuntimeTypeHandle.cs
mcs/class/corlib/Test/System/ChangeLog
mcs/class/corlib/Test/System/RuntimeFieldHandleTest.cs [new file with mode: 0644]
mcs/class/corlib/Test/System/RuntimeMethodHandleTest.cs
mcs/class/corlib/Test/System/RuntimeTypeHandleTest.cs [new file with mode: 0644]
mcs/class/corlib/corlib_test.dll.sources

index 7f741a578da199bf607f96b9b992f3dfc7ba340f..bae6a9f69923bab2171ed007d316f0848499b642 100644 (file)
@@ -1,3 +1,7 @@
+2008-05-07  Robert Jordan  <robertj@gmx.net>
+
+       * corlib_test.dll.sources: Add System/Runtime*HandleTest.cs
+
 2008-05-07  Sebastien Pouliot  <sebastien@ximian.com>
 
        * corlib.dll.sources: Remove System.IO/Check[Argument|Permission].cs
index a4752dc61ea60bec436cba87613e30c871007788..9dc99713c8829531a754fdb57579a121244c8582 100644 (file)
@@ -1,3 +1,8 @@
+2008-05-07  Robert Jordan  <robertj@gmx.net>
+
+       * RuntimeFieldHandle.cs, RuntimeTypeHandle.cs, RuntimeMethodHandle.cs:
+       Don't try to serialize uninitialized handles. Fixes #386641.
+
 2008-05-06  Marek Safar  <marek.safar@gmail.com>
 
        * IntPtr.cs (eplicit long, GetObjectData): Use ToInt64.
index c796f588f415088aa6ef92cb7c941dca93b35677..c2e88355d819846e4a263727df32281f48e6e705 100644 (file)
@@ -77,6 +77,9 @@ namespace System
                        if (info == null)
                                throw new ArgumentNullException ("info");
 
+                       if (value == IntPtr.Zero)
+                               throw new SerializationException ("Object fields may not be properly initialized");
+
                        info.AddValue ("FieldObj", (MonoField) FieldInfo.GetFieldFromHandle (this), typeof (MonoField));
                }
 
index bff94af25a8a51815e0bf7430856adbc0015fe2f..8f2266cfe36c6b67bf2a0ee46374f42764059043 100644 (file)
@@ -77,6 +77,9 @@ namespace System
                        if (info == null)
                                throw new ArgumentNullException ("info");
 
+                       if (value == IntPtr.Zero)
+                               throw new SerializationException ("Object fields may not be properly initialized");
+
                        info.AddValue ("MethodObj", (MonoMethod) MethodBase.GetMethodFromHandle (this), typeof (MonoMethod));
                }
 
index f4f8721541d8046ccd0c19d1459e1ed6df27a45b..20c0cf9b448df0ed41f2b628c972b244ed3e43c9 100644 (file)
@@ -76,6 +76,9 @@ namespace System
                        if (info == null)
                                throw new ArgumentNullException ("info");
 
+                       if (value == IntPtr.Zero)
+                               throw new SerializationException ("Object fields may not be properly initialized");
+
                        info.AddValue ("TypeObj", Type.GetTypeHandle (this), typeof (MonoType));
                }
 
@@ -122,7 +125,14 @@ namespace System
 
                [CLSCompliant (false)]
                [ReliabilityContractAttribute (Consistency.WillNotCorruptState, Cer.Success)]
-               public ModuleHandle GetModuleHandle () {
+               public ModuleHandle GetModuleHandle ()
+               {
+                       // Although MS' runtime is crashing here, we prefer throwing an exception.
+                       // The check is needed because Type.GetTypeFromHandle returns null
+                       // for zero handles.
+                       if (value == IntPtr.Zero)
+                               throw new InvalidOperationException ("Object fields may not be properly initialized");
+
                        return Type.GetTypeFromHandle (this).Module.ModuleHandle;
                }
 #endif
index 61e3b505172feea88ff0103e3b766bf141ca0621..7bc42dbad698f11c890198098d069635efeef92e 100644 (file)
@@ -1,3 +1,7 @@
+2008-05-07  Robert Jordan  <robertj@gmx.net>
+
+       * Runtime*Handle.cs: Add serialization tests.
+
 2008-05-02  Atsushi Enomoto  <atsushi@ximian.com>
 
        * ConvertTest.cs : added test for Convert.FromBase64String()
diff --git a/mcs/class/corlib/Test/System/RuntimeFieldHandleTest.cs b/mcs/class/corlib/Test/System/RuntimeFieldHandleTest.cs
new file mode 100644 (file)
index 0000000..f28a0b0
--- /dev/null
@@ -0,0 +1,49 @@
+//
+// RuntimeFieldHandleTest.cs - Unit tests for System.RuntimeFieldHandle
+//
+// Author:
+//     Robert Jordan  <robertj@gmx.net>
+//
+// Copyright (C) 2008 Novell, Inc. (http://www.novell.com)
+//
+// Permission is hereby granted, free of charge, to any person obtaining
+// a copy of this software and associated documentation files (the
+// "Software"), to deal in the Software without restriction, including
+// without limitation the rights to use, copy, modify, merge, publish,
+// distribute, sublicense, and/or sell copies of the Software, and to
+// permit persons to whom the Software is furnished to do so, subject to
+// the following conditions:
+// 
+// The above copyright notice and this permission notice shall be
+// included in all copies or substantial portions of the Software.
+// 
+// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+// EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+// MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+// NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+// LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+// OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+// WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+//
+
+using System;
+using System.IO;
+using System.Runtime.Serialization;
+using System.Runtime.Serialization.Formatters.Binary;
+
+using NUnit.Framework;
+
+namespace MonoTests.System
+{
+       [TestFixture]
+       public class RuntimeFieldHandleTest
+       {
+               [Test]
+               [ExpectedException (typeof (SerializationException))]
+               public void Serialization_Of_Empty_Handle ()
+               {
+                       RuntimeFieldHandle handle = new RuntimeFieldHandle ();
+                       new BinaryFormatter ().Serialize (Stream.Null, handle);
+               }
+       }
+}
index 124f9422e0a08de0d856894f9a3b3370438d390f..e7c1f70badfc44191043841bcb25a9e837e6f4ed 100644 (file)
@@ -27,6 +27,9 @@
 //
 
 using System;
+using System.IO;
+using System.Runtime.Serialization;
+using System.Runtime.Serialization.Formatters.Binary;
 
 using NUnit.Framework;
 
@@ -98,5 +101,13 @@ namespace MonoTests.System
                        Assert.IsTrue (rmhB1 != rmhA1, "#B6");
                }
 #endif
+
+               [Test]
+               [ExpectedException (typeof (SerializationException))]
+               public void Serialization_Of_Empty_Handle ()
+               {
+                       RuntimeMethodHandle handle = new RuntimeMethodHandle ();
+                       new BinaryFormatter ().Serialize (Stream.Null, handle);
+               }
        }
 }
diff --git a/mcs/class/corlib/Test/System/RuntimeTypeHandleTest.cs b/mcs/class/corlib/Test/System/RuntimeTypeHandleTest.cs
new file mode 100644 (file)
index 0000000..0405b75
--- /dev/null
@@ -0,0 +1,60 @@
+//
+// RuntimeTypeHandleTest.cs - Unit tests for System.RuntimeTypeHandle
+//
+// Author:
+//     Robert Jordan  <robertj@gmx.net>
+//
+// Copyright (C) 2008 Novell, Inc. (http://www.novell.com)
+//
+// Permission is hereby granted, free of charge, to any person obtaining
+// a copy of this software and associated documentation files (the
+// "Software"), to deal in the Software without restriction, including
+// without limitation the rights to use, copy, modify, merge, publish,
+// distribute, sublicense, and/or sell copies of the Software, and to
+// permit persons to whom the Software is furnished to do so, subject to
+// the following conditions:
+// 
+// The above copyright notice and this permission notice shall be
+// included in all copies or substantial portions of the Software.
+// 
+// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+// EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+// MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+// NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+// LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+// OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+// WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+//
+
+using System;
+using System.IO;
+using System.Runtime.Serialization;
+using System.Runtime.Serialization.Formatters.Binary;
+
+using NUnit.Framework;
+
+namespace MonoTests.System
+{
+       [TestFixture]
+       public class RuntimeTypeHandleTest
+       {
+               [Test]
+               [ExpectedException (typeof (SerializationException))]
+               public void Serialization_Of_Empty_Handle ()
+               {
+                       RuntimeTypeHandle handle = new RuntimeTypeHandle ();
+                       new BinaryFormatter ().Serialize (Stream.Null, handle);
+               }
+
+#if NET_2_0
+               [Test]
+               [ExpectedException (typeof (InvalidOperationException))]
+               [Category ("NotDotNet")] // it crashes the runtime on MS.NET
+               public void GetModuleHandle_Of_Empty_Handle ()
+               {
+                       RuntimeTypeHandle handle = new RuntimeTypeHandle ();
+                       handle.GetModuleHandle ();
+               }
+#endif
+       }
+}
index 7861565ee3e5b041381bc4aebcb5a14860f5c3fa..44d2b6ba86028489eb7ef8dd0bf4dc5b7e6db730 100644 (file)
@@ -358,8 +358,10 @@ System/ConsoleCas.cs
 System/EnvironmentCas.cs
 System/ExceptionCas.cs
 System/MarshalByRefObjectCas.cs
+System/RuntimeFieldHandleTest.cs
 System/RuntimeMethodHandleCas.cs
 System/RuntimeMethodHandleTest.cs
+System/RuntimeTypeHandleTest.cs
 System/TypedReferenceCas.cs
 System.Diagnostics/StackFrameCas.cs
 System.Diagnostics/StackTraceCas.cs