Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Companion opening some OpenTX dot bin files crashes the application #5034

Closed
1 task done
elecpower opened this issue May 18, 2024 · 1 comment · Fixed by #5037
Closed
1 task done

Companion opening some OpenTX dot bin files crashes the application #5034

elecpower opened this issue May 18, 2024 · 1 comment · Fixed by #5037
Assignees
Labels
bug/regression ↩️ A new version of EdgeTX broke something companion Related to the companion software

Comments

@elecpower
Copy link
Collaborator

Is there an existing issue for this problem?

  • I have searched the existing issues

What part of EdgeTX is the focus of this bug?

Companion

Current Behavior

The issue manifests itself when the current radio profile's associated hardware definition has more features than those supported in the bin file trying to be opened.

Expected Behavior

Application opens file

Steps To Reproduce

Try to open a bin file

Version

2.10.0

Transmitter

Other (Please specify below)

Operating System (OS)

Linux, macOS, Windows

OS Version

All

Anything else?

When reading binary files the code loops through every radio type attempting to find a supported radio and decodes the whole file for each tested radio type.
The ADC refactor changed hardware definitions from static to dynamic and this old looping behaviour is not triggering loading of the loop's current radio hardware definition.
The result is the current radio profile hardware definition is being used as the reference. Therefore if the profile hardware has more features eg pots than supported in the bin file radio type an array out of bounds condition occurs and the application crashes.

@elecpower elecpower added companion Related to the companion software bug/regression ↩️ A new version of EdgeTX broke something labels May 18, 2024
@elecpower elecpower self-assigned this May 18, 2024
@3djc
Copy link
Collaborator

3djc commented May 19, 2024

If users are blocked by this issue trying to convert Otx eeprom file to version 2.10, they can use Companion 2.9.4 to convert the eeprom, save etx file, and then open that etx file with Companion 2.10 until this gets resolved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug/regression ↩️ A new version of EdgeTX broke something companion Related to the companion software
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants