rtpengine

Commit Graph

Author	SHA1	Message	Date
Richard Fuchs	abf1ad73d2	MT#55283 combine in/out_lock With selected_sfd being protected by in_lock, we pretty much have to hold at least in_lock everywhere, and end up requiring both locks in many places. The distinction has become pointless. Change-Id: Ic0ad976c2d68d9639b9434da7f0e6e9c0d84c185 (cherry picked from commit `e03f814855`) (cherry picked from commit `222fcaa4b0`)	9 months ago
Richard Fuchs	2e714ebedd	MT#55283 locking updates Protect selected_sfd with in_lock. Protect RTCP sending with in_lock and out_lock as appropriate. Has the odd side effect of RTCP reports expected in tests to be sent one packet later than before. Closes #1966 Probably fixes #1927 Change-Id: I225b43dff8e8fbb938d3be6aad50249997615d77 (cherry picked from commit `ffc539c0d8`)	9 months ago
Richard Fuchs	d0c557a7dc	MT#55283 add T.38 fill-in This triggers an initial "no carrier" indication if the sender is slow to start PCM. Change-Id: I3513efafae06f992b3c9abd58b8ec797320c3b56 (cherry picked from commit `dafb68ef80`)	1 year ago
Richard Fuchs	adef9e9cc6	MT#55283 additional SpanDSP logging Change-Id: I41364fa1957b195772b8901456e93b6df2dee679 (cherry picked from commit `0a3a105d8b`)	1 year ago
Richard Fuchs	bca98ab764	MT#61404 clone previous sequence number for T.38 If the receiver of a previously passthrough T.30 stream gets switched to a generated PCM stream from a T.38 gateway, continue the sequencing of the previous SSRC. Technically this is not necessary as the generated PCM stream gets a new SSRC, but at least Asterisk seems to ignore this and expect sequencing to continue, and will ignore PCM if the sequence jump is too large. Change-Id: Ia4656770db11f5fa1a1e9bf5bd71a0398deb1e00 (cherry picked from commit `18579227a9`)	1 year ago
Richard Fuchs	ea3cfdbd38	MT#55283 add codec_def_supported() helper Make sure a codec is not only known to us, but that it can actually be used, in places where it makes sense. This is partially redundant because ensure_codec_def_type already takes care of this, but a codec definition may come from a different source, so it doesn't help to double check. Change-Id: I91af84afc2477840f1400674b2538ad8fb7746ee (cherry picked from commit `7b60e85970`)	1 year ago
Richard Fuchs	24ce8de2bb	MT#59038 shortcut media_player_new() Change-Id: I25fa42eb98684ea087b3e48d4fcdb6c94baeda27	2 years ago
Richard Fuchs	9da775ad5a	MT#55283 convert list of attributes from strings ... into a list of object that hold some of the individual pieces of an attribute. Change-Id: Id5de990991c2f9e7892eac8ce2b382fe566a4a40	2 years ago
Richard Fuchs	8c99588550	MT#55283 directly print T.38 attributes Make sure sdp_attributes is only used for _received_ SDP attributes, and not ones that we want to send out. The only places where self-generated attributes are being used is the T.38 gateway. Change-Id: I1f035cd42a35fa250ea477700a1a88ad59efcc38	2 years ago
Richard Fuchs	b7b4890ecd	MT#55283 introduce STR_INIT_GS Construct a `str` from a `GString` Change-Id: I17b1e61cd99380a02bf194eea3538be6e745686a	2 years ago
Richard Fuchs	9bac9bc683	MT#55283 include explicit cast in STR_INIT Some source strings are `const char *` and we don't have an alternative `str` type pointing to a const buffer. Strictly speaking some buffers aren't even char buffers, yet we use the `str` type to manage them. This eliminates the need to explicitly cast every usage. Change-Id: Ifd3cdd3d400fc483e6c488a95e9381938e63cc7f	2 years ago
Donat Zenichev	bebe291656	MT#59086 Unify header files inclusions. In the file implementations follow the rules: 1. Firstly goes the correlated header file, then one empty row. 2. Secondly go system headers, so in angle-brackets, then one empty row. 3. Thirdly, go custom header files, so in double quotes, then one empty row. 4. If there is "xt_RTPENGINE.h", it's mentioned next, but separately, then one empty row. 5. If there are pre-processor definitions, they are added. 6. And eventually at least one empty row before the code. In some situations it's allowed to step aside from the rules, when inclusions are dependent on each other, so on the sequence, and also possibly on some inline objects definitions, but if possible to follow the rules, it's being done. Change-Id: Ie512a970e230fe202398656d1942e8874bb14cd9	2 years ago
Richard Fuchs	cd4f6608cd	MT#55283 convert struct stream_fd to typedef Change-Id: I1435a5f755cb99833535d995871f1e9050f0ec73	2 years ago
Richard Fuchs	74b8a6cce4	MT#55283 use typed GQueue for attributes Change-Id: Ie26f6050461bda8cd863d6ce0d7703952979c362	2 years ago
Richard Fuchs	d9e682b7db	MT#56374 add convenience macros for string creation ... and use them in a few places where it makes sense Change-Id: I849f2841990808de6f697c92e2760645286dacca	3 years ago
Richard Fuchs	8266b2da78	MT#57848 safeguard against NULL dereferences Make sure we don't try to send on closed sockets or to endpoints which haven't been initialised. Fixes unexpected fallout from `83c7336e` Change-Id: If73d61e52edeb72257515adab7428ecef82c2797	3 years ago
Richard Fuchs	5e5e08e377	MT#55283 rename variables to eliminate shadowing Change-Id: Iaf5f98dd4820669e1ee0133dceed8a6a6d791b23	3 years ago
Richard Fuchs	0863302a7b	MT#56471 reset codec handlers when player finishes Return true/false from media player run functions to signal whether playback is finished and whether codec handlers should be reset. Change-Id: Ieb2e0861190ad2851d986bae28becc5548e215ce	3 years ago
Richard Fuchs	70ef2f1a02	MT#55283 separate destination PT determination Split out the logic to determine the destination payload type into a separate functions. This makes it possible to supply a different, or pre-determined, payload type instead. Change-Id: I9b67b29cafc0c6ce4e18eede64dea3d1973f8b63	3 years ago
Richard Fuchs	1d387b98ef	TT#112700 rework codec offer/answer routine Complete overhaul of the codec handling code: ) obsolete flags `asymmetric codecs`, `symmetric codecs`, `reorder codecs` ) support proper codec offer/answer ) split codec manipulation (strip/offer/accept/etc) into separate functions for clarity and better code maintenance ) fully update codec handlers in both directions after an answer ) explicit allocation and handling of codecs and payload types in a codec_store object ) improve codec matchup logic during answer ) more explicit handling of supplemental codecs (CN/DTMF) ) remove now obsolete hacks for handling certain use cases Change-Id: I996705ba8fe339524c2f70e6bb0fd854f9a1f4fb	5 years ago
Richard Fuchs	d5d0a3a994	TT#111150 convert str.len to size_t This makes the type in line with string(3) functions and eliminates some compiler warnings. Also update the related bencode data type. Change-Id: I7ef4024f4b5a0f737b3dbe03bcd078032395bce6	5 years ago
Richard Fuchs	21d1057f68	TT#108003 fix possible deadlock Change-Id: I604bd76248aa0bc33589ee379a99b6f82c840662	5 years ago
Richard Fuchs	30733ec5cd	TT#97301 support granular log levels Change-Id: Ife458bd2449f61113a3e6db1708821570d92dc23	5 years ago
Richard Fuchs	64ba12c4c7	TT#99503 fix possible T.38 gateway deadlock closes #1100 Change-Id: Ifd4be1ca5d4f343a805be31d39abdb73011ec6c4 (cherry picked from commit `0c581e59a1`)	6 years ago
Richard Fuchs	1147f856c0	TT#14008 support building against FreeSwitch version of SpanDSP Change-Id: I472161e6368886240a4ba08e269542fd3f6e4d41	6 years ago
Richard Fuchs	28efacc8d8	TT#76711 more resilient UDPTL handling Change-Id: I11b40cdd0c0562f49bd702f7efa58f9cf01d87e9	6 years ago
Richard Fuchs	d611bbbb74	TT#78501 add T.38 options Change-Id: I4f173f384db18e832c1d24a7015a836ec4215a96	6 years ago
Richard Fuchs	48c3af7f9d	TT#76711 add spandsp logging Change-Id: I75dbf1e420818bf5c4033b0807b38824f96e2915	6 years ago
Richard Fuchs	d872802ba2	TT#76711 T.38 <> audio gateway Change-Id: I4552e07eee632fa730745410b08c3bf70ab67ab2	6 years ago

29 Commits (rfuchs/2095)