Skip to content

Fix NPE when processing connection status event with ECU#38

Merged
kaushalaroraharman merged 1 commit into
mainfrom
bugfix/37-bug-fix-NPE-with-no-ecu-configured
May 27, 2025
Merged

Fix NPE when processing connection status event with ECU#38
kaushalaroraharman merged 1 commit into
mainfrom
bugfix/37-bug-fix-NPE-with-no-ecu-configured

Conversation

@kaushalaroraharman
Copy link
Copy Markdown
Contributor

Resolves #37


Describe behaviour before the change

  • Stream base throwing error while processing subscribe/unsubscribe events from the device without the property dma.dispatcher.ecu.types

Describe behaviour after the change

  • ECU type list has been initialized with an empty list, in case no ecus are defined in configuration to avoid NPE.

Pull request checklist

  • I have read the CONTRIBUTING.md
  • My code follows the code style of this project
  • Tests for the changes have been added (for bug fixes / features)
  • All new and existing tests passed.
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?

  • Yes
  • No

@sonarqubecloud
Copy link
Copy Markdown

@kaushalaroraharman kaushalaroraharman merged commit 97e3a37 into main May 27, 2025
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG]: Stream base throwing error while processing subscribe/unsubscribe events from the device without the property dma_dispatcher_ecu_types

1 participant