bugmake - Bugs: bug #51400, SIGCHLD gets unblocked

 
 

bug #51400: SIGCHLD gets unblocked

Submitted by:  Koen Van Hoof <kvho>
Submitted on:  Thu 06 Jul 2017 09:29:48 AM UTC  
 
Severity: 3 - NormalItem Group: Bug
Status: FixedPrivacy: Public
Assigned to: Paul D. Smith <psmith>Open/Closed: Closed
Component Version: 4.2.1Operating System: POSIX-Based
Fixed Release: SCMTriage Status: Medium Effort

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Mon 10 Jul 2017 01:58:48 AM UTC, comment #1:

Thanks for the report!

The change needs to be slightly more complicated, since there are some parts of the code (after fork, in the child process) where we really do want to unblock all signals.

The fix has been pushed to Git and will be available in the next release.

Paul D. Smith <psmith>
Project AdministratorIn charge of this item.
Thu 06 Jul 2017 09:29:48 AM UTC, original submission:

in main.c SIGCHLD is blocked.
in unblock_sigs in job.c, the signal is unblocked again.

solution:
diff --git a/src/job.c b/src/job.c
--- a/src/job.c
+++ b/src/job.c
@@ -1051,9 +1051,7 @@ block_sigs (void)
void
unblock_sigs (void)
{
- sigset_t empty;
- sigemptyset (&empty);
- sigprocmask (SIG_SETMASK, &empty, (sigset_t *) 0);
+ (void) sigprocmask (SIG_UNBLOCK, &fatal_signal_set, (sigset_t *) 0);
}
#endif

Koen Van Hoof <kvho>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by psmith (Posted a comment)
  • -unavailable- added by kvho (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon 10 Jul 2017 01:58:48 AM UTCpsmithStatusNone=>Fixed
      Assigned toNone=>psmith
      Open/ClosedOpen=>Closed
      Fixed ReleaseNone=>SCM
      Triage StatusNone=>Medium Effort

    Back to the top


    Powered by Savane 3.1-cleanup1