Re: [reSIProcate] Maybe a bug ? SipMessage::remove() is crashed in DumFeature::process() function

[Karlsson <boost.regex@xxxxxxxxx>]

I have DEBUG it, the  SipMessage* sipMsg  = dynamic_cast<SipMessage*>(msg);  always return 0, it's never succeed.

 

 

And I have wrote these code:

 

SipMessage* sipMsg = (SipMessage*)msg;

Data Text = Data::From(*sipMsg);

MessageBox(NULL, Text.c_str(), "message", MB_ICONINFORMATION);

 

The message box show it's SIP message correctly, so I don't know why the dynamic_cast return 0.

 

 

 

I have read your code, I saw you have delete the sipMsg->remove(*headerBase);  why ?

 

Thanks

 

On Sat, Oct 4, 2008 at 7:42 PM, Justin Matthews <jmatthewsr@xxxxxxxxx> wrote:

You need to dynamic cast the Message pointer and if it's 0 then it's not a SipMessage and you should ignore it.

 

Also, I think you still need to make a copy of the HeaderFieldValue to allocate a buffer that is owned by the underlying HeaderFieldValue.

 

Untested code:

 

 

DumFeature::ProcessingResult MyFeature::process(Message* msg)
{

SipMessage* sipMsg  = dynamic_cast<SipMessage*>(msg);

 if (sipMsg)

 {

  const resip::Data headerName("User-Agent");

  resip::Data headerValue("test");

 

  Headers::Type headerType = Headers::getType(headerName.data(), headerName.size());

 

  if (headerType != Headers::UNKNOWN)

  {

   HeaderFieldValueList * hfvs = new HeaderFieldValueList();

   hfvs->push_back(new HeaderFieldValue(HeaderFieldValue(headerValue.data(), headerValue.size())));

   sipMsg->setRawHeader(hfvs, headerType);

  }

 }

 postCommand(auto_ptr<Message>(sipMsg));

 return DumFeature::FeatureDone;

}

 

From: resiprocate-devel-bounces@xxxxxxxxxxxxxxx [mailto:resiprocate-devel-bounces@xxxxxxxxxxxxxxx] On Behalf Of Karlsson
Sent: Friday, October 03, 2008 11:42 PM
To: resiprocate-devel
Subject: [reSIProcate] Maybe a bug ? SipMessage::remove() is crashed in DumFeature::process() function

 

Hi all, I have write a class which derived from DumFeature, and implemented the process function:

 

 

DumFeature::ProcessingResult MyFeature::process(Message* msg)
{

 SipMessage* sipMsg = (SipMessage*)msg;
 if (sipMsg)
 {
  const resip::Data headerName("User-Agent");
  resip::Data headerValue("test);

  resip::HeaderBase * headerBase = HeaderBase::getInstance(resip::Headers::getType(headerName.c_str(),headerName.size()));

  if (headerBase)
  {
   if (sipMsg->exists(*headerBase) == true)
   {
    sipMsg->remove(*headerBase);
   }

   HeaderFieldValueList * hfvs = new HeaderFieldValueList();
   hfvs->push_back(new HeaderFieldValue(headerValue.data(), headerValue.size()));

   sipMsg->setRawHeader(hfvs, Headers::getType(headerName.data(), headerName.size()));

  }
 }

 postCommand(auto_ptr<Message>(sipMsg));

 return DumFeature::FeatureDone;
}

 

My application is crashed on sipMsg->remove(*headerBase); I have debug this function:

 

void
SipMessage::remove(const HeaderBase& headerType)
{
   delete mHeaders[headerType.getTypeNum()];   ------------------------->when runing to this line, it will be crashed.
   mHeaders[headerType.getTypeNum()] = 0;
};

 

 

Above code I have using with MessageDecorator::decorateMessage function, it's working fine.

 

And I try to use SipMessage* sipMsg = dynamic_cast<SipMessage*>(msg); to instead SipMessage* sipMsg = (SipMessage*)msg;

but it's failed, the dynamic_cast result it 0.

 

Thanks