helpThe GNU Bourne-Again SHell - Support: sr #109664, Bash using WIFCORED instead of...

 
 

sr #109664: Bash using WIFCORED instead of WCOREDUMP

Submitted by:  J <now3d>
Submitted on:  Fri 29 Mar 2019 11:42:19 AM UTC  
 
Category:  None Priority:  5 - Normal
Severity:  3 - Normal Status:  Wont Do
Privacy:  Public Assigned to:  None
Open/Closed:  Open Operating System:  None

Add a New Comment (Rich Markup)
   

You are not logged in

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

 

Fri 29 Mar 2019 04:15:13 PM UTC, comment #2: 

Hi Chet!
Feels like being too cautious about using the WCOREDUMP macro from /usr/include/bits/waitstatus.h

It's more risky to keep a duplicate macro with hard coded value in Bash own header WIFCORED right?
At least Bash should just use the official macro.

Everything uses the libc macro, as well as FreeBSD, NetBSD, and even Python.
https://www.gnu.org/software/libc/manual/html_node/Process-Completion-Status.html

http://man7.org/tlpi/code/online/book/procexec/print_wait_status.c.html

J <now3d>
Fri 29 Mar 2019 12:34:19 PM UTC, comment #1: 

In the absence of standardization, I'm comfortable with WIFCORED, which is consistent with WIFEXITED, WIFSIGNALED, WIFSTOPPED, and WIFCONTINUED. It's a cosmetic issue, and there's no reason to change.

Chet Ramey <chet>
Project Administrator
Fri 29 Mar 2019 11:42:19 AM UTC, original submission:  

Hi!

Bash uses WIFCORED  bash/posixwait.h

Can it be updated to remove WIFCORED macro from posixwait.h and use regular WCOREDUMP from /usr/include/bits/waitstatus.h?
http://man7.org/linux/man-pages/man2/waitpid.2.html

Note, WCOREDUMP is not in POSIX, but at least it is standard across with BSD, eventually it will be in POSIX
https://www.freebsd.org/cgi/man.cgi?query=wait&sektion=2

J <now3d>

 

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

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by chet (Posted a comment)
  • -email is unavailable- added by now3d (Submitted the item)
  •  

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

    Only logged-in users can vote.

     

     

     

    Follows 1 latest change.

    Date Changed by Updated Field Previous Value => Replaced by
    2019-03-29 chet StatusNone => Wont Do

    Back to the top


    Powered by Savane 3.5