You want -1 here instead of 1?  Or do we exit with -1, 0, and 1, depending on failure type?