Skip to content

Filename truncation / ambiguity risk #11

@xayaank

Description

@xayaank

Current:
name_len = struct.unpack("!I", data[:4])[0]

relies on trusting first 4 bytes blindly

Problem:
No upper bound check before unpacking beyond MAX_NAME_LEN is good, but attacker can still craft huge valid int if MAX_NAME_LEN increases later

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions