Fix code that probably is never run since the existing code is totally bogus. The condition, !open(...), will almost always be true. (sole exception is when starting a program with stdin initially closed) --- a/ntreg.c +++ b/ntreg.c @@ -4133,7 +4133,7 @@ if ( !(hdesc->state & HMODE_DIRTY)) return(0); if ( !(hdesc->state & HMODE_OPEN)) { /* File has been closed */ - if (!(hdesc->filedesc = open(hdesc->filename,O_RDWR))) { + if ((hdesc->filedesc = open(hdesc->filename,O_RDWR)) < 0) { fprintf(stderr,"writeHive: open(%s) failed: %s, FILE NOT WRITTEN!\n",hdesc->filename,strerror(errno)); return(1); }