New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

check for 0 length data file causes exception #2

Open

andreasplesch opened this issue Dec 18, 2017 · 1 comment · May be fixed by #3

andreasplesch commented Dec 18, 2017 •

edited

Loading

In the checkNRRD function line
https://github.com/scijs/nrrd-js/blob/master/nrrd.js#L1131
causes a problem if ret.data is undefined. Then accessing ret.data.length is not possible and causes a type error.

Author

andreasplesch commented Dec 18, 2017

Although it would be silly, a no data nrrd file may be legal ?

andreasplesch linked a pull request

that will close this issue

avoid type errors when checking #3

Open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment