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

	F2987193: D25516.1740110746.diff
	Thu, Feb 20, 04:05

	F2982287: D25516.1739948802.diff
	Tue, Feb 18, 07:06

	F2938345: D25516.1737994266.diff
	Sun, Jan 26, 16:11

	F2938344: D25516.1737994262.diff
	Sun, Jan 26, 16:11

	F2938343: D25516.1737994257.diff
	Sun, Jan 26, 16:10

	F2936258: D25516.1737911635.diff
	Sat, Jan 25, 17:13

	F2927385: D25516.1737647677.diff
	Wed, Jan 22, 15:54

	F2908533: D25516.1737379263.diff
	Jan 19 2025, 13:21

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