下面程序是将两个输入的字符串相连后输出，用vc6编译正确后运行，输入两个字符串之后按回车出现错误，提示内存不可读，请问是什么原因？
#include <stdio.h>
void xzstrcat(char *s1,char *s2)
{
    char *p,*q,ch;
    p=s1,p=s2;
    while((ch=*p)!='\0')
        p++;
    while((ch=*q)!='\0')
    {    *p=*q;
       p++;
       q++;
       }
    *p='\0';
}
void main()
{
    char s1[20],s2[20];
    printf("please enter the first string:\n");
    scanf("%s",s1);
    printf("please enter the second string:\n");
    scanf("%s",s2);
    xzstrcat(s1,s2);
    printf("\n%s",s1);
}

p=s1,p=s2; //应为p=s1,q=s2，楼主手误啦！·

见笑了。。。

p = (char *)malloc(sizeof(s1+s2));
bzero(p,sizeof(*p));
p=s1,q=s2;
让后再：
    while((ch=*q)!='\0')
    {    *p=*q;
       p++;
       q++;
        ...
否则P指向未知区域，成为野指针，就会发生内存泄漏问题的；

p = (char *)malloc(sizeof(s1+s2));
bzero(p,sizeof(*p));
这两句能解释一下吗

为p 指针申请 s1+s2 大小的内存空间， bzero(p,sizeof(*p))是指将P所指的内存空间清空。
一般的话，内存是连续的，不会出现内存泄漏问题，当产生分页的时候就会显然易见了！

所以lz以后还是谨慎为好！