Core: Fix exception when converting InjectedParameterAnalysisContext …

…to AsmResolver
SamboyCoding · Aug 31, 2024 · 4184388 · 4184388
1 parent 87c61bd
commit 4184388
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/Cpp2IL.Core/Model/Contexts/InjectedParameterAnalysisContext.cs b/Cpp2IL.Core/Model/Contexts/InjectedParameterAnalysisContext.cs
@@ -1,3 +1,4 @@
+using System.Reflection;
 using Cpp2IL.Core.Utils;
 using LibCpp2IL.BinaryStructures;
 
@@ -7,7 +8,9 @@ public class InjectedParameterAnalysisContext : ParameterAnalysisContext
 {
     public override TypeAnalysisContext ParameterTypeContext { get; }
 
-    public override bool IsRef => false; //For now
+    public override bool IsRef => ParameterTypeContext is ReferencedTypeAnalysisContext;
+
+    public override ParameterAttributes ParameterAttributes => ParameterAttributes.None;
 
     public InjectedParameterAnalysisContext(string? name, Il2CppType type, int paramIndex, MethodAnalysisContext declaringMethod)
         : this(name, declaringMethod.DeclaringType!.DeclaringAssembly.ResolveIl2CppType(type) ?? throw new($"Type {type} could not be resolved."), paramIndex, declaringMethod)

diff --git a/Cpp2IL.Core/Model/Contexts/ParameterAnalysisContext.cs b/Cpp2IL.Core/Model/Contexts/ParameterAnalysisContext.cs
@@ -47,7 +47,7 @@ public class ParameterAnalysisContext : HasCustomAttributesAndName, IParameterIn
     /// <summary>
     /// The ParameterAttributes of this parameter.
     /// </summary>
-    public ParameterAttributes ParameterAttributes => (ParameterAttributes)ParameterType.Attrs;
+    public virtual ParameterAttributes ParameterAttributes => (ParameterAttributes)ParameterType.Attrs;
 
     /// <summary>
     /// True if this parameter is passed by reference.