Check open() return is non-negative integer

open() returns -1 to indicate an error, the code checked for 0 for errors.

Suggested-by: https://github.com/csutils/csmock
See-also: https://github.com/pst-format/libpst/issues/6