projects / platform / upstream / llvm.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: ee5772f) | patch
Re-order the base classes to prevent a crash in Linux.
authorHafiz Abid Qadeer <hafiz_abid@mentor.com>
Wed, 26 Nov 2014 10:19:32 +0000 (10:19 +0000)
committerHafiz Abid Qadeer <hafiz_abid@mentor.com>
Wed, 26 Nov 2014 10:19:32 +0000 (10:19 +0000)
commit05200e3b1929ce1d6f34b30beffac805b5a41b5e
tree34fa7822cfc7036a552f69f38acedcf30c68b49ctree | snapshot
parentee5772faaab01fb5328b9a53ece3c791b06210accommit | diff
Re-order the base classes to prevent a crash in Linux.

In the initialization list of IOHandlerConfirm, *this is basically casting
IOHandlerConfirm to its base IOHandlerDelegate and passing it to constructor of
IOHandlerEditline which uses it and crashes as constructor of IOHandlerDelegate
is still not called. Re-ordering the base classes makes sure that constructor of
IOHandlerDelegate runs first.

It would be good to have a test case for this case too.

llvm-svn: 222816
lldb/include/lldb/Core/IOHandler.h diff | blob | history
Domain: System / Toolchain;