Skip to content

MAVFTP Payload Format doc: data byte index off-by-one #728

Description

@calebstott

In the MAVFTP Payload Format table, the data field currently says:

| 12 to (max) 251 | `uint8_t data[]` | Data | | Command/response data. Varies by [OpCode](#opcodes). This contains the `path` for operations that act on a file or directory. For an ACK for a read or write this is the requested information. For an ACK for a `OpenFileRO` operation this is the size of the file that was opened. For a NAK the first byte is the [error code](#error_codes) and the (optional) second byte may be an error number. |

It should say:

12 to (max) 250

The FILE_TRANSFER_PROTOCOL message field has a payload length of 251, meaning the index should end at 250

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions