Merge pull request #186 from 10up/fix/184

Ensure the svg file loads properly before we access it's properties
10up · Mar 26, 2024 · 941da6c · 941da6c
2 parents d47a4a1 + 741cc55
commit 941da6c
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/safe-svg.php b/safe-svg.php
@@ -597,6 +597,11 @@ protected function svg_dimensions( $attachment_id ) {
 			} elseif ( $svg ) {
 				$svg = @simplexml_load_file( $svg ); // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
 
+				// Ensure the svg could be loaded.
+				if ( ! $svg ) {
+					return false;
+				}
+
 				$attributes = $svg->attributes();
 
 				if ( isset( $attributes->viewBox ) ) { // phpcs:ignore WordPress.NamingConventions.ValidVariableName.UsedPropertyNotSnakeCase