Code Review / apps / agl-service-can-low-level.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 386fb28) | patch
socketcan-bcm: Fix wrongly opened socket
authorRomain Forlot <romain.forlot@iot.bzh>
Mon, 25 Nov 2019 14:04:07 +0000 (15:04 +0100)
committerRomain Forlot <romain.forlot@iot.bzh>
Tue, 3 Dec 2019 18:46:26 +0000 (19:46 +0100)
commit01bd3e12d223aeac1a4b4bfd873fc3cbb0cd0036
treebcb86b623547c2024f5bd4373aaa507ee03253a4tree | snapshot
parent386fb28d799f6f1557860c7b31e879eb547d78c2commit | diff
socketcan-bcm: Fix wrongly opened socket

This commit fixes a problem once a bcm socket has been opened
but the following ioctl failed. Before we still return the socket
number instead of returning a negative value as error.
This resulted in problem because the subscription was considered as valid
and binding was continuing using the closed file descriptor.

Change-Id: Iaa64e7bb00980cb8c0e7080aca238cfb57d628b6
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
low-can-binding/utils/socketcan-bcm.cpp diff | blob | history
Low level CAN service made to decode and write on CAN bus.