Page MenuHomePhabricator

[scudo][standalone] Lists fix
ClosedPublic

Authored by cryptoad on Oct 28 2019, 1:28 PM.

Details

Summary

Apparently during the review of D69265, and my flailing around with
git, a somewhat important line disappeared.

On top of that, there was no test exercising that code path, and
while writing the follow up patch I intended to write, some CHECKs
were failing.

Re-add the missing line, and add a test that fails without said line.

Event Timeline

cryptoad created this revision.Oct 28 2019, 1:28 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptOct 28 2019, 1:28 PM
Herald added a subscriber: Restricted Project. · View Herald Transcript
cryptoad updated this revision to Diff 226745.Oct 28 2019, 1:36 PM

Expanding the new test a bit.

hctim accepted this revision.Oct 28 2019, 1:39 PM
This revision is now accepted and ready to land.Oct 28 2019, 1:39 PM
This revision was automatically updated to reflect the committed changes.