Issue #3177541 by alexpott, andypost: stream_open() needs to cope with a...

Issue #3177541 by alexpott, andypost: stream_open() needs to cope with a failure in \Drupal\Core\StreamWrapper\LocalStream::getLocalPath() better

Issue #3177541 by alexpott, andypost: stream_open() needs to cope with a...
fc7d4a51 · catch · 94708f5a · fc7d4a51
Commit fc7d4a51 authored 4 years ago by catch
--- a/core/lib/Drupal/Core/StreamWrapper/LocalStream.php
+++ b/core/lib/Drupal/Core/StreamWrapper/LocalStream.php
@@ -163,6 +163,12 @@ protected function getLocalPath($uri = NULL) {
  public function stream_open($uri, $mode, $options, &$opened_path) {
    $this->uri = $uri;
    $path = $this->getLocalPath();
+    if ($path === FALSE) {
+      if ($options & STREAM_REPORT_ERRORS) {
+        trigger_error('stream_open() filename cannot be empty', E_USER_WARNING);
+      }
+      return FALSE;
+    }
    $this->handle = ($options & STREAM_REPORT_ERRORS) ? fopen($path, $mode) : @fopen($path, $mode);

    if ((bool) $this->handle && $options & STREAM_USE_PATH) {