大家帮我看看有什么问题吧

[skywatcher]

就这么一点代码，不知怎么回事，只要用上，程序就喜欢自动退出，或者因为出现叫做什么 内存访问无效（英文，我自己瞎翻的）什么的错误而退出！
我实在没有看出错在哪儿，谁给点意见？


int ClassName::Send(LPSTR QQNumber, LPSTR msgin,LPSTR msgout)
{
CString m_download,cmsgin(msgin);
char * m_location;
CInternetSession mySession;
CInternetFile * myFile=NULL;
m_location="http://localhost/?";
strcat(m_location,msgin);
try
{
myFile=(CInternetFile*)mySession.OpenURL(m_location,1,INTERNET_FLAG_RELOAD,0);
}
catch(CInternetException * pEx)
{
myFile=NULL;
pEx->Delete();
return 0;
}
if(myFile)
{
CString line;
for(int i=0;i<10 && myFile->ReadString(line);i++)
m_download+=line+"\r\n";
myFile->Close();
delete myFile;
}
if(m_download=="")
{
m_download+="Error!";
}
wsprintf(msgout,"%s",m_download);
msgin="";
return 1;
}

[abitz]

m_location="http://localhost/?";
strcat(m_location,msgin);
这个不行。m_location="http://localhost/?"使m_location指向一块只读空间，
不能再进行连接。
应该先分配空间，再strcpy。

[skywatcher]

应该怎样分配才算好呢？有一段源代码吗？
还有，我总觉得不应该是设个问题一样！
因为如果是这个问题，就应该整个不能通过编译阿

[cgsw12345]

应该是 abitz(阿奈) 兄弟所说的问题 试试使用new 和delete
分配内存看能不能通过！

[zhjie007]

把m_location定义成CString的，试试

[abitz]

>> 应该怎样分配才算好呢？有一段源代码吗？
>> 还有，我总觉得不应该是设个问题一样！
>> 因为如果是这个问题，就应该整个不能通过编译阿

这个是runtime error，编译不会有问题。
给m_location分配空间 或 声明为c-style string 或 string都能解决这个问题。

[rorot]

int ClassName::Send(LPSTR QQNumber, LPSTR msgin,LPSTR msgout)
{
// 这里用字符数组(栈)
char m_location[1024];

CString m_download,cmsgin(msgin);
CInternetSession mySession;
CInternetFile * myFile=NULL;

// 字符串复制
strcpy( m_location, "http://localhost/?" );

strcat(m_location,msgin);
try
{
myFile=(CInternetFile*)mySession.OpenURL(m_location,1,INTERNET_FLAG_RELOAD,0);
}
catch(CInternetException * pEx)
{
myFile=NULL;
pEx->Delete();
return 0;
}
if(myFile)
{
CString line;
for(int i=0;i<10 && myFile->ReadString(line);i++)
m_download+=line+"\r\n";
myFile->Close();
delete myFile;
}
if(m_download=="")
{
m_download+="Error!";
}
wsprintf(msgout,"%s",m_download);
msgin="";
return 1;
}