diff options
| author |  Rosen Penev | 2020-12-23 17:39:01 -0800 |
|---|---|---|
| committer |  Nikias Bassen | 2022-04-30 13:34:21 +0200 |
| commit | cf2c7d48380969200034afeab93169e68c34c397 (patch) | |
| tree | a96f518be279922d00898a3ce81dc0ff483a40b2 /src/mobileactivation.c | |
| parent | 6cb13f9e6d3939930aecf91d8e23d1896a3b92e5 (diff) | |
| download | libimobiledevice-cf2c7d48380969200034afeab93169e68c34c397.tar.gz libimobiledevice-cf2c7d48380969200034afeab93169e68c34c397.tar.bz2 | |
[clang-tidy] Do not use else after return
Signed-off-by: Rosen Penev <rosenp@gmail.com>
Diffstat (limited to 'src/mobileactivation.c')
| -rw-r--r-- | src/mobileactivation.c | 15 |
1 files changed, 6 insertions, 9 deletions
diff --git a/src/mobileactivation.c b/src/mobileactivation.c index 2de4333..79c7003 100644 --- a/src/mobileactivation.c +++ b/src/mobileactivation.c @@ -109,8 +109,6 @@ static plist_t plist_data_from_plist(plist_t plist) static mobileactivation_error_t mobileactivation_check_result(plist_t dict, const char *command) { - mobileactivation_error_t ret = MOBILEACTIVATION_E_UNKNOWN_ERROR; - if (!dict || plist_get_node_type(dict) != PLIST_DICT) { return MOBILEACTIVATION_E_PLIST_ERROR; } @@ -118,14 +116,13 @@ static mobileactivation_error_t mobileactivation_check_result(plist_t dict, cons plist_t err_node = plist_dict_get_item(dict, "Error"); if (!err_node) { return MOBILEACTIVATION_E_SUCCESS; - } else { - char *errmsg = NULL; - plist_get_string_val(err_node, &errmsg); - debug_info("ERROR: %s: %s", command, errmsg); - ret = MOBILEACTIVATION_E_REQUEST_FAILED; - free(errmsg); } - return ret; + + char *errmsg = NULL; + plist_get_string_val(err_node, &errmsg); + debug_info("ERROR: %s: %s", command, errmsg); + free(errmsg); + return MOBILEACTIVATION_E_REQUEST_FAILED; } static mobileactivation_error_t mobileactivation_send_command_plist(mobileactivation_client_t client, plist_t command, plist_t *result) |