Page MenuHomePhorge

Fix strlen(null) in DifferentialChangesetViewController loadCoverage()
ClosedPublic

Authored by Sten on Jul 4 2023, 14:19.
Tags
None
Referenced Files
F3212056: D25328.1741447627.diff
Fri, Mar 7, 15:27
F2982835: D25328.1739971963.diff
Tue, Feb 18, 13:32
F2982744: D25328.1739971748.diff
Tue, Feb 18, 13:29
F2982573: D25328.1739963371.diff
Tue, Feb 18, 11:09
F2982571: D25328.1739963369.diff
Tue, Feb 18, 11:09
F2982532: D25328.1739962514.diff
Tue, Feb 18, 10:55
F2982468: D25328.1739960387.diff
Tue, Feb 18, 10:19
F2979680: D25328.1739868662.diff
Mon, Feb 17, 08:51

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