Merge pull request #1500 from criteo-forks/criteo

[mono.git] / mcs / class / corlib / System.Security / SecureString.cs

diff --git a/mcs/class/corlib/System.Security/SecureString.cs b/mcs/class/corlib/System.Security/SecureString.cs
index ba39ce7d5da4c0a1d80e1f80db48c36c238db15a..765f61981df9a2f11fbca50627965f35295e4174 100644 (file)
--- a/mcs/class/corlib/System.Security/SecureString.cs
+++ b/mcs/class/corlib/System.Security/SecureString.cs
@@ -33,11 +33,12 @@ using System.Runtime.InteropServices;
 using System.Runtime.ConstrainedExecution;
 using System.Security.Cryptography;
 using System.Security.Permissions;
+using System.Runtime.ExceptionServices;
 
 namespace System.Security {
 
        [MonoTODO ("work in progress - encryption is missing")]
-       public sealed class SecureString : CriticalFinalizerObject, IDisposable {
+       public sealed class SecureString : IDisposable {
 
                private const int BlockSize = 16;
                private const int MaxSize = 65536;
@@ -88,6 +89,7 @@ namespace System.Security {
                        }
                }
 
+               [HandleProcessCorruptedStateExceptions]
                public void AppendChar (char c)
                {
                        if (disposed)
@@ -144,6 +146,7 @@ namespace System.Security {
                        length = 0;
                }
 
+               [HandleProcessCorruptedStateExceptions]
                public void InsertAt (int index, char c)
                {
                        if (disposed)
@@ -185,6 +188,7 @@ namespace System.Security {
                        read_only = true;
                }
 
+               [HandleProcessCorruptedStateExceptions]
                public void RemoveAt (int index)
                {
                        if (disposed)
@@ -198,7 +202,7 @@ namespace System.Security {
 
                        try {
                                Decrypt ();
-                               Buffer.BlockCopy (data, index + 1, data, index, data.Length - index - 1);
+                               Buffer.BlockCopy (data, index * 2 + 2, data, index * 2, data.Length - index * 2 - 2);
                                Alloc (--length, true);
                        }
                        finally {
@@ -206,6 +210,7 @@ namespace System.Security {
                        }
                }
 
+               [HandleProcessCorruptedStateExceptions]
                public void SetAt (int index, char c)
                {
                        if (disposed)