Page MenuHomePhabricator
Differential D21795

Fix various "strlen(null)" PHP 8.1 issues on "bin/phd" and "bin/drydock" pathways
ClosedPublic
Actions

Authored by epriestley on May 3 2022, 6:10 PM.
Tags
None
Referenced Files
F22775501: D21795.diff
Mon, Oct 5, 11:11 PM
F22754567: D21795.id.diff
Mon, Oct 5, 12:37 AM
F22655357: D21795.diff
Thu, Oct 1, 3:03 AM
F22558107: D21795.diff
Wed, Sep 23, 7:51 PM
Unknown Object (File)
Aug 14 2026, 6:02 PM
Unknown Object (File)
Aug 8 2026, 4:55 PM
Unknown Object (File)
Aug 7 2026, 4:35 AM
Unknown Object (File)
Aug 6 2026, 7:04 PM
View All Files
Subscribers
Restricted Owners Package

Details

Reviewers
None
Maniphest Tasks
T13588: PHP 8 Compatibility
T13676: Drydock may reclaim recently-used resources
Commits
rP00a20d3cdc54: Fix various "strlen(null)" PHP 8.1 issues on "bin/phd" and "bin/drydock"…
Summary

Ref T13676. Ref T13588. Fix some issues that prevent "bin/phd" and "bin/drydock" from executing under PHP 8.1, broadly because null is being passed to strlen().

Test Plan

Ran bin/phd debug task and bin/drydock ... under PHP 8.1.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.May 3 2022, 6:10 PM
Owners added a subscriber: Restricted Owners Package.May 3 2022, 6:10 PM
epriestley requested review of this revision.May 3 2022, 6:10 PM
Harbormaster completed remote builds in B25689: Diff 51951.May 3 2022, 6:10 PM
This revision was not accepted when it landed; it landed in state Needs Review.May 3 2022, 10:57 PM
Closed by commit rP00a20d3cdc54: Fix various "strlen(null)" PHP 8.1 issues on "bin/phd" and "bin/drydock"… (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rP00a20d3cdc54: Fix various "strlen(null)" PHP 8.1 issues on "bin/phd" and "bin/drydock"….

Revision Contents
Changeset List

PathSize
src/
applications/
auth/
provider/
2 lines
cache/
management/
4 lines
daemon/
event/
7 lines
drydock/
management/
4 lines
policy/
query/
4 lines
repository/
storage/
2 lines
transactions/
storage/
2 lines
infrastructure/
customfield/
standard/
4 lines
query/
policy/
2 lines

Diff 51964

View Options

src/applications/auth/provider/PhabricatorOAuth1AuthProvider.php

Loading...
View Options

src/applications/cache/management/PhabricatorCacheManagementPurgeWorkflow.php

Loading...
View Options

src/applications/daemon/event/PhabricatorDaemonEventListener.php

Loading...
View Options

src/applications/drydock/management/DrydockManagementLeaseWorkflow.php

Loading...
View Options

src/applications/policy/query/PhabricatorPolicyQuery.php

Loading...
View Options

src/applications/repository/storage/PhabricatorRepository.php

Loading...
View Options

src/applications/transactions/storage/PhabricatorApplicationTransaction.php

Loading...
View Options

src/infrastructure/customfield/standard/PhabricatorStandardCustomFieldPHIDs.php

Loading...
View Options

src/infrastructure/query/policy/PhabricatorCursorPagedPolicyAwareQuery.php

Loading...
Phabricator · Built by Phacility