Skip to content

REF-039 - Classify MediaInfo Loader Failures And Bound Metadata Extraction Counts

Summary

Follow-up hardening after REF-038. The MediaInfo DLL loader now preserves specific failure state so UI hints can distinguish intentional disablement from missing or unusable DLLs. MediaInfo-reported stream and chapter counts are bounded before they drive loops, and short codec strings no longer feed FOURCC extraction.

Decisions

  • Keep MediaInfo.dll optional and external.
  • Keep MediaInfo_MediaInfoDllPath=<noload> as an intentional silent disable.
  • Keep current extraction precedence: MediaInfo DLL, built-in RIFF/RM/WM, retained id3lib fallback.
  • Keep DirectShow MediaDet as a legacy last fallback for this release.

Acceptance

  • [x] Loader status separates loaded, disabled, missing, incompatible, bad exports, and load failure states.
  • [x] File Info install hints are suppressed when MediaInfo loading is intentionally disabled.
  • [x] Missing/unusable MediaInfo hints mention lookup/configuration context.
  • [x] MediaInfo stream and chapter counts are bounded before loop use.
  • [x] FOURCC extraction requires at least four codec bytes.
  • [x] Native seam tests cover loader status, count clamping, chapter clamping, and short codec strings.