Page MenuHomePhorge

Fix $boot_length comparison in PhagePHPAgentBootloader
ClosedPublic

Authored by aklapper on Aug 11 2024, 20:48.
Tags
None
Referenced Files
F3350005: D25770.1743761753.diff
Thu, Apr 3, 10:15
F3349963: D25770.1743761438.diff
Thu, Apr 3, 10:10
F3336660: D25770.1743577912.diff
Tue, Apr 1, 07:11
F3336539: D25770.1743576231.diff
Tue, Apr 1, 06:43
F3329946: D25770.1743482248.diff
Mon, Mar 31, 04:37
F3328906: D25770.1743465110.diff
Sun, Mar 30, 23:51
F3328044: D25770.1743449436.diff
Sun, Mar 30, 19:30
F3327326: D25770.1743436797.diff
Sun, Mar 30, 15:59

Details

Summary

$boot_length = strlen($boot_sequence->toString()) returns an int and strlen() expects a string as a parameter. Thus calling if (strlen($boot_length) > 8192) afterwards to get the string length of an integer makes no sense.

Test Plan

Read the code.

Diff Detail

Repository
rARC Arcanist
Branch
bootLengthStrlen
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 1514
Build 1514: arc lint + arc unit