tree 683ba01a1d6aa1a62bc9f4e640a640d0f01b204f
parent 38f510c096bcc6ce4bdf901b936cda280832f57c
author Nivi Sarkar <55898241+nivi-apple@users.noreply.github.com> 1675466897 -0800
committer GitHub <noreply@github.com> 1675466897 +0000
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJj3ZiRCRBK7hj4Ov3rIwAA3p8IAAi3Hx4gJGlEE9cfoGBnD1Tj
 TnOZ3Z4EXTy4zOZZ3pK2roNvgV6CrW1jnSxXL/Bwne8wZKRH+eJ2EjhtMX6/lrps
 qzCHmXCtT30BYluZ0sdzCHKsAREVfW/t3bVfAUKR1H3mHo8qJf+LMTwBL0gK4lam
 Y+CBaqhPOZ0cbYgtCk0qHZZdTcY8O1QTkh+jQw80pdLcvfWfnCqJUl379ICesgH/
 n0RijgvGFKqLSc5ORB7H/1FM/EIbPq3XZOv11r6jmB4olp2cvv1eC5Hc7Axjcjo8
 wX1Frz/DOj1ZEfD8Xr7IUa++Y+Is1FPq1wGeEGaxUD19PBUmtNoxyKJrrf95Bi8=
 =otah
 -----END PGP SIGNATURE-----
 

Add a timer to track whether we have received BDX init after a query … (#24777)

* Add a timer to track whether we have received BDX init after a query image was successful

- Currently if an ota requester has a successful query image and an image it available but
  if for any reason, the ota requester doesn't send BDX init, the provider will be stuck until
  we reboot the resident. In order to have a fail safe, we are adding a timer that starts after
  query image returns image available and waits for a BDX init to come. In case BDX init doesn't
  come, it times out and resets state

- Add code to reset state if any API fails on the provider once we prepare for BDX transfer

- Stop polling when BDX transfer reset is called

- Return QueryImageResponse status busy instead of general failure if the sdk is busy and
  gets a sexond query image so accessory can handle the error correctly and retry until the
  sdk is done

- When we send a message, handle the case where if an error happens in sending the message
  close the exchange context and reset state. If the message is a status report and it
  succeeds, null out the exchange and reset state

* Update src/darwin/Framework/CHIP/MTROTAProviderDelegateBridge.mm

Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>

* Update src/darwin/Framework/CHIP/MTROTAProviderDelegateBridge.mm

Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>

* Reset mStopPolling to false when we call PrepareForTransfer to start polling again

- This covers the use case where we have an image available but the requestor rejects
  the transfer for some reason. On the next query image the requestor is able to
  continue with the BDX transfer. The provider state was not being reset for this scenario
  due to the polling flag not being reset.

- Fixes #23573

* Addressed review comments

* Fix the handling of the use case when hasUpdate is false we need to send the response as is

- Log the error if the BDX timer fails to start

---------

Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>