Если не используется unicode LPTSTR определен как
typedef char* LPTSTR

У CString есть оператор приведения типа только к LPCTSTR, который в свою очередь определен как
typedef const char* LPCTSTR

Вариантов получения из CString char* два:

1.
CString str;
.....
char* pStr = str.GetBuffer();


2.
char pStr[128];
CString str;
....
strcpy(pStr, str);

EcherА вот еще нашел:
CString str;
...
item.pszText = (LPTSTR)(LPCTSTR)str;

Поступать так - значит искать больших неприятностей. Константный буфер на то и константный, чтобы не меняться. Такое преобразование допускается компилятором и даже иногда работает, но гораздо чаще приводит к большим проблемам.

msdn по поводу оператора LPCTSTRThis useful casting operator provides an efficient method to access the null-terminated C string contained in a CString object. No characters are copied; only a pointer is returned. Be careful with this operator. If you change a CString object after you have obtained the character pointer, you may cause a reallocation of memory that invalidates the pointer.

В данном конкретном случае действительно неприятностей ждать не стоит. Поле pszText в структуре TVITEM объявлено char* не потому, что сам контрол может поменять его содержимое (он-то видимо сразу после InsertItem копирует содержимое во внутренний буфер), а чтобы сам пользователь в случае необходимости мог менять текст. Однако название поста предполагало более широкую трактовку вопроса - сюда могут заглянуть не только те, кому надо вставить итем в TreeCtrl.

CEMb strcpy(szTmp,(char*)(LPCTSTR)strTmp);
А не проще ли
strcpy(szTmp,strTmp); ???

Хотя по смыслу согласен - в случае использования сразу ничего страшного в таком преобразовании нет.