bad mfc code

char *nText;
CString oldText, newText;

sprintf(nText, "Invalid address at line %d for %s.\r\n", j, address1);
CString newText(nText);

if (oldText) oldText += newText;
else oldText = newText;

pDialog->SetDlgItemText(IDC_ERROR, oldText);