Re: different CVS_SERVER for different hosts

[Mihai Bazon]

On Wed, 2003-10-29 at 11:57, Mark D. Baushke wrote:
> I sent private code review comments to Mihai, but it is worth noting one
> that is a 'security' concern. Do not assume that the file is not
> a zero-length file. Check that the len > 0 before potentially changing
> '\n' bytes into '\0' bytes at the end of it.

Sorry to have to contradict you about the above.  My code looks like
this:

            char *res = fgets(buf, sizeof(buf), f);
            fclose(f);
            if (!res)
                // forget it
                break;

For a zero-length file fgets returns NULL so we exit before removing the
last newline character.

Breaking C89 guidelines is, well, something I usually don't care..  I am
a former C++ programmer and this gave me some real bad habits--or good,
depends on how you're seeing it ;-)

But thanks for your notes, I'll keep them in mind if I submit other
code.

Regards,
-- 
Mihai Bazon,
http://dynarch.com/mishoo/   [ applied W3 ]