Skip to content

bug: bound XML import memory usage by limiting payload size and read strategy #262

@somethingwithproof

Description

@somethingwithproof

Summary

Alert/report/removal import reads full uploaded file into memory with no explicit size guard.

Evidence

  • syslog_alerts.php:947-948
  • syslog_reports.php:809-810
  • syslog_removal.php:747-748

Pattern: fopen + fread(... filesize(...)) + xml2array($xml_data).

Impact

Large uploads can exhaust PHP memory and destabilize admin requests.

Expected fix

  • Enforce explicit maximum import size
  • Fail fast with user-visible validation message
  • Prefer safer read/parse strategy that avoids unnecessary full-buffer pressure

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions