fix(api): stse_data_storage_change_read_access_condition fails due to…

[Grolleau-Benjamin]

Description

There is a functional regression or logic error in the stse_data_storage_change_read_access_condition function. Currently, this function calls stsafea_read_data_zone with a read_length set to 0x00.

However, stsafea_read_data_zone includes a parameter validation check that returns STSE_SERVICE_INVALID_PARAMETER if the length is zero, making it impossible to update the access conditions via this API function.

The current way to update read access condition is :

stsafea_read_option_t read_option = {0};
read_option.new_read_ac = STSE_AC_HOST;
read_option.new_read_ac_change_right = STSE_ACCR_ENABLE;
read_option.change_ac_indicator = STSE_AC_CHANGE;

PLAT_UI8 dummyBuffer[1] = {0};
ret = stsafea_read_data_zone(&stse_handler, zone_id, read_option, 0, dummyBuffer,
			     sizeof(dummyBuffer), STSE_NO_PROT);

After changes

This function now works correctly.

ret = stse_data_storage_change_read_access_condition(&stse_handler,
							   2,            // zone
							   STSE_AC_HOST, // new access condition
							   STSE_ACCR_ENABLE, // change right
							   STSE_NO_PROT      // protection
);
if (ret != STSE_OK) {
  printk("Failed to update READ AC to HOST: %d\n", ret);
  return -1;
}

Thanks @nils-cercariolo-st for advices. This PR closes #61

[TofMassilia13320]

@Grolleau-Benjamin,
Due to your branch out-of-date with the base branch, could you rebase and merge your branch and re-push?
NB: Auth AC is deprecated, only Auth + Host is authorized

[Grolleau-Benjamin]

@TofMassilia13320
Just did my rebase, is it good for you now?

[TofMassilia13320]

I don't know what's happening during your rebase; some commits are duplicated.
Anyway, I cherry pick your job into this commit.