Merge pull request #62 from johntseng/master

fix error when method doesn't have a declaring type
RoushTech · Oct 23, 2017 · 69c73e5 · 69c73e5
2 parents f47ad9f + 6687cce
commit 69c73e5
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/src/RollbarDotNet/Builder/ExceptionBuilder.cs b/src/RollbarDotNet/Builder/ExceptionBuilder.cs
@@ -45,7 +45,12 @@ protected List<Frame> BuildFrames(Exception exception)
         {
             var frames = new List<Frame>();
             var stacktrace = new StackTrace(exception, true);
-            foreach (var stackTraceFrame in stacktrace.GetFrames())
+            var stackTraceFrames = stacktrace.GetFrames();
+            if (stackTraceFrames == null)
+            {
+                return frames;
+            }
+            foreach (var stackTraceFrame in stackTraceFrames)
             {
                 var method = stackTraceFrame.GetMethod();
                 var methodParameters = method.GetParameters();
@@ -54,7 +59,7 @@ protected List<Frame> BuildFrames(Exception exception)
                     : method.GetParameters()
                         .Select(p => $"{p.ParameterType.FullName} {p.Name}")
                         .Aggregate((p1, p2) => $"{p1}, {p2}");
-                var methodName = $"{method.DeclaringType.FullName}.{method.Name}({parameters})";
+                var methodName = $"{method.DeclaringType?.FullName ?? "(unknown)"}.{method.Name}({parameters})";
                 var frame = new Frame
                 {
                     Filename = stackTraceFrame.GetFileName(),