余計なstrcat()を消したらよいのでは、としか言えないな。この情報じゃ。

すいません，情報不足だったみたいで・・・．

ちなみに下記の方法でこの問題を解消することが出来ました．
int nN = strlen(pNAME2);
pNAME2[nN-1] = '\0';
int nD = strlen(pDATA2);
pDATA2[nD-1] = '\0';

ついでに質問なのですが，
char pDATA;
char buf;
sprintf_s(pDATA,'%s',buf);

としたところ，bufの内容しかpDATAに入らず
''を入れることが出来ません．
調べたところ'' ''と二つ重ねたら出来ると聞いたのですが
それを実行してみても出来ませんでした．
他に何か良い方法等あれば教えていただきたいです．

うーん。この方向での解決策を模索するより先に
SQLインジェクション脆弱性について調べるべし。
するとぜんぜん違う方策が提案されているのがわかるだろう。

このまま作ってもただのセキュリティホールだだ漏れプログラムになりかねない
全部作り直し、って可能性がちらほら見え隠れしているような気のせいがする

// ん～　反映されるけど・・
#include <windows.h>
#include <iostream>
using namespace std ;

int main ()
{

char pDATA[128] ;
char buf[] = hoge ;
sprintf_s(pDATA,'%s',buf) ;

cout << pDATA << endl ; // 'hoge'
printf( pDATA ) ; // 'hoge'
MessageBoxA ( NULL, pDATA, 0, 0 ) ; // 'hoge'

cin.ignore() ;
return 0 ;
}

> char pDATA;
> char buf;
> sprintf_s(pDATA,'%s',buf);

文字と文字列の区別が付いてないです。
sprintfの例。

{
char pData[1024];
char buf[] = hoge;
sprintf(pData, '%s', buf);

cout << : << pData << : << endl;
}