<div dir="ltr">Thanx, The invalid iterator problem was only in the last iteration, but I think you modified all the For Loops.</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 20, 2015 at 12:13 AM, Greg Clayton <span dir="ltr"><<a href="mailto:gclayton@apple.com" target="_blank">gclayton@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I fixed this:<br>
<br>
% svn commit<br>
Sending        include/lldb/Symbol/TypeMap.h<br>
Sending        source/Symbol/SymbolContext.cpp<br>
Sending        source/Symbol/TypeMap.cpp<br>
Transmitting file data ...<br>
Committed revision 253618.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
> On Nov 18, 2015, at 12:54 AM, Ravitheja Addepally via lldb-dev <<a href="mailto:lldb-dev@lists.llvm.org">lldb-dev@lists.llvm.org</a>> wrote:<br>
><br>
> Hello,<br>
>      Pavel- my question is how do we solve it ? should Mikhail log a bug ? for this issue<br>
><br>
> BR,<br>
> A Ravi Theja<br>
><br>
> On Tue, Nov 17, 2015 at 10:42 AM, Ravitheja Addepally <<a href="mailto:ravithejawork@gmail.com">ravithejawork@gmail.com</a>> wrote:<br>
> Hello,<br>
><br>
>                 Yeah you are right Mikhail, thanks for pointing it out, I must ask, is there any bug already logged for this issue ?<br>
><br>
> Ravi<br>
><br>
><br>
> On Mon, Nov 16, 2015 at 5:24 PM, Mikhail Filimonov via lldb-dev <<a href="mailto:lldb-dev@lists.llvm.org">lldb-dev@lists.llvm.org</a>> wrote:<br>
> Hi guys and thank you for the excellent community project!<br>
><br>
><br>
><br>
> Recently I’ve stumbled on a pesky, but trivial Invalid iterator dereference bug in SymbolContext and TypeMap implementations at revisions<br>
><br>
> <a href="https://github.com/llvm-mirror/lldb/blob/e528da256d14ecac7df858462b44dca931879509/source/Symbol/SymbolContext.cpp#L823" rel="noreferrer" target="_blank">https://github.com/llvm-mirror/lldb/blob/e528da256d14ecac7df858462b44dca931879509/source/Symbol/SymbolContext.cpp#L823</a><br>
><br>
> and<br>
><br>
> <a href="https://github.com/llvm-mirror/lldb/blob/5ac1fc5bc961688505334395598a2bb174eabd3b/source/Symbol/TypeMap.cpp#L172" rel="noreferrer" target="_blank">https://github.com/llvm-mirror/lldb/blob/5ac1fc5bc961688505334395598a2bb174eabd3b/source/Symbol/TypeMap.cpp#L172</a><br>
><br>
><br>
><br>
> From the code below it is obvious that TypeMap::ForEach calls the pre-increment operator on m_types iterator right after it has been invalidated by m_types.erase<br>
><br>
><br>
><br>
> SymbolContext::SortTypeList(TypeMap &type_map, TypeList &type_list ) const<br>
><br>
> {<br>
><br>
>         TypeMaptoList callbackM2L (type_map, type_list);<br>
><br>
>         type_map.ForEach(callbackM2L);<br>
><br>
>                 return ;<br>
><br>
> }<br>
><br>
><br>
><br>
> void<br>
><br>
> TypeMap::ForEach (std::function <bool(lldb::TypeSP &type_sp)> const &callback)<br>
><br>
> {<br>
><br>
>     for (auto pos = m_types.begin(), end = m_types.end(); pos != end; ++pos)<br>
><br>
>     {<br>
><br>
>         if (!callback(pos->second))<br>
><br>
>             break;<br>
><br>
>     }<br>
><br>
> }<br>
><br>
><br>
><br>
> bool<br>
><br>
> TypeMap::RemoveTypeWithUID (user_id_t uid)<br>
><br>
> {<br>
><br>
>     iterator pos = m_types.find(uid);<br>
><br>
><br>
><br>
>     if (pos != m_types.end())<br>
><br>
>     {<br>
><br>
>         m_types.erase(pos);<br>
><br>
>         return true;<br>
><br>
>     }<br>
><br>
>     return false;<br>
><br>
> }<br>
><br>
><br>
><br>
> class TypeMaptoList<br>
><br>
> {<br>
><br>
> public:<br>
><br>
>     TypeMaptoList(TypeMap &typem, TypeList &typel) :<br>
><br>
>         type_map(typem),type_list(typel)<br>
><br>
>     {<br>
><br>
>     }<br>
><br>
><br>
><br>
>     bool<br>
><br>
>     operator() (const lldb::TypeSP& type)<br>
><br>
>     {<br>
><br>
>         if(type)<br>
><br>
>         {<br>
><br>
>             type_list.Insert(type);<br>
><br>
>             type_map.RemoveTypeWithUID(type->GetID());<br>
><br>
>             if (type_map.Empty())<br>
><br>
>                 return false;<br>
><br>
>         }<br>
><br>
>         return true;<br>
><br>
>     }<br>
><br>
><br>
><br>
> private:<br>
><br>
>     TypeMap &type_map;<br>
><br>
>     TypeList &type_list;<br>
><br>
> };<br>
><br>
><br>
><br>
> Regards,<br>
><br>
> Mikhail Filimonov<br>
><br>
><br>
><br>
><br>
><br>
><br>
><br>
</div></div><span class="im HOEnZb">> This email message is for the sole use of the intended recipient(s) and may contain confidential information.  Any unauthorized review, use, disclosure or distribution is prohibited.  If you are not the intended recipient, please contact the sender by reply email and destroy all copies of the original message.<br>
><br>
</span><div class="HOEnZb"><div class="h5">> _______________________________________________<br>
> lldb-dev mailing list<br>
> <a href="mailto:lldb-dev@lists.llvm.org">lldb-dev@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev</a><br>
><br>
><br>
><br>
> _______________________________________________<br>
> lldb-dev mailing list<br>
> <a href="mailto:lldb-dev@lists.llvm.org">lldb-dev@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev</a><br>
<br>
</div></div></blockquote></div><br></div>