Merge pull request #177 from saran2020/patch-1

Fix cursor.close() not getting called for empty cursor
klinker41 · Sep 11, 2020 · f6fa7d3 · f6fa7d3
2 parents 5b530a8 + 3a50d40
commit f6fa7d3
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/library/src/main/java/com/klinker/android/send_message/Transaction.java b/library/src/main/java/com/klinker/android/send_message/Transaction.java
@@ -246,8 +246,10 @@ private void sendSmsMessage(String text, String[] addresses, long threadId, int
                 Log.v("send_transaction", "inserted to uri: " + messageUri);
 
                 Cursor query = context.getContentResolver().query(messageUri, new String[] {"_id"}, null, null, null);
-                if (query != null && query.moveToFirst()) {
-                    messageId = query.getInt(0);
+                if (query != null) {
+                    if (query.moveToFirst()) {
+                        messageId = query.getInt(0);
+                    }
                     query.close();
                 }