Feature/crash loop detection

.gitignore

@@ -89,3 +89,5 @@ hs_err_pid*
 obj/
 .externalNativeBuild
 **/.cxx
+
+backtrace-library/src/main/jniLibs/**

backtrace-library/src/main/cpp/backends/backend.cpp

@@ -82,4 +82,29 @@ void Disable() {
                         "Disable not supported on this backend");
 #endif
 }
+
+bool EnableCrashLoopDetectionBackend() {
+#ifdef CRASHPAD_BACKEND
+    return EnableCrashLoopDetectionCrashpad();
+#elif BREAKPAD_BACKEND
+    return false;
+#endif
+}
+
+bool IsSafeModeRequiredBackend() {
+#ifdef CRASHPAD_BACKEND
+    return IsSafeModeRequiredCrashpad();
+#elif BREAKPAD_BACKEND
+    return false;
+#endif
+}
+
+int ConsecutiveCrashesCountBackend() {
+#ifdef CRASHPAD_BACKEND
+    return ConsecutiveCrashesCountCrashpad();
+#elif BREAKPAD_BACKEND
+    return 0;
+#endif
+}
+
 }

backtrace-library/src/main/cpp/backends/crashpad-backend.cpp

@@ -11,6 +11,8 @@ extern std::atomic_bool disabled;
 static crashpad::CrashpadClient *client;
 static std::unique_ptr<crashpad::CrashReportDatabase> database;
 
+static int consecutive_crashes_count = 0;
+
 bool InitializeCrashpad(jstring url,
                         jstring database_path,
                         jstring handler_path,
@@ -19,6 +21,7 @@ bool InitializeCrashpad(jstring url,
                         jobjectArray attachmentPaths,
                         jboolean enableClientSideUnwinding,
                         jint unwindingMode) {
+
     // avoid multi initialization
     if (initialized) {
         __android_log_print(ANDROID_LOG_ERROR, "Backtrace-Android", "Crashpad is already initialized");
@@ -120,9 +123,12 @@ bool InitializeCrashpad(jstring url,
 
     // Start crash handler
     client = new crashpad::CrashpadClient();
+    client->EnableCrashLoopDetection();
 
-    initialized = client->StartHandlerAtCrash(handler, db, db, backtraceUrl, attributes,
-                                              arguments);
+    // Get consecutive crashes count BEFORE any handler started,
+    // as it writes extra line into CSV, what leads to getting 0 for each next ConsecutiveCrashesCount call
+    consecutive_crashes_count = crashpad::CrashpadClient::ConsecutiveCrashesCount(db);
+    initialized = client->StartHandlerAtCrash(handler, db, db, backtraceUrl, attributes, arguments);
 
     env->ReleaseStringUTFChars(url, backtraceUrl);
     env->ReleaseStringUTFChars(handler_path, handlerPath);
@@ -219,10 +225,27 @@ void ReEnableCrashpad() {
     // Re-enable uploads if disabled
     if (disabled) {
         if (database == nullptr) {
-            __android_log_print(ANDROID_LOG_ERROR, "Backtrace-Android", "Crashpad database is null, this should not happen");
+            __android_log_print(ANDROID_LOG_ERROR, "Backtrace-Android",
+                                "Crashpad database is null, this should not happen");
             return;
         }
         database->GetSettings()->SetUploadsEnabled(true);
         disabled = false;
     }
 }
+
+bool EnableCrashLoopDetectionCrashpad() {
+    if (client != nullptr) {
+        return client->EnableCrashLoopDetection();
+    } else {
+        return false;
+    }
+}
+
+bool IsSafeModeRequiredCrashpad() {
+    return consecutive_crashes_count >= 5;
+}
+
+int ConsecutiveCrashesCountCrashpad() {
+    return consecutive_crashes_count;
+}

backtrace-library/src/main/cpp/backtrace-native.cpp

@@ -135,4 +135,19 @@ Java_backtraceio_library_BacktraceDatabase_disable(JNIEnv *env, jobject thiz) {
     Disable();
 }
 
+JNIEXPORT jboolean JNICALL
+Java_backtraceio_library_BacktraceDatabase_EnableCrashLoopDetection(JNIEnv *env, jclass clazz) {
+    return EnableCrashLoopDetectionBackend();
+}
+
+JNIEXPORT jboolean JNICALL
+Java_backtraceio_library_BacktraceDatabase_IsSafeModeRequired(JNIEnv *env, jclass clazz) {
+    return IsSafeModeRequiredBackend();
+}
+
+JNIEXPORT int JNICALL
+Java_backtraceio_library_BacktraceDatabase_ConsecutiveCrashesCount(JNIEnv *env, jclass clazz) {
+    return ConsecutiveCrashesCountBackend();
+}
+
 }

backtrace-library/src/main/cpp/include/backend.h

@@ -19,6 +19,12 @@ void DumpWithoutCrash(jstring message, jboolean set_main_thread_as_faulting_thre
 void AddAttribute(jstring key, jstring value);
 
 void Disable();
+
+bool EnableCrashLoopDetectionBackend();
+
+bool IsSafeModeRequiredBackend();
+
+int ConsecutiveCrashesCountBackend();
 }
 
 #endif //BACKTRACE_ANDROID_BACKEND_H

backtrace-library/src/main/cpp/include/crashpad-backend.h

@@ -28,4 +28,8 @@ void DisableCrashpad();
 
 void ReEnableCrashpad();
 
+bool EnableCrashLoopDetectionCrashpad();
+bool IsSafeModeRequiredCrashpad();
+int ConsecutiveCrashesCountCrashpad();
+
 #endif //BACKTRACE_ANDROID_CRASHPAD_BACKEND_H

backtrace-library/src/main/java/backtraceio/library/BacktraceDatabase.java

@@ -464,4 +464,8 @@ private boolean validateDatabaseSize() {
     public long getDatabaseSize() {
         return backtraceDatabaseContext.getDatabaseSize();
     }
+
+    public static native boolean EnableCrashLoopDetection();
+    public static native boolean IsSafeModeRequired();
+    public static native int ConsecutiveCrashesCount();
 }

example-app/src/main/java/backtraceio/backtraceio/MainActivity.java

@@ -1,5 +1,7 @@
 package backtraceio.backtraceio;
 
+import static backtraceio.backtraceio.BuildConfig.BACKTRACE_SUBMISSION_URL;
+
 import android.content.Context;
 import android.os.Bundle;
 import androidx.appcompat.app.AppCompatActivity;
@@ -27,6 +29,7 @@
 import backtraceio.library.enums.BacktraceBreadcrumbType;
 import backtraceio.library.enums.database.RetryBehavior;
 import backtraceio.library.enums.database.RetryOrder;
+import backtraceio.library.models.BacktraceData;
 import backtraceio.library.models.BacktraceExceptionHandler;
 import backtraceio.library.models.BacktraceMetricsSettings;
 import backtraceio.library.models.database.BacktraceDatabaseSettings;
@@ -49,10 +52,22 @@ protected void onCreate(Bundle savedInstanceState) {
         setContentView(R.layout.activity_main);
 
         // Set this value in your local.properties
-        if (BuildConfig.BACKTRACE_SUBMISSION_URL != null) {
-            backtraceClient = initializeBacktrace(BuildConfig.BACKTRACE_SUBMISSION_URL);
+        if (BACKTRACE_SUBMISSION_URL != null) {
+            backtraceClient = initializeBacktrace(BACKTRACE_SUBMISSION_URL);
         }
 
+        // Crash Loop Detector example
+        boolean isCLSafeModeReq = BacktraceDatabase.IsSafeModeRequired();
+        int crashesCount = BacktraceDatabase.ConsecutiveCrashesCount();
+        Log.i("BacktraceAndroid", "ConsecutiveCrashesCount: " + crashesCount);
+
+        View viewBackground = findViewById(R.id.viewBackground);
+        if(viewBackground != null) {
+            viewBackground.setBackgroundColor(isCLSafeModeReq
+                    ? getResources().getColor(R.color.colorAccent)
+                    : getResources().getColor(R.color.colorWhite));
+        }
+
         symlinkAndWriteFile();
     }
 

example-app/src/main/res/layout/activity_main.xml

@@ -1,11 +1,17 @@
 <?xml version="1.0" encoding="utf-8"?>
-<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
+<androidx.constraintlayout.widget.ConstraintLayout
+    xmlns:android="http://schemas.android.com/apk/res/android"
     xmlns:app="http://schemas.android.com/apk/res-auto"
     xmlns:tools="http://schemas.android.com/tools"
     android:layout_width="match_parent"
     android:layout_height="match_parent"
     tools:context=".MainActivity">
 
+    <View
+        android:id="@+id/viewBackground"
+        android:layout_width="match_parent"
+        android:layout_height="match_parent"
+        android:background="@color/colorWhite"/>
     <Button
         android:id="@+id/handledException"
         android:layout_width="wrap_content"

example-app/src/main/res/values/colors.xml

@@ -3,4 +3,5 @@
     <color name="colorPrimary">#008577</color>
     <color name="colorPrimaryDark">#00574B</color>
     <color name="colorAccent">#D81B60</color>
+    <color name="colorWhite">#FFFFFF</color>
 </resources>