[patch] cygcheck.cc update for cygpath()

Brian Dessent brian@dessent.net
Tue Mar 11 17:09:00 GMT 2008


Corinna Vinschen wrote:

> Btw., you don't need to make the buffers MAX_PATH + 1.  MAX_PATH is
> defined including the trailing NUL.  Existing code shows a lot of
> irritation about this...

Oh, I wasn't even thinking of that... the reason I used MAX_PATH + 1 was
because earlier I had written

+      static char tmp[SYMLINK_MAX + 1];

so that the following sizes would not need to be SYMLINK_MAX - 1, 

+      if (!readlink (fh, tmp, SYMLINK_MAX))

+	  strncpy (tmp, cygpath (papp, NULL), SYMLINK_MAX);

+	  strncpy (lastsep+1, ptr, SYMLINK_MAX - (lastsep-tmp));


I.e. pure lazyness of wanting to type the least necessary.  But now that
you mention it, it makes more sense to have the "- 1" than the "+ 1"
form, so I'll change that.

Brian



More information about the Cygwin-patches mailing list