[RFA 5/5 v3] Darwin: fix SIGTRAP when debugging

[Xavier Roirand]

Thanks. Pushed to master.

https://sourceware.org/git/?p=binutils-gdb.git;a=commit;h=de1ec836c8cc4eeb68e3e4eb8223eeff9840546e

Le 9/10/18 à 2:52 PM, Simon Marchi a écrit :
> On 2018-09-10 13:32, Xavier Roirand wrote:
>> Debugging a program under Darwin does not work:
>>
>> (gdb) start
>> Temporary breakpoint 1 at 0x100000fb4: file /tmp/helloworld.c, line 1.
>> Starting program: /private/tmp/helloworld
>> [New Thread 0x2903 of process 60326]
>> During startup program terminated with signal SIGTRAP, Trace/breakpoint
>> trap.
>>
>> Field signaled from darwin_thread_info is not initialized thus signal
>> sent to the debuggee is considered as not sent by GDB whereas it should.
>>
>> This patch fixes this problem and also updates (change type and/or
>> initialize) other fields in the same structure at the same time.
>>
>> gdb/ChangeLog:
>>
>> Â Â Â Â Â Â Â  * darwin-nat.h (struct darwin_thread_info) <gdb_port,
>> Â Â Â Â inf_port, msg_state>: Initialize.
>> Â Â Â Â Â Â Â  (struct darwin_thread_info) <signaled, single_step>: Change
>> Â Â Â Â Â Â Â  type and initialize.
>> Â Â Â Â Â Â Â  (struct darwin_thread_info) <event>: Initialize.
>>
>> Change-Id: I0fe2a6985df9d0dfcc8a2a258a3ef70cfa19b403
>> ---
>> Â gdb/darwin-nat.h | 12 ++++++------
>> Â 1 file changed, 6 insertions(+), 6 deletions(-)
>>
>> diff --git a/gdb/darwin-nat.h b/gdb/darwin-nat.h
>> index 5965df08ab6..89a7c06f6a1 100644
>> --- a/gdb/darwin-nat.h
>> +++ b/gdb/darwin-nat.h
>> @@ -114,25 +114,25 @@ enum darwin_msg_state
>> Â struct darwin_thread_info : public private_thread_info
>> Â {
>>    /* The thread port from a GDB point of view.  */
>> -Â  thread_t gdb_port;
>> +Â  thread_t gdb_port = 0;
>>
>>    /* The thread port from the inferior point of view.  Not to be used 
>> inside
>>       gdb except for get_ada_task_ptid.  */
>> -Â  thread_t inf_port;
>> +Â  thread_t inf_port = 0;
>>
>> Â Â  /* Current message state.
>> Â Â Â Â Â  If the kernel has sent a message it expects a reply and the 
>> inferior
>>       can't be killed before.  */
>> -Â  enum darwin_msg_state msg_state;
>> +Â  enum darwin_msg_state msg_state = DARWIN_RUNNING;
>>
>>    /* True if this thread is single-stepped.  */
>> -Â  unsigned char single_step;
>> +Â  bool single_step = false;
>>
>>    /* True if a signal was manually sent to the thread.  */
>> -Â  unsigned char signaled;
>> +Â  bool signaled = false;
>>
>>    /* The last exception received.  */
>> -Â  struct darwin_exception_msg event;
>> +Â  struct darwin_exception_msg event {};
>> Â };
>> Â typedef struct darwin_thread_info darwin_thread_t;
> 
> LGTM.  You do have push access I believe, is that right?
> 
> Simon