Comments (5)
my suggested fix is one line: change strcpy, that MSVS is complaining about,
to memcpy.
old:
strcpy(clone, c_string);
new:
memcpy(clone, c_string, len + 1);
patch file is attached.
Original comment by [email protected]
on 6 Oct 2008 at 6:43
Attachments:
from googletest.
Original comment by [email protected]
on 8 Oct 2008 at 9:10
- Added labels: OpSys-Windows, Priority-Low
- Removed labels: Priority-Medium
from googletest.
Keith,
I am putting the change in, together with another sample fix (that one for VS
7.1).
They may take a week or two to filter to svn, though -- we are busy with the
other
things right now.
Original comment by vladlosev
on 9 Oct 2008 at 1:02
from googletest.
Original comment by [email protected]
on 12 Oct 2008 at 3:21
- Added labels: Type-Enhancement
- Removed labels: Type-Defect
from googletest.
Fixed in revision 135.
Original comment by vladlosev
on 22 Nov 2008 at 6:08
- Changed state: Fixed
from googletest.
Related Issues (20)
- Unable to determine if a test should run before calling RUN_ALL_TESTS HOT 2
- My assert is not caught and the test fails HOT 2
- Throwing an exception as default breaks call count expectation HOT 3
- I put my tests in a library and Google Test doesn't run them - GCC 4.8 HOT 3
- Unknown option specified: gtest_output
- error:SEH exception with code 0xc0000005 thrown in auxiliary test code<environment or event listeners>
- Compilation errors on MinGW
- Duplicate Symbols adding Samples (Xcode)
- PATCH: Fix ‘make check’ in separate build directory
- SEGV on 4 or more test cases with Address Sanitizer HOT 5
- Add the ability to specify parameterized test names
- moving to GITHUB ?
- Plase add a way to skip/disable some tests during runtime.
- README CMake section missing step of putting built libraries into lib subdirectory
- New release
- memory leak when gtest are printing
- Include BUILD for building with bazel
- Consider #define NOMINMAX before including windows.h
- Make gtest usable with -fvisibility=hidden HOT 2
- Linking error in google-test sample using arm platform
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from googletest.