public inbox for gdb-prs@sourceware.org
help / color / mirror / Atom feed
* [Bug gdb/17229] New: infinite loop in linux_check_ptrace_features
@ 2014-08-05 16:53 mageofdancingdragons at gmail dot com
  2014-08-05 16:59 ` [Bug gdb/17229] " mageofdancingdragons at gmail dot com
  0 siblings, 1 reply; 2+ messages in thread
From: mageofdancingdragons at gmail dot com @ 2014-08-05 16:53 UTC (permalink / raw)
  To: gdb-prs

https://sourceware.org/bugzilla/show_bug.cgi?id=17229

            Bug ID: 17229
           Summary: infinite loop in linux_check_ptrace_features
           Product: gdb
           Version: 7.7
            Status: NEW
          Severity: normal
          Priority: P2
         Component: gdb
          Assignee: unassigned at sourceware dot org
          Reporter: mageofdancingdragons at gmail dot com

In linux_check_ptrace_features there's an infinite loop possibility when the
child dies unexpectedly. 

332   child_pid = linux_fork_to_function (NULL, linux_child_function);
333       
334   ret = my_waitpid (child_pid, &status, 0);

After these two calls complete successfully, there is a child process, and the
value of status fulfills WIFSTOPPED(status).

Now the child process gets killed (for some reason or another).

Finally we hit the cleanup loop at the end:

349   do
350     {
351       ret = ptrace (PTRACE_KILL, child_pid, (PTRACE_TYPE_ARG3) 0,
352                     (PTRACE_TYPE_ARG4) 0);
353       if (ret != 0)
354         warning (_("linux_check_ptrace_features: failed to kill child"));
355       my_waitpid (child_pid, &status, 0);
356     }
357   while (WIFSTOPPED (status));

both ptrace and my_waitpid should fail because the child_pid is no longer a
valid process, but the value of status never changes because waitpid is
returning an error.

The return value of waitpid should be checked and the loop exited if it fails
(with appropriate error reporting).

-- 
You are receiving this mail because:
You are on the CC list for the bug.


^ permalink raw reply	[flat|nested] 2+ messages in thread

* [Bug gdb/17229] infinite loop in linux_check_ptrace_features
  2014-08-05 16:53 [Bug gdb/17229] New: infinite loop in linux_check_ptrace_features mageofdancingdragons at gmail dot com
@ 2014-08-05 16:59 ` mageofdancingdragons at gmail dot com
  0 siblings, 0 replies; 2+ messages in thread
From: mageofdancingdragons at gmail dot com @ 2014-08-05 16:59 UTC (permalink / raw)
  To: gdb-prs

https://sourceware.org/bugzilla/show_bug.cgi?id=17229

--- Comment #1 from Daniel Hoffman <mageofdancingdragons at gmail dot com> ---
note: this will occur on systems where PPTRACE_O_TRACEFORK does not cause an
error, but the child process does not stop after forking, which is explicitly
mentioned in the comments as a possibility.

-- 
You are receiving this mail because:
You are on the CC list for the bug.


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2014-08-05 16:59 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-08-05 16:53 [Bug gdb/17229] New: infinite loop in linux_check_ptrace_features mageofdancingdragons at gmail dot com
2014-08-05 16:59 ` [Bug gdb/17229] " mageofdancingdragons at gmail dot com

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).