| Message ID | 9a1270540f0c2db13e81a9d69098391f1ad22107.1667113164.git.christophe.jaillet@wanadoo.fr |
|---|---|
| State | Accepted |
| Commit | 8fceb58d84ab83ab588bbed69cd143deb81b2ac5 |
| Headers | show |
| Series | Bluetooth: Fix EALREADY and ELOOP cases in bt_status() | 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=690214 ---Test result--- Test Summary: CheckPatch PASS 1.37 seconds GitLint PASS 0.78 seconds SubjectPrefix PASS 0.62 seconds BuildKernel PASS 41.34 seconds BuildKernel32 PASS 37.50 seconds Incremental Build with patchesPASS 53.34 seconds TestRunner: Setup PASS 613.53 seconds TestRunner: l2cap-tester PASS 19.92 seconds TestRunner: iso-tester PASS 19.44 seconds TestRunner: bnep-tester PASS 7.62 seconds TestRunner: mgmt-tester PASS 122.06 seconds TestRunner: rfcomm-tester PASS 14.95 seconds TestRunner: sco-tester PASS 11.14 seconds TestRunner: ioctl-tester PASS 12.85 seconds TestRunner: mesh-tester PASS 9.31 seconds TestRunner: smp-tester PASS 11.39 seconds TestRunner: userchan-tester PASS 7.99 seconds --- Regards, Linux Bluetooth
Hello: This patch was applied to bluetooth/bluetooth-next.git (master) by Luiz Augusto von Dentz <luiz.von.dentz@intel.com>: On Sun, 30 Oct 2022 08:00:03 +0100 you wrote: > 'err' is known to be <0 at this point. > > So, some cases can not be reached because of a missing "-". > Add it. > > Fixes: ca2045e059c3 ("Bluetooth: Add bt_status") > Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr> > > [...] Here is the summary with links: - Bluetooth: Fix EALREADY and ELOOP cases in bt_status() https://git.kernel.org/bluetooth/bluetooth-next/c/8fceb58d84ab You are awesome, thank you!
diff --git a/net/bluetooth/lib.c b/net/bluetooth/lib.c index 469a0c95b6e8..53a796ac078c 100644 --- a/net/bluetooth/lib.c +++ b/net/bluetooth/lib.c @@ -170,7 +170,7 @@ __u8 bt_status(int err) case -EMLINK: return 0x09; - case EALREADY: + case -EALREADY: return 0x0b; case -EBUSY: @@ -191,7 +191,7 @@ __u8 bt_status(int err) case -ECONNABORTED: return 0x16; - case ELOOP: + case -ELOOP: return 0x17; case -EPROTONOSUPPORT:
'err' is known to be <0 at this point. So, some cases can not be reached because of a missing "-". Add it. Fixes: ca2045e059c3 ("Bluetooth: Add bt_status") Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr> --- net/bluetooth/lib.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)