Paths

Table of Contentst

Differential D25516

preg_match() null exception setting custom user profile image with empty files.viewable-mime-types
ClosedPublic
Actions

Authored by aklapper on Jan 14 2024, 11:18.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Apr 25, 22:35

	Unknown Object (File)
	Wed, Apr 17, 18:20

	Unknown Object (File)
	Wed, Apr 17, 06:51

	Unknown Object (File)
	Thu, Apr 4, 01:34

	Unknown Object (File)
	Mar 31 2024, 05:16

	Unknown Object (File)
	Mar 31 2024, 05:16

	Unknown Object (File)
	Mar 31 2024, 04:08

	Unknown Object (File)
	Mar 30 2024, 00:05

View All 32 Files

Subscribers

valerio.bozzolan

Details

Reviewers

Group Reviewers

O1: Blessed Committers

Maniphest Tasks

T15710: preg_match() null exception setting custom user profile image with empty files.viewable-mime-types

Commits

rP7702772b2b86: preg_match() null exception setting custom user profile image with empty files.

Summary

When files.viewable-mime-types is not set, getViewableMimeType() passes null to preg_match() which is deprecated behavior since PHP 8.1.
Only call preg_match() when there are some MIME types to compare.

ERROR 8192: preg_match(): Passing null to parameter #2 ($subject) of type string is deprecated at [/var/www/html/phorge/phorge/src/applications/files/storage/PhabricatorFile.php:974]

Closes T15710

Test Plan

Go to a user profile and try to upload a custom profile picture in BMP format.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

aklapper created this revision.Jan 14 2024, 11:18

Owners added a reviewer: O1: Blessed Committers.Jan 14 2024, 11:18

Herald added subscribers: Cigaryno, Matthew, valerio.bozzolan, tobiaswiese. · View Herald TranscriptJan 14 2024, 11:18

aklapper requested review of this revision.Jan 14 2024, 11:18

Harbormaster completed remote builds in B1022: Diff 1638.Jan 14 2024, 11:18

speck accepted this revision.Jan 14 2024, 13:43

This revision is now accepted and ready to land.Jan 14 2024, 13:43

Closed by commit rP7702772b2b86: preg_match() null exception setting custom user profile image with empty files.. · Explain WhyJan 17 2024, 15:13

This revision was automatically updated to reflect the committed changes.

aklapper added a commit: rP7702772b2b86: preg_match() null exception setting custom user profile image with empty files..

Revision Contents
Changeset List

Path

Size

Packages

src/

applications/

files/

storage/

PhabricatorFile.php

11 lines

O1: Blessed Committers

Diff 1643

src/applications/files/storage/PhabricatorFile.php

Loading...