究竟错在哪里??

Question

最近写代码，遇到一个简单却又大花时间的问题!!郁闷郁闷......
现把代码描述如下（是在VC++ 6.0环境下调试的）节省了不重要代码/*宏定义与结构体*/
#define STRLEN 32
typedef struct
{
char cArr[STRLEN];
BYTE *bpData;
}STRSEG;/* 下面是在某函数中的代码   */ FILE *pMasterFile;
//假设此文件在本地磁盘上存在，且能进行读写操作
//各位自己可以指定一定大小的文件（不要太小哦不能小于512 * 64 + STRLEN）
char cFileName[]="c:\\1.srb";
STRSEG *pStrSeg;
DWORD dwCount;
DWORD dwOffset = 0;
pStrSeg =(STRSEG *) malloc(STRLEN);
memset(pStrSeg->cArr, 0, STRLEN);
pStrSeg->bpData =(BYTE *) malloc(512 * 64);
dwCount = 512 * 64 + STRLEN;
if ((pMasterFile = fopen(cFileName, "r+b")) == NULL)
{
return FALSE;
}
if (fseek(pMasterFile, dwOffset, SEEK_SET) != 0)
{
return FALSE;
}
if (fread(pStrSeg, sizeof(BYTE), dwCount, pMasterFile) != dwCount)
{
return FALSE;
}
每次调试fread都通不过...
请各位有意者，察看察看代码，检查检查代码，
找到错误有奖 ，哈哈，开个玩笑!以次告诫跟我一样的初学者，要练好基本功，
努力使自己能了解每一行代码的机理（不过就
现在的所谓的"高级环境","高级语言"中我们只
能尽力而为了）+++++++++++++++++++++++++++++++++++++++
初学C++,初写C++,初人者；
初上网，初文章，初网也；
在网上还没找到家，
有事联系:[email protected]一个编程者的初衷：
一只小蚂蚁在崎岖的山路上爬，
时不时踹几脚路边的老树，
——移动的终会比静止的强!
+++++++++++++++++++++++++++++++++++++++

Answer 1

cArr和bpData的空间不是连在一起的，你这样做会覆盖了bpData的指针和后面的区域，应该会死的很难看……不信，可以看看sizeof(STRSEG)是多少你就不能分开写么？fread一次cArr，再一次bpData

Answer 2

pStrSeg =(STRSEG *) malloc(STRLEN);
这里没有给bpData成员留出空间，当然会出错。
pStrSeg =(STRSEG *) malloc(sizeof(STRSEG));