bugattr - Bugs: bug #33417, use reasonable exit statuses

 
 

bug #33417: use reasonable exit statuses

Submitter:  Christoph Anton Mitterer <calestyo>
Submitted:  Fri 27 May 2011 08:13:49 PM UTC
   
 
Category:  None Severity:  3 - Normal
Item Group:  None Status:  None
Privacy:  Public Assigned to:  None
Open/Closed:  Open
* Mandatory Fields

Add a New Comment Rich Markup
   

Sun 14 Jan 2024 01:30:18 AM UTC, comment #3: 

feel free to put together a patch & send it to the mailing list for review/discussion

Mike Frysinger <vapier>
Group Member
Sat 07 Oct 2023 02:11:37 PM UTC, comment #2: 

What's IMO still open is:

There should be a separate exit status at least for the case when an XATTR is just not set, but apart from that, no errors (like non-existent file, etc.) occurred.

Christoph Anton Mitterer <calestyo>
Fri 23 Jun 2023 11:26:36 PM UTC, comment #1: 

For the records, I think at least the part "Trying to read an XATTR that doesn't exist" has been fixed meanwhile by commit 93c92ed1d826c6759bb83d2168ee818a05280f35.

Haven't checked the other points, though. I guess actually that at least all the read cases are fixed.

Christoph Anton Mitterer <calestyo>
Fri 27 May 2011 08:13:49 PM UTC, original submission:  

Hi.

Currently it seems to me that setfattr/getfattr don't use exit statuses != 0 at all.
Could you please add this (very important) functionality?


Exit codes (!= 0) should at least be provided for the following cases:
- Trying to read/write XATTRs when not supported by the underlying filesystem.
- Trying to read/write XATTRs when not allowed (e.g. not the "user" namespace, or on files that are not regular or not directories.
- Trying to read an XATTR that doesn't exist (currently it's not easily possible to determine whether an XATTR is set or not.
- etc.

Cheers,
Chris.

Christoph Anton Mitterer <calestyo>

 

(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 vapier (Posted a comment)
  • -email is unavailable- added by calestyo (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.

     

    No changes have been made to this item

    Back to the top

    Powered by Savane 3.13-5884.
    Corresponding source code