Handle more than one level of output buffering when cleaning and ending them so we can send the video file so we don't run out of ram. Fixes #4110

Author: connortechnology

web/skins/classic/views/video.php

@@ -77,11 +77,7 @@
   $deleteIndex = validInt($_REQUEST['deleteIndex']);
   unlink($videoFiles[$deleteIndex]);
   unset($videoFiles[$deleteIndex]);
-}
-
-if ( isset($_REQUEST['downloadIndex']) ) {
-  // can't be output buffering, as this file might be large
-  ob_end_clean();
+} else if (isset($_REQUEST['downloadIndex'])) {
   $downloadIndex = validInt($_REQUEST['downloadIndex']);
   ZM\Debug("Download $downloadIndex, file: " . $videoFiles[$downloadIndex]);
   header('Pragma: public');
@@ -93,6 +89,11 @@
   header('Content-Transfer-Encoding: binary');
   header('Content-Type: application/force-download');
   header('Content-Length: '.filesize($videoFiles[$downloadIndex])); 
+  // can't be output buffering, as this file might be large
+  while (ob_get_level()) {
+    ob_end_clean();
+  }
+  set_time_limit(0);
   readfile($videoFiles[$downloadIndex]);
   exit;
 }