Merged revisions 204243,204246 via svnmerge from
authorMark Michelson <mmichelson@digium.com>
Mon, 29 Jun 2009 21:48:54 +0000 (21:48 +0000)
committerMark Michelson <mmichelson@digium.com>
Mon, 29 Jun 2009 21:48:54 +0000 (21:48 +0000)
commit200f1dc19eba8f44164e9f9846d03dbf834d7736
tree3d3c850cdfa84a1e67dd892f3c65e5229d2fb0db
parente840307ad185cd3dfb898836fd9f64267b63dd65
Merged revisions 204243,204246 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r204243 | mmichelson | 2009-06-29 16:23:43 -0500 (Mon, 29 Jun 2009) | 22 lines

  Fix a problem where chan_sip would ignore "old" but valid responses.

  chan_sip has had a problem for quite a long time that would manifest when
  Asterisk would send multiple SIP responses on the same dialog before receiving
  a response. The problem occurred because chan_sip only kept track of the highest
  outgoing sequence number used on the dialog. If Asterisk sent two requests out,
  and a response arrived for the first request sent, then Asterisk would ignore
  the response. The result was that Asterisk would continue retransmitting the
  requests and ignoring the responses until the maximum number of retransmissions
  had been reached.

  The fix here is to rearrange the code a bit so that instead of simply comparing
  the sequence number of the response to our latest outgoing sequence number, we
  walk our list of outstanding packets and determine if there is a match. If there is,
  we continue. If not, then we ignore the response.

  In doing this, I found a few completely useless variables that I have now removed.

  (closes issue #11231)
  Reported by: flefoll

  Review: https://reviewboard.asterisk.org/r/298
........
  r204246 | mmichelson | 2009-06-29 16:37:05 -0500 (Mon, 29 Jun 2009) | 3 lines

  Fix build oops.
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@204247 65c4cc65-6c06-0410-ace0-fbb531ad65f3
channels/chan_sip.c