From d311557f099a7a4b3d5f0f05a00bd5ec0813dd1c Mon Sep 17 00:00:00 2001
From: akwizgran <akwizgran@users.sourceforge.net>
Date: Mon, 29 May 2017 10:05:49 +0100
Subject: [PATCH] Check whether first visible message ID is null before
 storing.

---
 .../threaded/ThreadListControllerImpl.java    | 25 ++++++++++---------
 1 file changed, 13 insertions(+), 12 deletions(-)

diff --git a/briar-android/src/main/java/org/briarproject/briar/android/threaded/ThreadListControllerImpl.java b/briar-android/src/main/java/org/briarproject/briar/android/threaded/ThreadListControllerImpl.java
index 0793d785a..e0554dee8 100644
--- a/briar-android/src/main/java/org/briarproject/briar/android/threaded/ThreadListControllerImpl.java
+++ b/briar-android/src/main/java/org/briarproject/briar/android/threaded/ThreadListControllerImpl.java
@@ -99,19 +99,20 @@ public abstract class ThreadListControllerImpl<G extends NamedGroup, I extends T
 
 	@Override
 	public void onActivityDestroy() {
-		dbExecutor.execute(new Runnable() {
-			@Override
-			public void run() {
-				try {
-					messageTracker
-							.storeMessageId(groupId,
-									listener.getFirstVisibleMessageId());
-				} catch (DbException e) {
-					if (LOG.isLoggable(WARNING))
-						LOG.log(WARNING, e.toString(), e);
+		final MessageId first = listener.getFirstVisibleMessageId();
+		if (first != null) {
+			dbExecutor.execute(new Runnable() {
+				@Override
+				public void run() {
+					try {
+						messageTracker.storeMessageId(groupId, first);
+					} catch (DbException e) {
+						if (LOG.isLoggable(WARNING))
+							LOG.log(WARNING, e.toString(), e);
+					}
 				}
-			}
-		});
+			});
+		}
 	}
 
 	@CallSuper