<div dir="ltr"><div dir="ltr">Thanks Shafik, addressed in ae8a5c68523c1d23dec721e28f89084d6561522a <br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Apr 13, 2021 at 9:56 AM Shafik Yaghmour <<a href="mailto:syaghmour@apple.com">syaghmour@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">I might be missing something here but I think <br>
<br>
m_launch_flavor<br>
<br>
Is also uninitialized. <br>
<br>
It looks like using in class member initialization would be a better fix for any case where the constructor is just a member initialization list with an empty body.<br>
<br>
> On Apr 13, 2021, at 9:47 AM, Jonas Devlieghere via lldb-commits <<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a>> wrote:<br>
> <br>
> <br>
> Author: Jonas Devlieghere<br>
> Date: 2021-04-13T09:46:59-07:00<br>
> New Revision: 8a5af9e28443ce8290388439f9e36cf2727d7761<br>
> <br>
> URL: <a href="https://github.com/llvm/llvm-project/commit/8a5af9e28443ce8290388439f9e36cf2727d7761" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/8a5af9e28443ce8290388439f9e36cf2727d7761</a><br>
> DIFF: <a href="https://github.com/llvm/llvm-project/commit/8a5af9e28443ce8290388439f9e36cf2727d7761.diff" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/8a5af9e28443ce8290388439f9e36cf2727d7761.diff</a><br>
> <br>
> LOG: [debugserver] Fix unintialized member variable<br>
> <br>
> Caught by ubsan (__ubsan_handle_load_invalid_value_abort) when running<br>
> the unit tests.<br>
> <br>
> Added: <br>
> <br>
> <br>
> Modified: <br>
>    lldb/tools/debugserver/source/RNBContext.h<br>
> <br>
> Removed: <br>
> <br>
> <br>
> <br>
> ################################################################################<br>
> diff  --git a/lldb/tools/debugserver/source/RNBContext.h b/lldb/tools/debugserver/source/RNBContext.h<br>
> index 0b46151e47857..03cd7f350e63b 100644<br>
> --- a/lldb/tools/debugserver/source/RNBContext.h<br>
> +++ b/lldb/tools/debugserver/source/RNBContext.h<br>
> @@ -46,7 +46,8 @@ class RNBContext {<br>
>   RNBContext()<br>
>       : m_pid(INVALID_NUB_PROCESS), m_pid_stop_count(0),<br>
>         m_events(0, all_event_bits), m_pid_pthread(), m_launch_status(),<br>
> -        m_arg_vec(), m_env_vec(), m_detach_on_error(false) {}<br>
> +        m_arg_vec(), m_env_vec(), m_detach_on_error(false),<br>
> +        m_unmask_signals(false) {}<br>
> <br>
>   virtual ~RNBContext();<br>
> <br>
> @@ -148,11 +149,11 @@ class RNBContext {<br>
>   std::string m_working_directory;<br>
>   std::string m_process_event;<br>
>   bool m_detach_on_error;<br>
> +  bool m_unmask_signals;<br>
> <br>
>   void StartProcessStatusThread();<br>
>   void StopProcessStatusThread();<br>
>   static void *ThreadFunctionProcessStatus(void *arg);<br>
> -  bool m_unmask_signals;<br>
> <br>
> private:<br>
>   RNBContext(const RNBContext &rhs) = delete;<br>
> <br>
> <br>
> <br>
> _______________________________________________<br>
> lldb-commits mailing list<br>
> <a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a><br>
> <a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits</a><br>
<br>
</blockquote></div>