While the intention is there, it should have been uint8_t **data for it to be (inout).
uint8_t **data