Concurrency Bug about nclone

[davisnw]

There are actually cares where it appears NClone gets stuck in an infinite loop. I'll try to create a minimal example to demonstrate the issue. I think both the NRE and infinite looping are related to the same issue of the internal dictionary not being thread-safe. I believe the issues are likely to arise when NClone is concurrently asked to clone the same type on different threads before the internal cache for that type has been fully constructed.

[davisnw]

The below program (tested targeting .NET Framework 4.5.2 with NClone version 1.1.1 from Nuget) consistently reproduces both the NRE and the infinite looping. The NRE happens almost every time, the infinite looping less often but still frequently enough to see the issue.

using System;
using System.Linq;
using System.Threading.Tasks;

namespace NCloneBug
{
    class Program
    {
        public class ValueObject
        {
            private readonly int _blah;

            private ValueObject() : this(Int32.MinValue){}

            public ValueObject(int blah)
            {
                _blah = blah;
            }

            public static readonly ValueObject Empty = new ValueObject();
        }

        public class Entity
        {

            private ValueObject _foo1 = NClone.Clone.ObjectGraph(ValueObject.Empty);
            public ValueObject Foo1
            {
                get { return _foo1; }
                set {_foo1 = value; }
            }

            private ValueObject _foo3 = NClone.Clone.ObjectGraph(ValueObject.Empty);
            public ValueObject Foo3
            {
                get { return _foo3; }
                set { _foo3 = value; }
            }
        }
  
        static void Main(string[] args)
        {
            Parallel.ForEach(Enumerable.Repeat<Func<Entity>>(() => new Entity(), 20), x =>
            {
                try
                {
                    x();
                }
                catch (Exception e)
                {
                    Console.WriteLine(e.ToString());
                }
            });
        }
    }
}

[mijay]

Thank you for reporting and fixing this one! I'll try to release a new version today.