Fix Type.IsPublic for pointer and byref types

src/coreclr/vm/runtimehandles.cpp

@@ -806,14 +806,8 @@ FCIMPL1(INT32, RuntimeTypeHandle::GetAttributes, ReflectClassBaseObject *pTypeUN
 
     TypeHandle typeHandle = refType->GetType();
 
-    if (typeHandle.IsTypeDesc()) {
-
-        if (typeHandle.IsGenericVariable()) {
-            return tdPublic;
-        }
-
-        return 0;
-    }
+    if (typeHandle.IsTypeDesc())
+        return typeHandle.GetElementTypeAttributes();
 
 #ifdef FEATURE_COMINTEROP
     // __ComObject types are always public.

src/coreclr/vm/typehandle.cpp

@@ -244,6 +244,26 @@ TypeHandle TypeHandle::GetTypeParam() const
     return TypeHandle();
 }
 
+// Obtain attributes of the element type from a byref or pointer type, returns tdPublic for generic type variables, 0 otherwise
+INT32 TypeHandle::GetElementTypeAttributes() const
+{
+    LIMITED_METHOD_DAC_CONTRACT;
+
+    if (IsGenericVariable())
+        return tdPublic;
+
+    if (IsPointer() || IsByRef()) {
+        TypeHandle elementType = GetTypeParam();
+
+        if (elementType.IsTypeDesc())
+            return elementType.GetElementTypeAttributes();
+
+        return (INT32)elementType.GetMethodTable()->GetAttrClass();
+    }
+
+    return 0;
+}
+
 #ifndef DACCESS_COMPILE
 TypeHandle TypeHandle::Instantiate(Instantiation inst) const
 {

src/coreclr/vm/typehandle.h

@@ -285,6 +285,9 @@ class TypeHandle
     // Obtain element type for an array, byref or pointer, returning NULL otherwise
     TypeHandle GetTypeParam() const;
 
+    // Obtain attributes of the element type from a byref or pointer type, returns tdPublic for generic type variables, 0 otherwise
+    INT32 GetElementTypeAttributes() const;
+
     // Obtain instantiation from an instantiated type
     // NULL if not instantiated
     Instantiation GetInstantiation() const;

src/libraries/System.Reflection/tests/TypeInfoTests.cs

@@ -1173,6 +1173,36 @@ public void MakePointerType(Type type)
             Assert.True(pointerType.IsPointer);
         }
 
+        public static IEnumerable<object[]> TypeDescTypesIsPublic_TestData()
+        {
+            yield return new object[] { typeof(int).MakeByRefType(), true };
+            yield return new object[] { typeof(int).MakePointerType(), true };
+            yield return new object[] { typeof(int).MakeArrayType(), true };
+            yield return new object[] { typeof(TI_BaseClass.InternalNestedClass).MakeByRefType(), false };
+            yield return new object[] { typeof(TI_BaseClass.InternalNestedClass).MakePointerType(), false };
+            yield return new object[] { typeof(TI_BaseClass.InternalNestedClass).MakeArrayType(), true };
+            yield return new object[] { typeof(TI_BaseClass.InternalNestedGenericClass<>).MakeGenericType(typeof(string)), false };
+            yield return new object[] { typeof(TI_BaseClass).MakeByRefType(), true };
+            yield return new object[] { typeof(TI_BaseClass).MakePointerType(), true };
+            yield return new object[] { typeof(TI_BaseClass).MakeArrayType(), true };
+        }
+
+        [Theory]
+        [MemberData(nameof(TypeDescTypesIsPublic_TestData))]
+        public void TypeDescTypesIsPublic(Type type, bool expected)
+        {
+            Assert.Equal(expected, type.IsPublic);
+            Assert.Equal(!expected, type.IsNestedAssembly);
+        }
+
+        [Fact]
+        public void NestedPointerIsPublic()
+        {
+            Type pointerType = typeof(int***);
+            Assert.True(pointerType.IsPointer);
+            Assert.True(pointerType.IsPublic);
+        }
+
         [Fact]
         public void MakePointerType_TypeAlreadyByRef_ThrowsTypeLoadException()
         {
@@ -1828,6 +1858,7 @@ public class PublicNestedClass1 { }
         public class PublicNestedClass2 { }
         private class PrivateNestedClass { } // Private, so not inherited
         internal class InternalNestedClass { } // Internal members are not inherited
+        internal class InternalNestedGenericClass<T>{ }
         protected class ProtectedNestedClass { }
 
         public string StringProperty1 { get { return ""; } set { } }