Page MenuHomePhabricator

Prevent crash caused by empty array when rendering Pholio mocks
ClosedPublic

Authored by hach-que on Jul 30 2014, 1:10 AM.
Tags
None
Referenced Files
F14008738: D10073.diff
Wed, Oct 30, 5:58 AM
F14004420: D10073.id24228.diff
Sat, Oct 26, 8:22 PM
F14003789: D10073.id24228.diff
Sat, Oct 26, 10:47 AM
F14003788: D10073.id24226.diff
Sat, Oct 26, 10:47 AM
F14003787: D10073.id24254.diff
Sat, Oct 26, 10:47 AM
F13995853: D10073.id.diff
Wed, Oct 23, 4:05 PM
F13983990: D10073.id24228.diff
Sun, Oct 20, 10:09 AM
F13983415: D10073.id24254.diff
Sun, Oct 20, 6:18 AM

Details

Summary

This was causing a crash according to a conversation with @rfergu in IRC. The issue is that if $images_to_show is empty, then $image never gets set, and it attempts to call getID on a null / unset variable.

Test Plan

@rfergu confirmed this fixed the crash.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

hach-que retitled this revision from to Prevent crash caused by variable usage outside foreach loop.
hach-que updated this object.
hach-que edited the test plan for this revision. (Show Details)
hach-que added a reviewer: Blessed Reviewers.
hach-que added a subscriber: rfergu.
hach-que edited edge metadata.

Correct fix?

hach-que retitled this revision from Prevent crash caused by variable usage outside foreach loop to Prevent crash caused by empty array when rendering Pholio mocks.Jul 30 2014, 1:49 AM
hach-que updated this object.
hach-que edited the test plan for this revision. (Show Details)

@rfergu has confirmed that this fixes the crash.

hach-que added a reviewer: epriestley.
epriestley edited edge metadata.

This could just be if ($images_to_show), I think.

This revision is now accepted and ready to land.Jul 30 2014, 3:21 AM
hach-que edited edge metadata.

Changes requested

hach-que updated this revision to Diff 24254.

Closed by commit rPde40cf5ef7d6 (authored by @hach-que).