View Bug Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0002523DCP-o-maticBugspublic2023-05-06 22:312023-05-11 22:27
Reportercarl Assigned Tocarl  
PriorityurgentSeveritycrashReproducibilityalways
Status resolvedResolutionfixed 
Target Version2.16.56 
Summary0002523: Crash on adding KDM to an imported DCP
Description

[]

TagsNo tags attached.
Branch2523-crash
Estimated weeks required
Estimated work requiredUndecided

Activities

carl

2023-05-07 22:14

administrator   ~0005670

Last edited: 2023-05-07 22:14

On adding the KDM we start a job thread to examine the DCP again.
This calls VideoContent::take_from_examiner() which ends up calling set_trim_start() on the content.
This signals the text panel to update (in the UI thread) which results in DCPDecoder being constructed in can_reference_text().
At the same time, in DCPContent::take_from_examiner() we set _kdm_valid_to true.
This change happens during the DCPDecoder constructor in the other thread, between the time when it decides not to create a video because can_be_played() is false to the time when it calls get_reader() and assumes that if can_be_played() there will be a video.

video is null -> crash.

carl

2023-05-09 23:47

administrator   ~0005671

@carl tests running

carl

2023-05-11 22:27

administrator   ~0005677

0f11f0c25465464b28db713cccd37d7d8de483e1

Bug History

Date Modified Username Field Change
2023-05-06 22:31 carl New Bug
2023-05-06 22:31 carl Assigned To => carl
2023-05-06 22:31 carl Status new => confirmed
2023-05-06 23:43 carl Branch => 2523-crash
2023-05-06 23:43 carl Estimated work required => Undecided
2023-05-07 22:14 carl Note Added: 0005670
2023-05-07 22:14 carl Note Edited: 0005670
2023-05-09 23:47 carl Status confirmed => in progress
2023-05-09 23:47 carl Note Added: 0005671
2023-05-11 22:27 carl Status in progress => resolved
2023-05-11 22:27 carl Resolution open => fixed
2023-05-11 22:27 carl Note Added: 0005677