Changeset 2144 for trunk/source/punch_fits.cpp

Timestamp:

07/01/08 17:30:06 (5 months ago)

Author:

peter

Message:

source/punch_fits.cpp:

Bug-fix: fix another potential array overrun while copying the chVersion string.

Files:

• trunk/source/punch_fits.cpp (modified) (1 diff)

trunk/source/punch_fits.cpp

@@ -737 +737 @@
 
         strcpy( tempString, "Generated by Cloudy " );
-        strcat( tempString, version::Inst().chVersion );
+        // strncat guarantees that terminating 0 byte will always be written...
+        strncat( tempString, version::Inst().chVersion, sizeof(tempString)-strlen(tempString) );
         bytesAdded += addComment( tempString );
         bytesAdded += addComment( version::Inst().chInfo );