diff options
author | Brad King <brad.king@kitware.com> | 2007-11-05 19:34:36 (GMT) |
---|---|---|
committer | Brad King <brad.king@kitware.com> | 2007-11-05 19:34:36 (GMT) |
commit | f410f8578eea403d0ca657e46ec7659cd0f805e7 (patch) | |
tree | 09715a217fb40d51d17508c95244dbdfa6d1d399 /Source/CTest/cmCTestSubmitHandler.cxx | |
parent | cfa723d45792ef6969304fb7d9274778f662a116 (diff) | |
download | CMake-f410f8578eea403d0ca657e46ec7659cd0f805e7.zip CMake-f410f8578eea403d0ca657e46ec7659cd0f805e7.tar.gz CMake-f410f8578eea403d0ca657e46ec7659cd0f805e7.tar.bz2 |
COMP: Fix warnings on 64-bit Mac OS X build. Patch from issue #3697.
Diffstat (limited to 'Source/CTest/cmCTestSubmitHandler.cxx')
-rw-r--r-- | Source/CTest/cmCTestSubmitHandler.cxx | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/Source/CTest/cmCTestSubmitHandler.cxx b/Source/CTest/cmCTestSubmitHandler.cxx index 05a8e5a..8e7919d 100644 --- a/Source/CTest/cmCTestSubmitHandler.cxx +++ b/Source/CTest/cmCTestSubmitHandler.cxx @@ -695,7 +695,15 @@ bool cmCTestSubmitHandler::SubmitUsingXMLRPC(const cmStdString& localprefix, return false; } - size_t fileSize = st.st_size; + // off_t can be bigger than size_t. fread takes size_t. + // make sure the file is not too big. + if (st.st_size > (size_t)-1) + { + cmCTestLog(this->CTest, ERROR_MESSAGE, " File too big: " + << local_file.c_str() << std::endl); + return false; + } + size_t fileSize = static_cast<size_t>(st.st_size); FILE* fp = fopen(local_file.c_str(), "rb"); if ( !fp ) { |