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

	F3330193: D25516.1743485119.diff
	Mon, Mar 31, 05:25

	F3329886: D25516.1743480852.diff
	Mon, Mar 31, 04:14

	F3309176: D25516.1743183316.diff
	Thu, Mar 27, 17:35

	F3304029: D25516.1743090705.diff
	Wed, Mar 26, 15:51

	F3303480: D25516.1743081285.diff
	Wed, Mar 26, 13:14

	F3301630: D25516.1743054965.diff
	Wed, Mar 26, 05:56

	F3301382: D25516.1743053636.diff
	Wed, Mar 26, 05:33

	F3297137: D25516.1742981814.diff
	Tue, Mar 25, 09:36

View All 27 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...