bugLiberty Eiffel - Bugs: bug #42990, no error handling implemented for...

 
 

bug #42990: no error handling implemented for io plugin (some commands in class FILE_TOOLS silently fail)

Submitter:  Raphael Mack <ramack>
Submitted:  Thu 14 Aug 2014 08:19:11 PM UTC
   
 
Category:  Kernel libraries Severity:  3 - Normal
Priority:  * 5 - Normal Status:  Fixed
Assigned to:  cadrian Open/Closed:  Closed
Release:  adler Fixed Release:  bell
Planned Release:  bell
* Mandatory Fields

Add a New Comment Rich Markup
   

Thu 29 Jan 2015 03:01:05 PM UTC, comment #1: 

Fixed by ccd2dcbfb21f046de7903457e2866556279bd1a2

Cyril Adrian <cadrian>
Group administrator
Thu 14 Aug 2014 08:19:11 PM UTC, original submission:  

in function io_copy (POSIX and non-posix variants) no error handling is implemented in case the target file cannot be copied.

Similar for FILE_TOOLS.delete and FILE_TOOLS.rename.

What about adding a query last_operation_succeeded: BOOLEAN?

Thanks to Hans for finding this by compiling on windows. (Compiler warning by PellesC) But the warning-free posix variant is also not clean, because retrying with "one byte more" in case of errors is definitely not the way to handle "-1" return values ;-)

Raphael Mack <ramack>
Group administrator

 

(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 ramack (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.

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2015-01-29 cadrian StatusIn Progress Fixed
        Open/ClosedOpen Closed
        Fixed ReleaseNone bell
    2015-01-29 cadrian StatusNone In Progress
        Assigned toNone cadrian
        ReleaseNone adler

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code