Skip to content

Off by one error in parseMessage? #10

@aaronvark

Description

@aaronvark

I think there's an off-by-one error in parsing strings in parseMessage. Whenever a string is parsed, the index is shifted by the length of the string, not accounting for the +1 of the null terminator, meaning my message that contained 2 subsequent strings, and then a bunch of floats, got entirely garbled.

This may have been because coincidentally the index aligned to 4 exactly, without the +1. In other cases, the misalignment would probably catch this issue.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions