This is the mail archive of the cygwin-apps@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] High priority bugfix for setup.exe


Dave Korn wrote:
Heh, guess how nicely this:-

 virtual void visitFile(String const &basePath, const WIN32_FIND_DATA
*theFile)
   {
     String fileName(theFile->cFileName);
     if (fileName.substr(fileName.size() - 5) == ".done")
       return;

bit of code plays with the file /etc/postinstall/d.sh?  (Hint: fileName ==
"d.sh")

 The attached patch fixes the access violation that results from substr
indexing off the front of the string.  It builds and fixes the problem in
testing.

I am confused.


First, the quoted code should not cause any error, since this is a C++ string class, not direct pointer indexing.

Second, I just installed the 'd' package, and the postinstall script was successfully executed.

Max.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]