Page MenuHomePhorge

Fix strlen(null) in DifferentialChangesetViewController loadCoverage()
ClosedPublic

Authored by Sten on Jul 4 2023, 14:19.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Jun 13, 13:48
Unknown Object (File)
Wed, Jun 12, 15:08
Unknown Object (File)
Wed, Jun 12, 09:18
Unknown Object (File)
Sun, Jun 9, 21:20
Unknown Object (File)
Sun, Jun 9, 16:53
Unknown Object (File)
Sun, Jun 9, 16:17
Unknown Object (File)
Sat, Jun 8, 14:14
Unknown Object (File)
Mon, May 27, 21:23

Details

Summary

DifferentialChangesetViewController loadCoverage() fails under PHP 8.1 when test coverage is provided for some but not all modified files.

Update the code to replace strlen() with phutil_nonempty_string().

Fixes T15521

Test Plan

View a diff with test coverage for some but not all files.

Diff Detail

Repository
rP Phorge
Branch
DifferentialChangesetViewController (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 632
Build 632: arc lint + arc unit