bugfix: make process_ext_cmd_rsp() reentrant.

1. Remove global data buffer defined in phNxpUciHal_Control.
  To make phNxpUciHal_process_ext_cmd_rsp() reentrant.

  Possible recursion path found from STS index check:
    SetAppConfig command --> processs_ext_cmd_rsp() -->
    UrskDeletion (async path)--> process_ext_cmd_rsp()

2. Use size_t and "const" pointer for UCI TX path:

  Split phTmlUwb_TransactInfo into Reader/Writer version.
  phTmlUwb_WriteTransactInfo/phTmlUwb_ReadTransactInfo.
  Only phTmlUwb_ReadTransactInfo has non-const buffer pointer.

  Split coresponding Read/Write callback types.

  Split phTmlUwb_ReadWriteInfo into Reader/Writer version.

3. cleanup: TmlUwb: hide internal data structures
  - move data type definitions for internal use to .cc
  - use unique_ptr instead of malloc/free.

Bug: 378432364
Test: stress test + ccc ranging start
Change-Id: I7630b1cf44dc399c8fe46c0ad812960f9213cc41
Merged-In: I7630b1cf44dc399c8fe46c0ad812960f9213cc41
8 files changed
tree: ea4c904e6ded32b9df3dcb9fefbfd9d2e45b31d4
  1. aidl/
  2. extns/
  3. halimpl/
  4. Android.bp