C++/CLI System::Stringをcharに変換

Question

起きている現象

C++/CLI System::Stringをcharに変換したいのですが、
下記の[method1]のようなコードでchar変換を行い、
memcpyでchar型の変数にコピーしようとすると
プログラムがクラッシュしてしまいました。

自分で調べたこと、試したこと

##(1) char変換処理の変更
以下のサイトを参考に、std::stringを使わず、
System::Stringをcharに変換する処理を作ると
memcpyでエラーとならずコピーができました。[method2]

(参考) https://support.microsoft.com/ja-jp/help/311259/how-to-convert-from-system-string-to-char-in-visual-c

##(2) char変換結果の比較
下記のコードのように、method1とmethod2の結果を比較すると、
異なっていることがわかりました。(終端文字以降の部分で)

知りたいこと

method1を使用してchar型文字列に変換すると、
サイズ超過、メモリ破壊　など 起きうること、危険性について教えてください。

また、どのようなコードの書き方が望ましいのか、教えていただけないでしょうか。

#実験に使用したコード

#include "stdafx.h"
#include <string>
#include <msclr/marshal_cppstd.h>
#include <time.h>

using namespace System;
using namespace System::Data;
using namespace System::Runtime::InteropServices;

// System::String^→std::stringへ変換
std::string StringTostdstring(String^ StrData)
{
	return msclr::interop::marshal_as<std::string>(StrData);
}


int main(array<System::String ^> ^args)
{
	System::String^ strTemp = "SELECT * FROM OOO WHERE XXXXXXXXXX;";  // 長さは250文字程度

	char szLog1[1024];
	char szLog2[1024];

	memset(szLog1,0x00,sizeof(szLog1));
	memset(szLog2,0x00,sizeof(szLog2));

	//method 1
	memcpy(szLog1, StringTostdstring(strTemp).c_str(), 1024);　　// 実際のコードは、ここで実行中エラーとなりクラッシュする
	
	//method 2
	char* str2 = (char*)Marshal::StringToHGlobalAnsi(strTemp).ToPointer();
	memcpy(szLog2, str2, 1024);
	Marshal::FreeHGlobal((IntPtr)str2);

	if (memcmp(szLog1, szLog2, 1024) == 0)
	{
		int nD1 = 0;
	}
	else
	{
		int nD2 = 0;   //szLog1, szLog2は一致しない
	}
	
    return 0;
}

Answer 1

//method 1

memcpy(szLog1, StringTostdstring(strTemp).c_str(), 1024);　　// 実際のコードは、ここで実行中エラーとなりクラッシュする

StringTostdstringの結果(std::string)が1024byteあると決めつけちゃダメちゃうかな。

std::string result = StringTostdstring(strTemp);
memcpy(szLog1, result.c_str(), result.length());

ではいかがっスか?

Comments

[Taffy]

回答ありがとうございます。
std::stringは、文字数分しかメモリを確保しないということですよね。
第一引数szLog1のサイズを超えていないことを確認した上で、std::string のサイズ分だけコピーする処理に修正したところ、クラッシュしなくなりました。

[episteme]

「少なくとも」文字数分の領域を確保する、でしょうね。
なんにせよ1024byte以上って保証はない。

あと、終端ナル文字がコピー対象でないなら、c_str()改めdata()でいい。