commit 151f1ad1613ecac0207144472872440517172b5a
author Paul Westbrook <pwestbro@google.com> Tue Apr 24 09:13:00 2012 -0700
committer Paul Westbrook <pwestbro@google.com> Tue Apr 24 09:13:00 2012 -0700
tree ca70c287a2023863053c0d208806010a7b7cbe82
parent ef7c992d8cb462be6da34d42965cf185f3bb1b22

Fix index out of bounds exception

Bug: 6371081
Change-Id: I7f0315998b0f62a1a0c4dae8541abce6c139ec75

src/com/android/mail/compose/ComposeActivity.java

diff --git a/src/com/android/mail/compose/ComposeActivity.java b/src/com/android/mail/compose/ComposeActivity.java
index bdf7070..e38243f 100644
--- a/src/com/android/mail/compose/ComposeActivity.java
+++ b/src/com/android/mail/compose/ComposeActivity.java
@@ -449,9 +449,10 @@
         }
 
         final List<ReplyFromAccount> replyFromAccounts = mFromSpinner.getReplyFromAccounts();
+        final int selectedPos = mFromSpinner.getSelectedItemPosition();
         final ReplyFromAccount selectedReplyFromAccount =
-                (replyFromAccounts.size() > mFromSpinner.getSelectedItemPosition()) ?
-                        replyFromAccounts.get(mFromSpinner.getSelectedItemPosition()) :
+                (replyFromAccounts.size() > 0 && replyFromAccounts.size() > selectedPos) ?
+                        replyFromAccounts.get(selectedPos) :
                         null;
         if (selectedReplyFromAccount != null) {
             state.putString(EXTRA_SELECTED_REPLY_FROM_ACCOUNT, selectedReplyFromAccount.serialize()