[reSIProcate] mExpires is not set

[Scott Godin]

I committed a fix for this.  Thanks.

 

________________________________

From: resiprocate-devel-bounces at list.sipfoundry.org
[mailto:resiprocate-devel-bounces at list.sipfoundry.org] On Behalf Of
Noiile
Sent: Wednesday, September 21, 2005 1:04 AM
To: resiprocate-devel
Subject: [reSIProcate] mExpires is not set

 

resiprocate-devel, Hello 

 

When UAS receviced the first PUBLISH, it will create a new
ServerPublication and call onInitial() handler.

But I found that mExpires which pass to function onInitial() was not set
value, and it was not be initialized too.

Maybe it is a bug.

 

In function :

ServerPublication::dispatch(const SipMessage& msg)

{

   assert(msg.isRequest());
   ServerPublicationHandler* handler =
mDum.getServerPublicationHandler(mEventType);
   mLastRequest = msg;

   /***********************************************

   //I think Some codes should be added as following

   mExpires = 3600; 
   if (msg.exists(h_Expires))
   {
      mExpires = msg.header(h_Expires).value();
   }

    ************************************************/
   

   if (msg.exists(h_SIPIfMatch))
   else
   {
      mLastBody = Helper::extractFromPkcs7(msg, *mDum.getSecurity());
      handler->onInitial(getHandle(), mEtag, msg, 
                         mLastBody.mContents.get(), 
                         mLastBody.mAttributes.get(), 
                         mExpires); //***************** mExpires was not
be initialized
   }
}

 

 

Noiile, noiile at hotmail.com

2005-9-21 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://list.resiprocate.org/pipermail/resiprocate-devel/attachments/20050921/21712c91/attachment.htm>