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.

Sign up for GitHub

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

Jump to bottom

AP_ROMFS: remove requirement that returned buffer is null-terminated #26184

Closed
wants to merge 4 commits into from
Closed

AP_ROMFS: remove requirement that returned buffer is null-terminated #26184

wants to merge 4 commits into from

Commits on Feb 10, 2024

  1. AP_HAL_ChibiOS: remove redundant fw size check 

    embed.py already ensures it's a multiple of 32 bytes. If it wasn't, then
the buffer would be overflowed and random garbage would be flashed at
the end.
    @tpwrules
    tpwrules committed Feb 10, 2024
    Configuration menu
    Copy the full SHA
    9cc889e View commit details
    Browse the repository at this point in the history

  2. AP_Param: clarify use of parameter text buffer 

    Refer to it 'buffer' instead of 'string' to clarify that the length is
respected and that it does not need to be null terminated.
    @tpwrules
    tpwrules committed Feb 10, 2024
    Configuration menu
    Copy the full SHA
    075afb2 View commit details
    Browse the repository at this point in the history

  3. AP_Bootloader: don't assume substitute_vars input is null-terminated

    @tpwrules
    tpwrules committed Feb 10, 2024
    Configuration menu
    Copy the full SHA
    42e5c9c View commit details
    Browse the repository at this point in the history

  4. AP_ROMFS: remove requirement that returned buffer is null-terminated 

    Also take the opportunity to clarify the function descriptions.
    @tpwrules
    tpwrules committed Feb 10, 2024
    Configuration menu
    Copy the full SHA
    e754938 View commit details
    Browse the repository at this point in the history