内存报错，ECX为0

[fandh]

原帖如下：
CheckData()中出错，
很奇怪的错，出错时调试
_info值证常，并且里面的指针值都正常，但从汇编调试看出，
出错在比较语句：*((uint32*)_info.DataPtr) == 0x01000000

先是MOV PTR this到EAX，再MOV [EAX+8]到ECX（ECX应该放的是_info.DataPtr的地址），用ECX的值与0x01000000比较，但此时ECX的值是0


C/C++ code
class MediaData
{
public:
/*
* 帧信息
*/
struct MediaDataInfo
{
MediaDataType MediaDataType;
PicSize PicSize;
uint8* DataPtr;
uint32 DataLength;
uint32 Sequence;
uint32 Frequence;
Timestamp Timestamp;
};

MediaDataInfo _info;
...
};

void MediaData::CheckData()
{
if(_info.DataLength < 8)
return;
if(_info.MediaDataType == VIDEO_H264)
{
if(*((uint32*)_info.DataPtr) == 0x01000000) //在这里出错
{
....
}
else if(*((uint32*)_info.DataPtr) == 0x01000000)
{
....
}
}
}


链接如下：http://topic.csdn.net/u/20090602/09/0a75cccc-9f70-427c-902a-3bd7508dadf0.html?seed=625246241
替peimoxu 发帖！
补充一下：
这个错误在控件中！

解决后，就给分！

[闪破风浪]

打日志，定位错误？
可以用BoundsChecker检测一下。。。

[greatws]

uint8* DataPtr;
如果这个指针为0，那么就是ecx为0。

看看代码其他地方有什么问题，导致DataPtr为NULL，这点代码看不出问题。

[cnzdgs]

DataPtr中储存的指针为空，看一下DataPtr的值是怎么来的。

[fandh]

还有其它建议么？

[会思考的草]

_info.DataPtr是不是在堆上分配的？
怎么感觉像是内存越界了导致有效内容被脏数据覆盖了。
建议用pageheap工具查一下。
简单用法：
gflags -p /enable <yourapp>.exe /full
详细用法：
http://support.microsoft.com/kb/286470

当然也可以等到_info.DataPtr指向内存被分配了之后，用windbg的ba命令监视_info.DataPtr指向的首地址。

[Kerrie]

DataPtr不为NULL，其内存内容为：00 00 00 01 21

DataPtr为一个指针，他的值不应该为00 00 00 01 21这样的吧，更像其内容或者无效值。


请检查一下this指针是否有效？以及DataPtr的分配代码是否有误

[fandh]

还有么？

[wocow3]

300分。。出手可真大方

[老邓]

应该是强制转换的问题。注意高、低位转换。

[oyljerry]

错误在控件中? 可以增加一些 _info.DataPtr 的日志等信息，出问题的时候查看

[peimoxu]

代码很少：

void MediaData::CheckData()
{
    if(_info.DataLength < 8)
        return;
    if(_info.MediaDataType == VIDEO_H264 || 
        _info.MediaDataType == VIDEO_H264_HID1 || 
        _info.MediaDataType == VIDEO_H264_IVS)
    {
        if(*((uint32*)_info.DataPtr) == 0x01000000 && //出错点。出错时，变量监视显示DataPtr不为NULL，其内存内容为：00 00 00 01 21 ....是预期值
                *(_info.DataPtr+4) == 0x27)
        {
            //I帧
        }
        else if(*((uint32*)_info.DataPtr) == 0x01000000 && 
                *(_info.DataPtr+4) == 0x21)
        {
            //P帧
        }
    }
}
; *((uint32*)_info.DataPtr) == 0x01000000
mov     eax, DWORD PTR _this$[ebp]
mov     ecx, DWORD PTR [eax+8]
cmp     DWORD PTR [ecx], 0x01000000 //ecx为0

[hendriclee]

ECX为0？ECX是this指针……

[bragi523]

*((uint32*)_info.DataPtr
这样对一个指针操作应该是不对的
如果想要这样操作的话得new出来

[fandh]

还有什么好的建议？

[autumn1225]

建议使用内存断点进行分析。

一般，内存断点在OD上，至少为4K，而且一次调试只可以设置一次。

个人感觉，只是内存被其它线程更改。

所以，检查多线程，必要时可考虑阻塞其它线程。

考虑到，错误不易重现，可对内存操作线程重点分析，并打印操作日志，打印时，保存时间信息和对应地址信息。

[oyljerry]

多线程操作同一个变量等，多打印一些信息等，看是否出现同时访问，注意线程同步操作等，加锁lock

[jian_tian_yang]

帮楼主顶一下

[fandh]

已经解决了，多线程的问题！
peimoxu 自己分析的方向很准的！

[UDX协议]

MediaData,我觉得这个对象本身已经被release了。

所以，他里面的一些转换或调用都是失败的，不可预知的。