You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by xi...@apache.org on 2021/12/03 07:17:27 UTC

[incubator-nuttx] branch master updated: sched_waitpid:delete debugassert

This is an automated email from the ASF dual-hosted git repository.

xiaoxiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git


The following commit(s) were added to refs/heads/master by this push:
     new 9f4d236  sched_waitpid:delete debugassert
9f4d236 is described below

commit 9f4d236531e584a049250600aa9cbe4454948970
Author: anjiahao <an...@xiaomi.com>
AuthorDate: Thu Dec 2 11:29:17 2021 +0800

    sched_waitpid:delete debugassert
    
    If a task registers SIGCHLD, uses waitpid(-1)
    in the handler, and the task also calls waitpid,
    it will fail to delete the child function
    
    Signed-off-by: anjiahao <an...@xiaomi.com>
---
 sched/sched/sched_waitpid.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/sched/sched/sched_waitpid.c b/sched/sched/sched_waitpid.c
index d13b718..92d4d10 100644
--- a/sched/sched/sched_waitpid.c
+++ b/sched/sched/sched_waitpid.c
@@ -424,7 +424,6 @@ pid_t nx_waitpid(pid_t pid, int *stat_loc, int options)
               /* Recover the exiting child */
 
               child = group_find_child(rtcb->group, info.si_pid);
-              DEBUGASSERT(child != NULL);
 
               /* Discard the child entry, if we have one */