Commit Graph
4824 Commits
Author SHA1 Message Date
Travis Cross 065043ce67 Remove unused variable in mod_rtmp
Unused variables break the build with recent versions of gcc and
clang.
2013-05-26 04:53:16 +00:00
William King 12f2f674f9 Let's bit a bit safer when dereferencing pointers. 2013-05-24 20:24:04 -07:00
William King 480407e5f3 Removing logically dead code 2013-05-24 19:59:32 -07:00
William King dc4209275d Another location where mod_rtmp would leak memory. 2013-05-24 19:45:38 -07:00
William King 194e1ae366 Fix improper handling of a double linked list that could have caused a memory leak. 2013-05-24 19:42:17 -07:00
William King e2d6bc3340 Appears to have been a copy paste error because this doesn't do what the api claims, and it also leaks the reference to the new node. 2013-05-24 19:35:56 -07:00
William King f3fd94b3e5 Fixing a memory leak. name needs to be freed since it's being strdup'd lower. 2013-05-24 19:31:26 -07:00
Anthony Minessale 7f3c160793 FS-5453 --resolve 2013-05-24 08:13:41 -05:00
Anthony Minessale e02992bc6c FS-5453 --resolve this is a different issue, we just have to hangup the loopback channel so the loopback and the sip don't both execute the extension 2013-05-23 09:59:00 -05:00
Anthony Minessale 1b1bdf6816 FS-5453 --resolve 2013-05-23 08:00:03 -05:00
Nathan Neulinger 514fc941c0 FS-5431: --resolve fix socket creation loop for mod_skinny to avoid segfault 2013-05-23 08:13:44 -05:00
William King 15f43bac09 FS-5440: don't assert on bad input rtmp data 2013-05-22 12:38:23 -07:00
Nathan Neulinger 23bd06881d FS-5429: --resolve add minimal idivert support for skinny 2013-05-22 09:58:28 -05:00
Nathan Neulinger 8afaf28a09 FS-5379: --resolve fix skinny phone codec agreement issues due to incorrect hardwired ptime, update field names for future coding to be more descriptive 2013-05-22 09:27:09 -05:00
Nathan Neulinger 337976e469 FS-5225: --resolve add ability to dial by pressing line button 2013-05-22 09:17:41 -05:00
Nathan Neulinger 0b05cf6843 FS-5267: --resolve avoid null deref if sessions missing 2013-05-22 09:12:46 -05:00
Anthony Minessale 363b58dc54 FS-5453 --resolve 2013-05-21 12:01:57 -05:00
Travis Cross d2f3a3114e Spelling fix 2013-05-21 02:00:43 +00:00
William King 9ace496fe3 full_contact must be freed by sofia. There isn't a chance of this having blown anything up, but Coverity is right that this should be initialized with NULL to be safe against future refactors. 2013-05-18 19:13:18 -07:00
William King d27a95a50e Possible leak if someone were to try and add extra headers to a channel that had either an inbound info on that session or the session were to be deflected. I don't see a way that these extra headers could be added outside of a C module subscribing to the session states, but just to make sure I'm adding these so that just like other extra header locations it's cleaned up. 2013-05-18 17:00:41 -07:00
William King f718a0572f Alice meet rabit hole. There is a leak here on every invite that comes back from a status 180 when the other side is not FS, but you have presence enabled. Thanks to Coverity scan for location this condition. This 'might' not have been leaking if sofia was automatically freeing this when the sofia object was destroyed, but all other locations where sip_header_as_string where called they were all followed by a su_free, so I assume not. 2013-05-18 16:35:03 -07:00
William King f519ad195a If the mod_sofia global presense hash is null, then make sure to free the matches variable before going to done. 2013-05-18 16:05:40 -07:00
Anthony Minessale ea881b191f media/srtp refactoring pass 2013-05-17 15:39:30 -05:00
William King 8d7aa3ff65 FS-5441: --resolve 2013-05-17 12:39:17 -07:00
William King 58e205ccdb how...? 2013-05-17 11:36:31 -07:00
William King cfeb6aaf54 Since there isn't neutral ground, on the rtmp state callback we actually have to force the write lock of the session, but we only need a try lock on removing the session from the rsession->session_hash. This removes a deadlock caused when we had to enforce with locks that the rsession couldn't be destroyed if the FS session was in the middle of a hangup. 2013-05-17 11:36:30 -07:00
William King f24d1cde3d Seems the session wasn't coming to the hangup in a locked state. 2013-05-17 11:36:30 -07:00
William King 36f198e883 Seems having a read lock still prevents a write lock to be taken, even if the thread that is trying to get the write lock currently has the read lock. 2013-05-17 11:36:29 -07:00
William King 9b0212ab59 Include the write lock since we're changing the tech_pvt 2013-05-17 11:36:29 -07:00
Anthony Minessale 97faaab42c FS-5430 --resolve 2013-05-15 09:27:18 -05:00
Anthony Minessale 1eae2a1941 FS-5410 --resolve 2013-05-13 13:15:33 -05:00
Raymond Chandler bba4540fe5 add context 2013-05-10 16:31:12 -04:00
Anthony Minessale cb7a6bd34b FS-5085 the last change helped weed out the real bug which is this commit 2013-05-10 10:45:28 -05:00
Anthony Minessale bb6b02f2d6 FS-5403 --resolve 2013-05-10 09:59:02 -05:00
Anthony Minessale 16690e491d FS-5399 --resolve 2013-05-09 14:05:09 -05:00
Giovanni Maruzzelli 0330520eba test 2, please ignore 2013-05-09 17:00:54 +00:00
root 0fe65187cd test, please ignore 2013-05-09 16:59:39 +00:00
William King 9fa4bed8e3 FS-5395: False alarm on a reported valgrind memory leak. The io_private is alloc'd from a memory pool which valgrind sucks at tracking. No need to free this, because it'll get cleared when the memory pool on the rsession is destroyed. 2013-05-08 18:45:24 -07:00
William King b3117e8f88 FS-5395: --resolve Resolve another mod_rtmp crash where oh specific hangup scenarios there would be a free of variable that wasn't ever malloced 2013-05-08 16:54:05 -07:00
Anthony Minessale 0a68bee88e FS-5085 I discovered that the bowout is happening from both legs at once, we only need to check this condition on the A side of the loopback bridge 2013-05-08 13:17:07 -05:00
Brian West 7c0dc4c25d Undo previous change 2013-05-08 14:40:10 -05:00
William King 0d5888de2f Revert "fixed stuck mod_rtmp session when the call is killed from fs side, such as using uuid_kill"
This reverts commit d8edc031fa.

Conflicts:
	src/mod/endpoints/mod_rtmp/mod_rtmp.c
2013-05-07 16:04:54 -07:00
Anthony Minessale 01a748869d regression parsing rtp-timeout-sec from the profile 2013-05-07 08:50:02 -05:00
Anthony Minessale ff05e5f2c7 FS-5383 --resolve 2013-05-07 08:31:23 -05:00
Anthony Minessale 283e5f5f6a FS-5365 --resolve 2013-05-03 13:14:01 -05:00
Anthony Minessale adb68d6bca FS-5085 adding this patch but still wondering what causes this issue 2013-05-03 12:30:35 -05:00
Anthony Minessale 44840fc97a fix regression in sofia profile param for t38-passthru 2013-05-02 22:36:39 -05:00
William King 7ce7cbda57 FS-4332: --resolve 2013-05-03 00:01:24 -05:00
William King 17781cba1c FS-5240: --resolve Please test 2013-05-02 23:57:50 -05:00
Anthony Minessale 9ef0323499 FS-5355 --resolve 2013-04-29 10:09:14 -05:00