FlapJackApps/libdivecomputer
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Limit the index to the fixed gas mixes

Browse Source 
The index in the gas change event should refer to the one of the fixed
gas mixes. All gas mixes with a higher index are either manual or
bailout gas mixes, and are reported with different events containing an
O2 and He percentages instead.
This commit is contained in:
Jef Driesen 2023-02-15 21:07:02 +01:00
parent 98c7887e9c
commit 328812e95b
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/hw_ostc_parser.c
View File

@ -902,8 +902,8 @@ hw_ostc_parser_samples_foreach (dc_parser_t *abstract, dc_sample_callback_t call
return DC_STATUS_DATAFORMAT; return DC_STATUS_DATAFORMAT;
} }
unsigned int idx = data[offset]; unsigned int idx = data[offset];
if (idx < 1 || idx > parser->ngasmixes) { if (idx < 1 || idx > parser->nfixed) {
ERROR(abstract->context, "Invalid gas mix."); ERROR(abstract->context, "Invalid gas mix (%u).", idx);
return DC_STATUS_DATAFORMAT; return DC_STATUS_DATAFORMAT;
} }
idx--; /* Convert to a zero based index. */ idx--; /* Convert to a zero based index. */