]> git.ipfire.org Git - thirdparty/asterisk.git/commit
Merged revisions 378593 via svnmerge from
authorAutomerge script <automerge@asterisk.org>
Fri, 4 Jan 2013 23:19:39 +0000 (23:19 +0000)
committerAutomerge script <automerge@asterisk.org>
Fri, 4 Jan 2013 23:19:39 +0000 (23:19 +0000)
commit2765de67b6f66ff36c4f7299caecf21addd251c0
treec6eb40f8332f684dda3e1633cb41b8a9ee64df6e
parent77150eecd97cddd28744f9b8d0fe3d69ad774803
Merged revisions 378593 via svnmerge from
file:///srv/subversion/repos/asterisk/trunk

................
  r378593 | jrose | 2013-01-04 17:14:54 -0600 (Fri, 04 Jan 2013) | 23 lines

  res_srtp: Prevent a crash from occurring due to srtp_create failures in srtp_create

  Under some circumstances, libsrtp's srtp_create function deallocates memory that
  it wasn't initially responsible for allocating. Because we weren't initially
  aware of this behavior, this memory was still used in spite of being unallocated
  during the course of the srtp_unprotect function. A while back I made a patch
  which would set this value to NULL, but that exposed a possible condition where
  we would then try to check a member of the struct which would cause a segfault.
  In order to address these problems, ast_srtp_unprotect will now set an error value
  when it ends without a valid SRTP session which will result in the caller of
  srtp_unprotect observing this error and hanging up the relevant channel instead of
  trying to keep using the invalid session address.

  (closes issue ASTERISK-20499)
  Reported by: Tootai
  Review: https://reviewboard.asterisk.org/r/2228/diff/#index_header
  ........

  Merged revisions 378591 from http://svn.asterisk.org/svn/asterisk/branches/1.8
  ........

  Merged revisions 378592 from http://svn.asterisk.org/svn/asterisk/branches/11
................

git-svn-id: https://origsvn.digium.com/svn/asterisk/team/mmichelson/threadpool@378600 65c4cc65-6c06-0410-ace0-fbb531ad65f3
res/res_srtp.c