Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ota-provider-app] fix ExchangeManager singleton access method #9943

Merged

Conversation

holbrookt
Copy link
Contributor

Problem

ota-provider-app needs access to ExchangeManager in order to set an UnsolicitedMessageHandler for BDX messages.

It was getting access via chip::ExchangeManager() in src/app/util/util.h which is somewhat out of date and not used by any other code paths now.

Change overview

  • Use the new Server::GetExchangeManager() to get access to the ExchangeManager instantiated by the Server singleton

Testing

Confirmed full OTA transfer occurs using ota-provider-app and ota-requestor-app

@woody-apple woody-apple merged commit 3c45842 into project-chip:master Sep 28, 2021
@holbrookt holbrookt deleted the ota-provider/exchange-mgr-access branch September 28, 2021 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants