From f2ce22ef083c8fc7c1965ea634d4d20004aa7b4c Mon Sep 17 00:00:00 2001 From: Yahor Berdnikau Date: Sun, 19 Nov 2017 21:39:13 +0100 Subject: [PATCH] Fix response check NPE Fix response check doens't take into account that body will be null if response is not successfull. Signed-off-by: Yahor Berdnikau --- .../java/org/moire/ultrasonic/service/RESTMusicService.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/ultrasonic/src/main/java/org/moire/ultrasonic/service/RESTMusicService.java b/ultrasonic/src/main/java/org/moire/ultrasonic/service/RESTMusicService.java index 16b4bfb5..db2066e5 100644 --- a/ultrasonic/src/main/java/org/moire/ultrasonic/service/RESTMusicService.java +++ b/ultrasonic/src/main/java/org/moire/ultrasonic/service/RESTMusicService.java @@ -1462,7 +1462,9 @@ public class RESTMusicService implements MusicService return; } - if (response.body().getStatus() == SubsonicResponse.Status.ERROR && + if (!response.isSuccessful()) { + throw new IOException("Server error, code: " + response.code()); + } else if (response.body().getStatus() == SubsonicResponse.Status.ERROR && response.body().getError() != null) { throw new IOException("Server error: " + response.body().getError().getCode()); } else {