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

 
 

sr #109664: Bash using WIFCORED instead of WCOREDUMP

Submitter:  J <now3d>
Submitted:  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
* Mandatory Fields

Add a New Comment (Rich Markup)
   

Comment Type & Canned Response:
       

 

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)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    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.9