Do not serialize FieldInfos in FieldInterceptorObjectReference

The fields should not change between serialization and deserialization.

Fixes nhibernate#1964

Author: hazzik

src/NHibernate.Test/StaticProxyTest/Model.cs

@@ -33,13 +33,19 @@ public class InterfacedEntity : IEntity
 	}
 
 	[Serializable]
-	public class LazyTextEntity
+	public class LazyTextEntity : BaseTextEntity
 	{
 		public virtual Guid Id { get; set; }
 		public virtual string Name { get; set; }
 		public virtual string Text { get; set; }
 	}
 
+	[Serializable]
+	public class BaseTextEntity
+	{
+		public virtual int Test { get; set; }
+	}
+	
 	[Serializable]
 	public class InterfacedLazyTextEntity : ILazyTextEntity
 	{

src/NHibernate/Proxy/FieldInterceptorObjectReference.cs

@@ -17,7 +17,6 @@ public sealed class FieldInterceptorObjectReference : IObjectReference, ISeriali
 		private readonly object _deserializedProxy;
 
 		private const string HasAdditionalDataName = "proxy$hasAdditionalData";
-		private const string AdditionalMemberName = "proxy$additionalMembers";
 
 		public FieldInterceptorObjectReference(NHibernateProxyFactoryInfo proxyFactoryInfo, IFieldInterceptor fieldInterceptorField)
 		{
@@ -30,39 +29,36 @@ private FieldInterceptorObjectReference(SerializationInfo info, StreamingContext
 			_proxyFactoryInfo = info.GetValue<NHibernateProxyFactoryInfo>(nameof(_proxyFactoryInfo));
 			_fieldInterceptor = info.GetValue<IFieldInterceptor>(nameof(_fieldInterceptor));
 
+			var proxy = _proxyFactoryInfo.CreateProxyFactory().GetFieldInterceptionProxy(null);
 			if (info.GetBoolean(HasAdditionalDataName))
 			{
-				_deserializedProxy = _proxyFactoryInfo.CreateProxyFactory().GetFieldInterceptionProxy(null);
-
-				var additionalMembers = info.GetValue<MemberInfo[]>(AdditionalMemberName);
-				if (additionalMembers == null)
-					return;
-
-				foreach (var member in additionalMembers)
+				var members = FormatterServices.GetSerializableMembers(_proxyFactoryInfo.PersistentClass, context);
+				foreach (var member in members)
 				{
 					switch (member)
 					{
 						case FieldInfo field:
 							field.SetValue(
-								_deserializedProxy,
+								proxy,
 								info.GetValue(GetAdditionalMemberName(field), field.FieldType));
 							break;
 						case PropertyInfo property:
 							property.SetValue(
-								_deserializedProxy,
+								proxy,
 								info.GetValue(GetAdditionalMemberName(property), property.PropertyType));
 							break;
 						default:
 							throw new NotSupportedException(
 								$"Deserializing a member of type {member.GetType()} is not supported.");
 					}
 				}
+				_deserializedProxy = proxy;
 			}
 			else
 			{
 				// Base type has a custom serialization, we need to call the proxy deserialization for deserializing
 				// base type members too.
-				var proxyType = _proxyFactoryInfo.CreateProxyFactory().GetFieldInterceptionProxy(null).GetType();
+				var proxyType = proxy.GetType();
 				var deserializationConstructor = proxyType.GetConstructor(
 					BindingFlags.Instance | BindingFlags.NonPublic | BindingFlags.Public,
 					null,
@@ -95,8 +91,6 @@ public void GetBaseData(SerializationInfo info, StreamingContext context, object
 			info.AddValue(HasAdditionalDataName, true);
 
 			var members = FormatterServices.GetSerializableMembers(proxyBaseType, context);
-			info.AddValue(AdditionalMemberName, members);
-
 			foreach (var member in members)
 			{
 				switch (member)

src/NHibernate/Proxy/NHibernateProxyFactoryInfo.cs

@@ -27,6 +27,8 @@ internal NHibernateProxyFactoryInfo(string entityName, System.Type persistentCla
 			_componentIdType = componentIdType;
 		}
 
+		internal System.Type PersistentClass => _persistentClass;
+
 		public IProxyFactory CreateProxyFactory()
 		{
 			var factory = new StaticProxyFactory();