This is an archive of the discontinued LLVM Phabricator instance.

Fix unsafe memory access.
Needs ReviewPublic

Authored by ruiu on Mar 14 2016, 5:50 PM.

Details

Reviewers
ruiu
Summary

Fix unsafe memory access.

Diff Detail

Event Timeline

ruiu updated this revision to Diff 50686.Mar 14 2016, 5:50 PM
ruiu updated this revision to Diff 50687.
ruiu retitled this revision from to Fix unsafe memory access..
ruiu updated this object.
ruiu added a reviewer: ruiu.
ruiu edited edge metadata.
  • update
filcab added a subscriber: filcab.Mar 15 2016, 1:12 AM

This does fix PR26908.
Please mention the bug in the commit message (you might want to mention it was a use-after-free problem too).

ruiu added a subscriber: ruiu.Mar 15 2016, 1:15 AM

This patch was just to demonstrate what I want to see so it lacked a commit
message. However, if you are okay with this, I'll write a description for
it and submit.

LGTM. Thanks for improving it!