| Message ID | 20240704074759.15844-1-r.smirnov@omp.ru |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [BlueZ,v2] client/player: add error code handling to transport_recv() | expand |
This is automated email and please do not reply to this email! Dear submitter, Thank you for submitting the patches to the linux bluetooth mailing list. This is a CI test results with your patch series: PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=868334 ---Test result--- Test Summary: CheckPatch PASS 0.45 seconds GitLint FAIL 0.49 seconds BuildEll PASS 25.33 seconds BluezMake PASS 1897.59 seconds MakeCheck PASS 13.34 seconds MakeDistcheck PASS 180.67 seconds CheckValgrind PASS 253.61 seconds CheckSmatch PASS 359.52 seconds bluezmakeextell PASS 127.39 seconds IncrementalBuild PASS 1666.22 seconds ScanBuild PASS 1035.74 seconds Details ############################## Test: GitLint - FAIL Desc: Run gitlint Output: [BlueZ,v2] client/player: add error code handling to transport_recv() WARNING: I3 - ignore-body-lines: gitlint will be switching from using Python regex 'match' (match beginning) to 'search' (match anywhere) semantics. Please review your ignore-body-lines.regex option accordingly. To remove this warning, set general.regex-style-search=True. More details: https://jorisroovers.github.io/gitlint/configuration/#regex-style-search 7: B2 Line has trailing whitespace: " V1 -> V2: the name of the patch has been shortened " --- Regards, Linux Bluetooth
diff --git a/client/player.c b/client/player.c index 0d031e4b0..1340a7584 100644 --- a/client/player.c +++ b/client/player.c @@ -4514,7 +4514,13 @@ static bool transport_recv(struct io *io, void *user_data) uint8_t buf[1024]; int ret, len; - ret = read(io_get_fd(io), buf, sizeof(buf)); + ret = io_get_fd(io); + if (ret < 0) { + bt_shell_printf("io_get_fd() returned %d\n", ret); + return true; + } + + ret = read(ret, buf, sizeof(buf)); if (ret < 0) { bt_shell_printf("Failed to read: %s (%d)\n", strerror(errno), -errno);