[patch] Add missing call to restore_child_signals_mask.
Pedro Alves
pedro@codesourcery.com
Tue May 3 10:06:00 GMT 2011
On Tuesday 03 May 2011 04:21:13, Doug Evans wrote:
> Hi.
>
> Perhaps there's an obscure reason why this patch is wrong,
> but I can't think of one.
Neither can I. No objections, thanks.
> [The function calls block_child_signals at the start - all exit
> paths should, presumably, call restore_child_signals_mask before returning.]
>
> I will check it in in a few days if there are no objections.
>
> 2011-05-02 Doug Evans <dje@google.com>
>
> * linux-nat.c (lin_lwp_attach_lwp): For !WIPSTOPPED case,
> adding missing call to restore_child_signals_mask.
>
> Index: linux-nat.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/linux-nat.c,v
> retrieving revision 1.202
> diff -u -p -r1.202 linux-nat.c
> --- linux-nat.c 29 Apr 2011 15:54:12 -0000 1.202
> +++ linux-nat.c 3 May 2011 03:17:33 -0000
> @@ -1476,7 +1476,10 @@ lin_lwp_attach_lwp (ptid_t ptid)
>
> status = linux_nat_post_attach_wait (ptid, 0, &cloned, &signalled);
> if (!WIFSTOPPED (status))
> - return -1;
> + {
> + restore_child_signals_mask (&prev_mask);
> + return -1;
> + }
>
> lp = add_lwp (ptid);
> lp->stopped = 1;
>
--
Pedro Alves
More information about the Gdb-patches
mailing list