From 14a1593ef6163c400d5e80bad24e4bb52893b14c Mon Sep 17 00:00:00 2001 From: Florens Douwes Date: Wed, 16 Apr 2014 17:13:02 +0200 Subject: [PATCH] Fixed npe in ReplyFragment callback from ReplyManager. The user could potentially navigate away, and than the activity is null. --- Chan/src/org/floens/chan/ui/fragment/ReplyFragment.java | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Chan/src/org/floens/chan/ui/fragment/ReplyFragment.java b/Chan/src/org/floens/chan/ui/fragment/ReplyFragment.java index 98733622..f63310d2 100644 --- a/Chan/src/org/floens/chan/ui/fragment/ReplyFragment.java +++ b/Chan/src/org/floens/chan/ui/fragment/ReplyFragment.java @@ -410,6 +410,8 @@ public class ReplyFragment extends DialogFragment { * @param response */ private void handleSubmitResponse(ReplyResponse response) { + if (getActivity() == null) return; + if (response.isNetworkError || response.isUserError) { int resId = response.isCaptchaError ? R.string.reply_error_captcha : (response.isFileError ? R.string.reply_error_file : R.string.reply_error); Toast.makeText(getActivity(), resId, Toast.LENGTH_LONG).show();