tamatebako / libdwarfs

C++ library to access DwarFS images
2 stars 1 forks source link

Invalid filesystem data causes program abort #75

Open maxirmx opened 2 years ago

maxirmx commented 2 years ago

On linux platform (with GCC and clang) invalid filesystem data causes program abort (exception generated in dwarfs library fails be handled outside) Failing tests:

It is a minor issue for Tebako packager since in a case of broken filesystem the execution has to be terminated on way or another

ronaldtse commented 2 years ago

Is this during compilation (on developer machine) or on execution (on client machine)?

maxirmx commented 2 years ago

It is during execution. It happens if filesystem image is broken which should not happen if tebako is used. If someone packages manually he can put there some broken file. In this case exception is not handled and the package terminates with message "Aborted" instead of "Invalid file system", "No filesysten found", ... I do not think it is a real issue for tebako case

ronaldtse commented 2 years ago

If someone packages manually he can put there some broken file. In this case exception is not handled and the package terminates with message "Aborted" instead of "Invalid file system", "No filesysten found", ...

Agree. We just need to fail gracefully with a reasonable message.