Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Panic on unwrapping of Err value #2

Open
daniellockyer opened this issue Mar 28, 2017 · 0 comments
Open

Panic on unwrapping of Err value #2

daniellockyer opened this issue Mar 28, 2017 · 0 comments

Comments

@daniellockyer
Copy link

Found using cargo-fuzz.

This library panics at src/lib.rs:82 if the file path requested does not exist.

Whilst in theory this is correct, I think the library should return a Result to indicate success or failure. Of course, this means changing the API.

As a side note, if changing the API is okay, it might be a good idea to switch to taking a PathBuf reference.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant