mainThe 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

Post a Comment

Add a New Comment Rich Markup
   

Discussion

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.o ... etion-Status.html

http://man7.org/t ... ait_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>
Group 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/l ... n2/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.freeb ... ait&sektion=2

J <now3d>

 

Attached Files

This item currently has no attached files.

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

Attach Files:
   
   
Comment:
   

 

Dependencies

This item does not depend on any other items.

No items depend on this one.

 

Mail Notification Carbon-Copy List

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

    Votes

    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.

     

    History

    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.16-ed84.
    Corresponding source code