Skip to content
This repository has been archived by the owner on Jun 2, 2023. It is now read-only.

Wrong behaviour for "READ_DTC_INFO_BY_STATUS_MASK" #6

Open
sigttou opened this issue Dec 10, 2020 · 0 comments
Open

Wrong behaviour for "READ_DTC_INFO_BY_STATUS_MASK" #6

sigttou opened this issue Dec 10, 2020 · 0 comments

Comments

@sigttou
Copy link

sigttou commented Dec 10, 2020

From my understanding the len(request) should be at least 3, no? I am not sure, as I do not have the exact specification at hand.

Looking at the implementation of udsoncan, the 0x19 service always takes an additional parameter, the status mask for the function 0x2(READ_DTC_INFO_BY_STATUS_MASK).

I think, therefore, there's an error in the function get_0x19_response for handling READ_DTC_INFO_BY_STATUS_MASK.

Hope this is a correct mistake, otherwise I will report a bug at udsoncan.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant