Release notes

CHANGELOG.md

@@ -1,5 +1,16 @@
 # Backtrace Unity Release Notes
 
+## Version 3.9.1
+
+Bugfixes
+- Fixed potential overflow that can happen during breadcrumb deletion flow (#220)
+- Added proguard support for Android uncaught exception captured by the Unity uncaught exception handler (#221)
+
+Improvements
+- Native crash reporting binary upgrade - PlCrashReporter upgrade from 1.7.2 to 1.11.2 (#222)
+- iOS crash handler: store client attributes in the PlCrashReporter crashData object.
+- iOS native attributes: Renaming process.vm.* attributes to vm.*
+
 ## Version 3.9.0
 
 Bugfixes

Runtime/BacktraceClient.cs

@@ -24,7 +24,7 @@ namespace Backtrace.Unity
     /// </summary>
     public class BacktraceClient : MonoBehaviour, IBacktraceClient
     {
-        public const string VERSION = "3.9.0";
+        public const string VERSION = "3.9.1";
         internal const string DefaultBacktraceGameObjectName = "BacktraceClient";
         public BacktraceConfiguration Configuration;
 
@@ -1146,8 +1146,14 @@ internal void HandleUnityMessage(string message, string stackTrace, LogType type
                     Type = type
                 };
             }
+            var report = new BacktraceReport(exception);
+#if UNITY_ANDROID
+            if(exception.NativeStackTrace && _useProguard) {
+                report.UseSymbolication("proguard");
+            }
+#endif
 
-            SendUnhandledExceptionReport(new BacktraceReport(exception), invokeSkipApi);
+            SendUnhandledExceptionReport(report, invokeSkipApi);
         }
 
         /// <summary>

Runtime/Model/BacktraceUnhandledException.cs

@@ -48,6 +48,14 @@ public override string StackTrace
         /// </summary>
         public readonly List<BacktraceStackFrame> StackFrames;
 
+        /// <summary>
+        /// Returns information if the stack trace is from the native environment (non-Unity)
+        /// </summary>
+        internal bool NativeStackTrace
+        {
+            get;
+            private set;
+        }
 
         public BacktraceUnhandledException(string message, string stacktrace) : base(message)
         {
@@ -172,6 +180,8 @@ private BacktraceStackFrame ConvertFrame(string frameString, int methodNameEndIn
                     FunctionName = frameString
                 };
             }
+
+            NativeStackTrace = true;
             // add length of the '('
             methodStartIndex += 1;
             var methodArguments = frameString.Substring(methodStartIndex, methodNameEndIndex - methodStartIndex);
@@ -250,7 +260,8 @@ private BacktraceStackFrame SetJITStackTraceInformation(string frameString)
                 {
                     stackFrame.Library = stackFrame.FunctionName.Substring(0, libraryNameSeparator).Trim();
                     stackFrame.FunctionName = stackFrame.FunctionName.Substring(++libraryNameSeparator).Trim();
-                } else
+                }
+                else
                 {
                     stackFrame.Library = "native";
                 }

Runtime/Model/Breadcrumbs/Storage/BacktraceStorageLogManager.cs

@@ -290,6 +290,11 @@ private long GetNextStartPosition()
             int nextLineBytes = NewRow.Length;
             while (numberOfFreeBytes < expectedFreedBytes)
             {
+                if (_logSize.Count == 0)
+                {
+                    return numberOfFreeBytes;
+                }
+
                 numberOfFreeBytes += (_logSize.Dequeue() + nextLineBytes);
             }
 

Tests/Runtime/BacktraceStackTraceTests.cs

@@ -345,6 +345,23 @@ public void TestStackTraceCreation_AndroidException_ValidStackTraceObject()
             }
         }
 
+        [Test]
+        public void TestNativeStackTraceDetection_AndroidExceptionShouldSetFlag_NativeStackTraceIsSet()
+        {
+            var stackTrace = ConvertStackTraceToString(_anrStackTrace);
+            var exception = new BacktraceUnhandledException(string.Empty, stackTrace);
+            Assert.IsTrue(exception.NativeStackTrace);
+        }
+
+
+        [Test]
+        public void TestNativeStackTraceDetection_UnityExceptionShouldNotSetFlag_NativeStackTraceIsNotSet()
+        {
+            var stackTrace = ConvertStackTraceToString(_simpleStack);
+            var exception = new BacktraceUnhandledException(string.Empty, stackTrace);
+            Assert.IsFalse(exception.NativeStackTrace);
+        }
+
 
         [Test]
         public void TestStackTraceCreation_AndroidMixModeCallStack_ValidStackTraceObject()

package.json

@@ -1,7 +1,7 @@
 {
   "name": "io.backtrace.unity",
   "displayName": "Backtrace",
-  "version": "3.9.0",
+  "version": "3.9.1",
   "unity": "2017.1",
   "description": "Backtrace's integration with Unity games allows customers to capture and report handled and unhandled Unity exceptions to their Backtrace instance, instantly offering the ability to prioritize and debug software errors.",
   "keywords": [