Jeremy Morse 2eb96e1711 [DebugInfo][NFC] Avoid a use-after-free ...

This patch swaps two lines -- the CurSucc reference can be invalidated
by the call to DFS.push_back, therefore that should happen last. The
usual hat-tip to asan for catching this.

This patch also swaps an ealier call to ToAdd.insert and DFS.push_back,
where a stable iterator (from successors()) is being used. This isn't
strictly necessary, but is good for consistency and avoiding readers
asking themselves why the two code portions have a different order.

2021-10-25 14:16:30 +01:00

125 KiB

Raw Blame History

View Raw