Question

向c++高手求教，应该是关于指针的问题，万分感谢！！

我做的是directshow自己添加Filter实现视频回放，程序调试时总出现如下错误：

代码是在太长只能粘贴调试时提示有问题的部分了，调试时黄色箭头指向这一句：HRESULT hr = pFilter->EnumPins(&pEnum);

下面是这句代码附近的：
HRESULT GetUnconnectedPin(
IBaseFilter *pFilter, // Pointer to the filter.
PIN_DIRECTION PinDir, // Direction of the pin to find.
IPin **ppPin=NULL
)
//pFilter=(IBaseFilter *)new(sizeof(IBaseFilter)) // Receives a pointer to the pin.
{
IEnumPins *pEnum = 0;
IPin *pPin = 0;
HRESULT hr = pFilter->EnumPins(&pEnum);
if (FAILED(hr))
{
return hr;
}
while (pEnum->Next(1, &pPin, NULL) == S_OK)
{
PIN_DIRECTION ThisPinDir;
pPin->QueryDirection(&ThisPinDir);
if (ThisPinDir == PinDir)
{
IPin *pTmp = 0;
hr = pPin->ConnectedTo(&pTmp);
if (SUCCEEDED(hr)) // Already connected, not the pin we want.
{
pTmp->Release();
}
else // Unconnected, this is the pin we want.
{
pEnum->Release();
*ppPin = pPin;
return S_OK;
}
}
pPin->Release();
}
pEnum->Release();
// Did not find a matching pin.
return E_FAIL;
}
实在不知道怎么把代码调通了，菜鸟求教啊。。等待大神的回复。。

Answer 1

pFilter是空指针吧

追问

是不是应该初始化？但是初始化之后就编译不过了。。真心不知道怎么回事= =

追答

你初始化的语句写错地方了，写在了大括号外面。。。
而且也不应该给pFilter赋值，因为它是外面传进来的，你赋值后就覆盖了原来的值，就没有意义了。你调试的时候从调用堆栈里往前找，看是为什么pFilter是空指针。

追问

那应该怎么写呢？pFilter=NULL; 是这样吗？我写在了大括号里也不对。。。

追答

我的意思是你根本就不应该初始化，因为pFilter的值是NULL才导致的问题，你又pFilter=NULL岂不是肯定会出问题！
你去调试，查查调用GetUnconnectedPin的地方，传的空指针是怎么回事！

Answer 2

我表示万分同情，还好我不用使用DirectShow编程，直接调开源的播放器了，我非常地同情+抱歉。。