Skip to content

Commit 643b250

Browse files
committed
NPE fix in TextHttpResponseHandler for onSuccess and onFailure methods
1 parent cc740d1 commit 643b250

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

library/src/com/loopj/android/http/TextHttpResponseHandler.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,8 @@ public void onSuccess(int statusCode, Header[] headers, String responseBody) {
100100
@Override
101101
public void onSuccess(int statusCode, Header[] headers, byte[] responseBody) {
102102
try {
103-
onSuccess(statusCode, headers, new String(responseBody, getCharset()));
103+
String response = responseBody == null ? null : new String(responseBody, getCharset());
104+
onSuccess(statusCode, headers, response);
104105
} catch (UnsupportedEncodingException e) {
105106
Log.v(LOG_TAG, "String encoding failed, calling onFailure(int, Header[], String, Throwable)");
106107
onFailure(0, headers, (String) null, e);
@@ -110,7 +111,8 @@ public void onSuccess(int statusCode, Header[] headers, byte[] responseBody) {
110111
@Override
111112
public void onFailure(int statusCode, Header[] headers, byte[] responseBody, Throwable error) {
112113
try {
113-
onFailure(statusCode, headers, new String(responseBody, getCharset()), error);
114+
String response = responseBody == null ? null : new String(responseBody, getCharset());
115+
onFailure(statusCode, headers, response, error);
114116
} catch (UnsupportedEncodingException e) {
115117
Log.v(LOG_TAG, "String encoding failed, calling onFailure(int, Header[], String, Throwable)");
116118
onFailure(0, headers, (String) null, e);

0 commit comments

Comments
 (0)