[PR #1870] [MERGED] fix(708): Handle null timing pointer in CEA-708 settings conversion #2642

New Issue

claunia · 2026-01-29T17:23:15Z

claunia commented

2026-01-29 17:23:15 +00:00

📋 Pull Request Information

Original PR: https://github.com/CCExtractor/ccextractor/pull/1870
Author: @cfsmp3
Created: 12/20/2025
Status: ✅ Merged
Merged: 12/21/2025
Merged by: @cfsmp3

Base: master ← Head: fix/cea708-null-timing-pointer

📝 Commits (1)

8f78a8b fix(708): Handle null timing pointer in CEA-708 settings conversion

📊 Changes

1 file changed (+8 additions, -2 deletions)

View changed files

📝 src/rust/src/ctorust.rs (+8 -2)

📄 Description

Summary

Fix CEA-708 decoder not activating when --service option is specified
Handle null timing pointer gracefully in from_ctype() conversion
Prevents critical settings (enabled, services_enabled) from being reset to defaults

Problem

When converting CEA-708 decoder settings from C to Rust via from_ctype(), a null timing pointer caused the entire conversion to fail and return None. This triggered the unwrap_or(default()) fallback, resetting:

enabled from true to false
active_services_count from 1 to 0

This caused CEA-708 captions to not be extracted (exit code 10) even when --service 1 was specified.

Root Cause

In src/rust/src/ctorust.rs, the ? operator on timing conversion propagated None when timing was null:

timing: CommonTimingCtx::from_ctype(settings.timing)?,  // Returns None if timing is null

Fix

Handle null timing pointer gracefully:

let timing = if settings.timing.is_null() {
    CommonTimingCtx::default()
} else {
    CommonTimingCtx::from_ctype(settings.timing).unwrap_or_default()
};

Test plan

Test with CEA-708 sample that was failing (exit code 10 → exit code 0)
Verify captions are extracted with --service 1
CI tests for CEA-708 (tests 141, 142, 146, 147, 149)

🤖 Generated with Claude Code

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/CCExtractor/ccextractor/pull/1870 **Author:** [@cfsmp3](https://github.com/cfsmp3) **Created:** 12/20/2025 **Status:** ✅ Merged **Merged:** 12/21/2025 **Merged by:** [@cfsmp3](https://github.com/cfsmp3) **Base:** `master` ← **Head:** `fix/cea708-null-timing-pointer` --- ### 📝 Commits (1) - [`8f78a8b`](https://github.com/CCExtractor/ccextractor/commit/8f78a8bbb2f9788b83c19e349d217d2527f89ebc) fix(708): Handle null timing pointer in CEA-708 settings conversion ### 📊 Changes **1 file changed** (+8 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `src/rust/src/ctorust.rs` (+8 -2) </details> ### 📄 Description ## Summary - Fix CEA-708 decoder not activating when `--service` option is specified - Handle null timing pointer gracefully in `from_ctype()` conversion - Prevents critical settings (`enabled`, `services_enabled`) from being reset to defaults ## Problem When converting CEA-708 decoder settings from C to Rust via `from_ctype()`, a null timing pointer caused the entire conversion to fail and return `None`. This triggered the `unwrap_or(default())` fallback, resetting: - `enabled` from `true` to `false` - `active_services_count` from `1` to `0` This caused CEA-708 captions to not be extracted (exit code 10) even when `--service 1` was specified. ## Root Cause In `src/rust/src/ctorust.rs`, the `?` operator on timing conversion propagated `None` when timing was null: ```rust timing: CommonTimingCtx::from_ctype(settings.timing)?, // Returns None if timing is null ``` ## Fix Handle null timing pointer gracefully: ```rust let timing = if settings.timing.is_null() { CommonTimingCtx::default() } else { CommonTimingCtx::from_ctype(settings.timing).unwrap_or_default() }; ``` ## Test plan - [x] Test with CEA-708 sample that was failing (exit code 10 → exit code 0) - [x] Verify captions are extracted with `--service 1` - [ ] CI tests for CEA-708 (tests 141, 142, 146, 147, 149) 🤖 Generated with [Claude Code](https://claude.com/claude-code) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

claunia added the pull-request label 2026-01-29 17:23:15 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/ccextractor#2642