@bvibber, @Umherirrender any thoughts on this one?

@brennen ah yes my old nemesis, getid3's MPEG parser

likely course of action:

• check for upstream updates that might have cleaned this code up
• if not, look in and follow the logic and make a patch for the bogus variable (possibly related to parsing a partial file chunk and not getting expected bytes, possibly not)
• submit patch upstream and ask for maintainer to branch a release if possible
• pull the updated release in composer.json and update TimedMediaHandler

I would not consider this very high priority right now while we're in crunch on Charts, but if it doesn't get cleaned up earlier it'll be on my tech debt list in late December and January.

Amusingly, Tim last touched that line in https://github.com/JamesHeinrich/getID3/commit/273e47678035658a6c1c6837448f1732026b77cb

https://github.com/JamesHeinrich/getID3/blame/143af33/getid3/module.audio-video.mpeg.php#L512

		if (!empty($info['mpeg']['video']['bitrate_mode']) && ($info['mpeg']['video']['bitrate_mode'] == 'vbr')) {
			$last_GOP_id = max(array_keys($FramesByGOP));
			$frames_in_last_GOP = count($FramesByGOP[$last_GOP_id]);
			$gopdata = &$info['mpeg']['group_of_pictures'][$last_GOP_id];
			$info['playtime_seconds'] = ($gopdata['time_code_hours'] * 3600) + ($gopdata['time_code_minutes'] * 60) + $gopdata['time_code_seconds'] + getid3_lib::SafeDiv($gopdata['time_code_pictures'] + $frames_in_last_GOP + 1, $info['mpeg']['video']['frame_rate']);
			if (!isset($info['video']['bitrate'])) {
				$overall_bitrate = getid3_lib::SafeDiv($info['avdataend'] - $info['avdataoffset'] * 8, $info['playtime_seconds']);
				$info['video']['bitrate'] = $overall_bitrate - (isset($info['audio']['bitrate']) ? $info['audio']['bitrate'] : 0);
			}
			unset($info['mpeg']['group_of_pictures']);
		}

I'm guessing $gopdata is null in this case, based on $info being in the guard condition to get in here...

The fact it is in chunked uploading code, we could be passing in bad data in the first place.