BigBlueButton/bigbluebutton-Github
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

recording: Skip EDL area if it is nil

Browse Source 
The recording processing would crash if an area was present in the
layout, but was missing in the EDL entry being processed.

This doesn't happen in normal conditions, since most of the methods for
generating an EDL will result in areas being present, even if there are
no videos for that area.

But the EDL cleanup recently added can sometimes cause an EDL entry with
no areas to be processed, so add the code to handle this possibility.
This commit is contained in:
Calvin Walton 2023-02-21 15:28:24 -05:00
parent 54e128f0d3
commit dc9126b67b
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
record-and-playback/core/lib/recordandplayback/edl/video.rb
View File

@ -430,6 +430,8 @@ module BigBlueButton
ffmpeg_filter = '[0]null' ffmpeg_filter = '[0]null'
layout[:areas].each do |layout_area| layout[:areas].each do |layout_area|
area = cut[:areas][layout_area[:name]] area = cut[:areas][layout_area[:name]]
next if area.nil?
video_count = area.length video_count = area.length
BigBlueButton.logger.debug " Laying out #{video_count} videos in #{layout_area[:name]}" BigBlueButton.logger.debug " Laying out #{video_count} videos in #{layout_area[:name]}"
next if video_count == 0 next if video_count == 0