Merged revisions 217806 via svnmerge from
authorDavid Vossel <dvossel@digium.com>
Thu, 10 Sep 2009 21:07:47 +0000 (21:07 +0000)
committerDavid Vossel <dvossel@digium.com>
Thu, 10 Sep 2009 21:07:47 +0000 (21:07 +0000)
commite716801ab247bca8b2db04aedefc6009a341199d
tree99eaf28573ebc526f7463bf4281a7c47a7cc4ff1
parentf558f01a819c98875b272ed54672f94e2d36e088
Merged revisions 217806 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r217806 | dvossel | 2009-09-10 16:06:07 -0500 (Thu, 10 Sep 2009) | 22 lines

  IAX2 encryption regression

  The IAX2 Call Token security patch inadvertently broke the use of
  encryption due to the reorganization of code in the socket_process()
  function.  When encryption is used, an incoming full frame must first
  be decrypted before the information elements can be parsed.  The
  security release mistakenly moved IE parsing before decryption in
  order to process the new Call Token IE.  To resolve this, decryption
  of full frames is once again done before looking into the frame.  This
  involves searching for an existing callno, checking the pvt to see if
  encryption is turned on, and decrypting the packet before the internal
  fields of the full frame are accessed.

  (closes issue #15834)
  Reported by: karesmakro
  Patches:
        iax2_encryption_fix_1.4.diff uploaded by dvossel (license 671)
  Tested by: dvossel, karesmakro

  Review: https://reviewboard.asterisk.org/r/355/
........

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