From 30d72ec3d0914bb905d4114dee9256f02cbdcc7f Mon Sep 17 00:00:00 2001 From: Edward Chernenko Date: Mon, 2 Apr 2018 00:39:59 +0300 Subject: [PATCH] Fix warning "ob_end_flush(): failed to delete and flush buffer" during uploads Bug: T186565 Change-Id: Ia4b29611ccee4acac11717f5220ff9e0fdbd55a9 --- includes/filerepo/FileRepo.php | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/includes/filerepo/FileRepo.php b/includes/filerepo/FileRepo.php index e430bc8f43..0c37d78b26 100644 --- a/includes/filerepo/FileRepo.php +++ b/includes/filerepo/FileRepo.php @@ -1634,7 +1634,11 @@ class FileRepo { $status = $this->newGood(); $status->merge( $this->backend->streamFile( $params ) ); - ob_end_flush(); + // T186565: Close the buffer, unless it has already been closed + // in HTTPFileStreamer::resetOutputBuffers(). + if ( ob_get_status() ) { + ob_end_flush(); + } return $status; } -- 2.20.1