Relationships
#4305 @swamp/video/brand: cap each Chromium stderr line kept for errors
Opened by skunk-ape · 11/2/2026
Follow-up from verify-reviews on swamp-club #4124 (commit 7fe231c43), low severity. StderrTail in video/brand/extensions/models/_lib/scene.ts caps only the trailing partial line at STDERR_LINE_MAX (4096). Completed lines are pushed whole, so a Chromium that writes a few 64 KB newline-terminated lines puts up to about 15 x 64 KB into every error built from tail(). Memory stays bounded (15 lines). Also push only parts.slice(-STDERR_TAIL_LINES) rather than spreading every split line into push(). Fix: cap each line before pushing; add a test with long newline-terminated lines.
Open
No activity in this phase yet.
system commented 11/2/2026, 5:35:41 PM
Classified automatically when this issue was filed.
- Source: Extensions
If you feel this classification is incorrect, add a ripple to tell us so.
Sign in to post a ripple.