なぜMSVC++は "std :: strcat"を "安全でない"とみなしていますか？ （C++）

Question

が、私はこのようなものをやろう：

char* prefix = "Sector_Data\\sector"; 
char* s_num = "0"; 
std::strcat(prefix, s_num); 
std::strcat(prefix, "\\"); 

などなど、私は警告を取得し

warning C4996: 'strcat': This function or variable may be unsafe. Consider using strcat_s instead. 

なぜstrcatのが危険とみなされ、そして方法がありますstrcat_sを使わずにこの警告を取り除くには？唯一の方法は警告を取り除くために場合

はまた、それが動作しないか、strcat_sを使用することです（構文ワイズ：どうやら、それは2つの引数を取りません）。

Answer 1

バッファのプレフィックスは、バッファオーバーランの原因となるよりも少ないスペースでコピーできます。 したがって、ハッカーは特別に細工された文字列を渡すことができ、リターンアドレスやその他の重要なメモリを上書きし、プログラムのコンテキストでコードの実行を開始します。

strcat_sあなたは、文字列をコピーする先のバッファの長さに渡すことを強制することでこれを解決。バッファがオーバーランしていないことを確認するために、必要に応じて文字列を切り捨てます。

google strcat_sを使用する方法を正確に確認してください。

Answer 2

それはあなたのケースでは、宛先の文字列（接頭辞）がその境界を越えて書き込まれるかどうかをチェックする手段がないので。 strcatは本質的に、ソース文字列を宛先にバイト単位でコピーするループ処理を行います。ヌル端末と呼ばれる値「0」（ '\ 0'で表記）を見ると停止します。 Cには境界チェックが組み込まれておらず、dest strはメモリ内の単なる場所であるため、strcatは元のstrまたはdestを吹き飛ばしてもad-infinidiumを続けます。 strにはヌル端末がありません。

上記の解決策は、ご使用のWindows環境に応じて異なります。あなたは、プラットフォームに依存しない何かをしたい場合は、strncatと口論する必要があります。

strncat(char* dest, const char* src, size_t count) 

これは、インテリジェントに使用される別のオプションです。 countを使用して、コピーする最大文字数を指定することができます。これを行うには、destにどれくらいのスペースがあるかを調べなければなりません（strlen（dest）をどのくらい割り当てたか）、それをcountとして渡してください。

Answer 3

C/C++の文字列操作関数の1つで、バッファオーバーランエラーが発生する可能性があります。

問題は、関数がバッファのサイズがわからないことです。 MSDNのドキュメントから：

最初の引数、strDestination、 は 現在strDestinationとstrSource組み合わせ とクロージング「\ 0」を保持するのに十分な大きさでなければなりません。 、バッファオーバーランが発生する可能性があります。

strcat_sには、バッファのサイズを示す追加の引数があります。これにより、連結を行う前にサイズを検証することができ、オーバーランを防ぐことができます。あなたは、C++を使用している場合は、なぜ全体の混乱を避け、std::stringを使用しないでhttp://msdn.microsoft.com/en-us/library/d45bbxx4.aspx

Answer 4

を参照してください。エラーのない同じ例は、次のようになります。

std::string prefix = "Sector_Data\\sector"; 
prefix += "0"; 
prefix += "\\" 

バッファサイズとそのすべてのことを心配する必要はありません。 const char *のAPIをお持ちの場合は、.c_str()のメンバーだけを使用できます。追加することにより、あなたはこれらの警告を取り除くことができます

some_c_api(prefix.c_str()); 

Answer 5

：

_CRT_SECURE_NO_WARNINGS 

し、プロジェクトのプリプロセッサ定義に

_SCL_SECURE_NO_WARNINGS 

。

Answer 6

strcatには2つの問題があります。まず、あなたはほとんどの機能と同じである仕事をして、関数の外ですべてのあなたの検証を行う必要があります：

あなたはちょうどそれが収まることを確認するために、両方の文字列の長さ全体を歩いする必要が

if(pDest+strlen(pDest)+strlen(pScr) < destSize) 

、コピーをするために全長をもう一度歩く前に。このため、多くのプログラマーは、テストに合ってスキップすると単純に仮定します。さらに悪いことに、コードが最初に書かれたときに適合するように保証されているかもしれませんが、誰かが別のstrcatを追加したり、プログラムのどこか他の場所でバッファサイズや定数を変更すると、問題が発生することがあります。

もう1つの問題は、pSrcとpDstが重複している場合です。あなたのコンパイラに依存して、strcatはpSrcの0の文字を一度にチェックする単純なループかもしれません。 pDstが0を上書きすると、プログラムがクラッシュするまで実行されるループに入ります。

Answer 7

警告をオフにするには、これを行うことができます。

#pragma warning(disable:4996) 

btw strcat_s（）を使用することを強くお勧めします。