diff options
| author | Brad King <brad.king@kitware.com> | 2019-08-22 10:56:55 -0400 |
|---|---|---|
| committer | Brad King <brad.king@kitware.com> | 2019-08-26 09:43:45 -0400 |
| commit | 3f4c4e7afe6075b18f99c67cf808e2300b3690c4 (patch) | |
| tree | d35747155c39491ebc97f0d9f46bbb57ce3d37b4 /Source/cmVSSetupHelper.h | |
| parent | a8ca5aea946e1154b4518ba28db3f4e695dbf165 (diff) | |
| download | cmake-3f4c4e7afe6075b18f99c67cf808e2300b3690c4.tar.gz | |
cmVSSetupHelper: Fix SmartBSTR copy operations
Fix the SmartBSTR copy constructor and copy assignment operator added by
commit 18c8278b62 (VS: Add helper class to interact with Visual Studio
Installer, 2016-12-14, v3.8.0-rc1~93^2~4) to use the string from the
source of the copy.
Issue: #19610
Diffstat (limited to 'Source/cmVSSetupHelper.h')
| -rw-r--r-- | Source/cmVSSetupHelper.h | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/Source/cmVSSetupHelper.h b/Source/cmVSSetupHelper.h index 1bda54a47b..9382c054bb 100644 --- a/Source/cmVSSetupHelper.h +++ b/Source/cmVSSetupHelper.h @@ -77,7 +77,8 @@ public: SmartBSTR(const SmartBSTR& src) { if (src.str != NULL) { - str = ::SysAllocStringByteLen((char*)str, ::SysStringByteLen(str)); + str = + ::SysAllocStringByteLen((char*)src.str, ::SysStringByteLen(src.str)); } else { str = ::SysAllocStringByteLen(NULL, 0); } @@ -87,7 +88,8 @@ public: if (str != src.str) { ::SysFreeString(str); if (src.str != NULL) { - str = ::SysAllocStringByteLen((char*)str, ::SysStringByteLen(str)); + str = + ::SysAllocStringByteLen((char*)src.str, ::SysStringByteLen(src.str)); } else { str = ::SysAllocStringByteLen(NULL, 0); } |