Debugger: Fix #13261.
ThreadsTableModel: - When returning field values for the stop reason column, only return valid if we actually populated it with a value (ergo, the thread was actually stopped). Otherwise, the table will attempt to perform comparisons on a BVariant which was never actually populated, leading to crashes.
This commit is contained in:
@@ -1,6 +1,6 @@
|
|||||||
/*
|
/*
|
||||||
* Copyright 2009, Ingo Weinhold, [email protected].
|
* Copyright 2009, Ingo Weinhold, [email protected].
|
||||||
* Copyright 2011, Rene Gollent, [email protected].
|
* Copyright 2011-2017, Rene Gollent, [email protected].
|
||||||
* Distributed under the terms of the MIT License.
|
* Distributed under the terms of the MIT License.
|
||||||
*/
|
*/
|
||||||
|
|
||||||
@@ -128,8 +128,9 @@ public:
|
|||||||
if (thread->State() != THREAD_STATE_RUNNING) {
|
if (thread->State() != THREAD_STATE_RUNNING) {
|
||||||
value.SetTo(thread->StoppedReasonInfo(),
|
value.SetTo(thread->StoppedReasonInfo(),
|
||||||
B_VARIANT_DONT_COPY_DATA);
|
B_VARIANT_DONT_COPY_DATA);
|
||||||
|
return true;
|
||||||
}
|
}
|
||||||
return true;
|
return false;
|
||||||
}
|
}
|
||||||
default:
|
default:
|
||||||
return false;
|
return false;
|
||||||
|
|||||||
Reference in New Issue
Block a user