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
F18078507: D21795.diff
Mon, Aug 4, 8:10 PM
F18052316: D21795.diff
Mon, Aug 4, 2:06 AM
F18044097: D21795.diff
Sun, Aug 3, 6:47 AM
F17951127: D21795.id51951.diff
Fri, Aug 1, 1:00 AM
F17706437: D21795.diff
Wed, Jul 16, 12:30 PM
F17674624: D21795.diff
Sun, Jul 13, 7:26 AM
F17657345: D21795.diff
Sat, Jul 12, 12:16 PM
F17626181: D21795.diff
Thu, Jul 10, 4:14 AM
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