Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[mod_sofia] Prevent unnecessary re-processing of unmodified SDP #2452

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

olofssonanton
Copy link
Contributor

Always clear flag TFLAG_NEW_SDP.

This prevents re-processing of unmodified SDP during re-INVITEs. Before this change a remote party's UNMODIFIED answer to a re-INVITE would always be re-processed once, due to TFLAG_NEW_SDP still being set. Any subsequent re-INVITE thereafter would then behave as expected, as TFLAG_NEW_SDP would then have been cleared.

This prevents re-processing of unmodified SDP during re-INVITEs.
Before this change a remote party's UNMODIFIED answer to a re-INVITE would always be re-processed once,
due to TFLAG_NEW_SDP still being set.
@olofssonanton
Copy link
Contributor Author

This fixes an issue where an unexpected payload is inserted into the wrong decoder after momentarily switching codecs (even though there was no need). This was made apparent after changes made in #2369, as mentioned here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant