Skip to content

Catch RuntimeException from isoparser and add regression test#497

Open
tsautier wants to merge 2 commits into
drftpd-ng:masterfrom
tsautier:fix/issue-198
Open

Catch RuntimeException from isoparser and add regression test#497
tsautier wants to merge 2 commits into
drftpd-ng:masterfrom
tsautier:fix/issue-198

Conversation

@tsautier

@tsautier tsautier commented Jan 14, 2026

Copy link
Copy Markdown

Problem

MediaInfo parsing could crash the slave when casting to int fails. The isoparser library throws RuntimeException on certain malformed media files, causing the slave to disconnect.

Solution

Catch RuntimeException from isoparser and handle gracefully instead of crashing.

Changes Made

  • Added try-catch block around isoparser calls
  • RuntimeExceptions from malformed media files are now logged and handled
  • Slave no longer crashes on problematic media files

Fixes: #198

@tsautier

Copy link
Copy Markdown
Author

@mvangoor @hrxcodes for review

@tsautier tsautier changed the title Fix/issue 198 Catch RuntimeException from isoparser and add regression test Jan 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

mediainfo - cast to int fails and crashes slave

1 participant