From 4d7a90d4e2c408af10dce738d6fc5ca491fcc83e Mon Sep 17 00:00:00 2001 From: Asterisk Development Team Date: Fri, 19 Jun 2020 12:47:29 -0500 Subject: [PATCH] Update for certified/16.8-cert3 --- .version | 2 +- ChangeLog | 110 ++++++++++++++++ asterisk-certified-16.8-cert2-summary.html | 33 ----- asterisk-certified-16.8-cert3-summary.html | 48 +++++++ ... asterisk-certified-16.8-cert3-summary.txt | 124 ++++++++++-------- 5 files changed, 225 insertions(+), 92 deletions(-) delete mode 100644 asterisk-certified-16.8-cert2-summary.html create mode 100644 asterisk-certified-16.8-cert3-summary.html rename asterisk-certified-16.8-cert2-summary.txt => asterisk-certified-16.8-cert3-summary.txt (51%) diff --git a/.version b/.version index 344bf7c545..abf41d8eeb 100644 --- a/.version +++ b/.version @@ -1 +1 @@ -certified/16.8-cert2 \ No newline at end of file +certified/16.8-cert3 \ No newline at end of file diff --git a/ChangeLog b/ChangeLog index 6161909e0f..a026c4a6d7 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,113 @@ +2020-06-19 17:47 +0000 Asterisk Development Team + + * asterisk certified/16.8-cert3 Released. + +2020-06-19 12:04 +0000 [ec8b3850d2] Kevin Harwell + + * Revert "core_unreal / core_local: Add multistream and re-negotiation." + + This reverts commit 2b5b9cd8e0b76f303d567187c4553a2a7edbc1ae. + + Reason for revert: + + Change-Id: I37c31857c5f16715a2debb99e1d68a0202fd1186 + +2020-06-17 03:58 +0000 [6777ec0340] Joshua C. Colp + + * res_pjsip_session: Preserve label on incoming re-INVITE. + + When a re-INVITE is received we create a new set of + streams that are then swapped in as the active streams. + We did not preserve the SDP label from the previous + streams, resulting in the label getting lost. + + This change ensures that if an SDP label is present + on the previous stream then it is set on the new stream. + + ASTERISK-28953 + + Change-Id: I9dd63b88b562fe96ce5c791a3dae5bcaca258445 + +2020-06-03 11:47 +0000 [2b5b9cd8e0] Joshua C. Colp + + * core_unreal / core_local: Add multistream and re-negotiation. + + When requesting a Local channel the requested stream topology + or a converted stream topology will now be placed onto the + resulting channels. + + Frames written in on streams will now also preserve the stream + identifier as they are queued on the opposite channel. + + Finally when a stream topology change is requested it is + immediately accepted and reflected on both channels. Each + channel also receives a queued frame to indicate that the + topology has changed. + + ASTERISK-28938 + + Change-Id: I4e9d94da5230d4bd046dc755651493fce1d87186 + +2020-06-08 06:27 +0000 [744bfb049c] Joshua C. Colp + + * res_rtp_asterisk: Don't assume setting retrans props means to enable. + + The "value" passed in when setting an RTP property determines + whether it should be enabled or disabled. The RTP send and + receive retrans props did not examine this to know if the + buffers should be enabled. They assumed they always should be. + + This change makes it so that the "value" passed in is + respected. + + ASTERISK-28939 + + Change-Id: I9244cdbdc5fd065c7f6b02cbfa572bc55c7123dc + +2020-06-10 12:11 +0000 [53a3be1891] Joshua C. Colp + + * bridge_softmix: Add additional old states for adding new source. + + There are three states that an old stream can be in to allow + becoming a source stream in a new stream: + + 1. Removed + 2. Inactive + 3. Sendonly + + This change adds the two missing ones, inactive and sendonly, + so if a stream transitions from those to a state where they are + providing video to Asterisk we properly re-negotiate the other + participants. + + ASTERISK-28944 + + Change-Id: Id8256b9b254b403411586284bbaedbf50452de01 + +2020-06-07 19:02 +0000 [0fe63c2f87] George Joseph + + * app_confbridge: Plug ref leak of bridge channel with send_events + + When send_events is enabled for a user, we were leaking a reference + to the bridge channel in confbridge_manager.c:send_message(). This + also caused the bridge snapshot to not be destroyed. + + Change-Id: I87a7ae9175e3cd29f6d6a8750e0ec5427bd98e97 + +2020-06-03 12:06 +0000 [10dad4ea3a] Kevin Harwell + + * Compiler fixes for gcc 10 + + This patch fixes a few compile warnings/errors that now occur when using gcc + 10+. + + Also, the Makefile.rules check to turn off partial inlining in gcc versions + greater or equal to 8.2.1 had a bug where it only it only checked against + versions with at least 3 numbers (ex: 8.2.1 vs 10). This patch now ensures + any version above the specified version is correctly compared. + + Change-Id: I54718496eb0c3ce5bd6d427cd279a29e8d2825f9 + 2020-04-30 18:43 +0000 Asterisk Development Team * asterisk certified/16.8-cert2 Released. diff --git a/asterisk-certified-16.8-cert2-summary.html b/asterisk-certified-16.8-cert2-summary.html deleted file mode 100644 index 5cb2717713..0000000000 --- a/asterisk-certified-16.8-cert2-summary.html +++ /dev/null @@ -1,33 +0,0 @@ -Release Summary - asterisk-certified/16.8-cert2

Release Summary

asterisk-certified/16.8-cert2

Date: 2020-04-30

<asteriskteam@digium.com>

Table of Contents

    -
  1. Summary
    2. -
  2. Contributors
    3. -
  3. Closed Issues
    4. -
  4. Other Changes
    5. -
  5. Diffstat
    6. -

Summary

[Back to Top]

This release is a point release of an existing major version. The changes included were made to address problems that have been identified in this release series, or are minor, backwards compatible new features or improvements. Users should be able to safely upgrade to this version if this release series is already in use. Users considering upgrading from a previous version are strongly encouraged to review the UPGRADE.txt document as well as the CHANGES document for information about upgrading to this release series.

The data in this summary reflects changes that have been made since the previous release, asterisk-certified/16.8-cert1.

Contributors

[Back to Top]

This table lists the people who have submitted code, those that have tested patches, as well as those that reported issues on the issue tracker that were resolved in this release. For coders, the number is how many of their patches (of any size) were committed into this release. For testers, the number is the number of times their name was listed as assisting with testing a patch. Finally, for reporters, the number is the number of issues that they reported that were affected by commits that went into this release.

- - -
CodersTestersReporters
2 Joshua Colp
2 Joshua C. Colp
1 George Joseph
1 Alexander Traud
1 Joshua C. Colp
1 Joeran Vinzens
1 Alexander Traud
1 Alexey Vasilyev

Closed Issues

[Back to Top]

This is a list of all issues from the issue tracker that were closed by changes that went into this release.

Bug

Category: Core/BuildSystem

ASTERISK-28824: BuildSystem: Search for Python/C API when possibly needed only.
Reported by: Alexander Traud
    -
  • [eeab038b01] Alexander Traud -- BuildSystem: Search for Python/C API when possibly needed only.
    • -
ASTERISK-27717: [patch] BuildSystem: In NetBSD, the Python Programming Language is python-2.7.
Reported by: Alexander Traud
    -
  • [eeab038b01] Alexander Traud -- BuildSystem: Search for Python/C API when possibly needed only.
    • -

Category: Core/General

ASTERISK-28839: Sporadic crashes with Segmentation fault
Reported by: Joeran Vinzens
    -
  • [02188feb32] Joshua C. Colp -- fax: Fix crashes in PJSIP re-negotiation scenarios.
    • -

Category: Tests/testsuite

ASTERISK-27717: [patch] BuildSystem: In NetBSD, the Python Programming Language is python-2.7.
Reported by: Alexander Traud
    -
  • [eeab038b01] Alexander Traud -- BuildSystem: Search for Python/C API when possibly needed only.
    • -

Category: pjproject/pjsip

ASTERISK-28811: Crash occurs when fax session switches from T.38 to audio
Reported by: Alexey Vasilyev
    -
  • [02188feb32] Joshua C. Colp -- fax: Fix crashes in PJSIP re-negotiation scenarios.
    • -
ASTERISK-28859: pjsip: Increase maximum candidate count
Reported by: Joshua C. Colp
    -
  • [05671db0c3] Joshua C. Colp -- pjsip: Increase maximum ICE candidate count.
    • -

Commits Not Associated with an Issue

[Back to Top]

This is a list of all changes that went into this release that did not reference a JIRA issue.

- - - - -
RevisionAuthorSummary
d3fae6d7a1George Josephapp_voicemail: Add workaround for a gcc 10 issue with -Wrestrict
40d379f59eJoshua ColpRevert "Revert "res_rtp_asterisk: Free payload when error on insertion to data buffer""
491f0bae7dJoshua ColpRevert "Revert "res_rtp_asterisk: Resolve loop when receive buffer is flushed""

Diffstat Results

[Back to Top]

This is a summary of the changes to the source code that went into this release that was generated using the diffstat utility.

apps/app_voicemail.c           |   13 +-
-configure                      |  198 ++++++++++++++++++++++++++++++++++++++++-
-configure.ac                   |   23 +++-
-contrib/scripts/install_prereq |   18 +--
-main/data_buffer.c             |    2
-res/res_pjsip_session.c        |   14 ++
-6 files changed, 242 insertions(+), 26 deletions(-)

\ No newline at end of file diff --git a/asterisk-certified-16.8-cert3-summary.html b/asterisk-certified-16.8-cert3-summary.html new file mode 100644 index 0000000000..aaec4d4ea8 --- /dev/null +++ b/asterisk-certified-16.8-cert3-summary.html @@ -0,0 +1,48 @@ +Release Summary - asterisk-certified/16.8-cert3

Release Summary

asterisk-certified/16.8-cert3

Date: 2020-06-19

<asteriskteam@digium.com>

Table of Contents

    +
  1. Summary
    2. +
  2. Contributors
    3. +
  3. Closed Issues
    4. +
  4. Other Changes
    5. +
  5. Diffstat
    6. +

Summary

[Back to Top]

This release is a point release of an existing major version. The changes included were made to address problems that have been identified in this release series, or are minor, backwards compatible new features or improvements. Users should be able to safely upgrade to this version if this release series is already in use. Users considering upgrading from a previous version are strongly encouraged to review the UPGRADE.txt document as well as the CHANGES document for information about upgrading to this release series.

The data in this summary reflects changes that have been made since the previous release, asterisk-certified/16.8-cert2.

Contributors

[Back to Top]

This table lists the people who have submitted code, those that have tested patches, as well as those that reported issues on the issue tracker that were resolved in this release. For coders, the number is how many of their patches (of any size) were committed into this release. For testers, the number is the number of times their name was listed as assisting with testing a patch. Finally, for reporters, the number is the number of issues that they reported that were affected by commits that went into this release.

+ + +
CodersTestersReporters
4 Joshua C. Colp
2 Kevin Harwell
1 George Joseph
4 Joshua C. Colp

Closed Issues

[Back to Top]

This is a list of all issues from the issue tracker that were closed by changes that went into this release.

Bug

Category: Bridges/bridge_softmix

ASTERISK-28944: bridge_softmix: Transitioning a stream from inactive -> sendrecv/sendonly doesn't re-negotiation
Reported by: Joshua C. Colp
    +
  • [53a3be1891] Joshua C. Colp -- bridge_softmix: Add additional old states for adding new source.
    • +

Category: Channels/chan_local

ASTERISK-28938: core_unreal / core_local: Add support for multistream and re-negotiation
Reported by: Joshua C. Colp
    +
  • [2b5b9cd8e0] Joshua C. Colp -- core_unreal / core_local: Add multistream and re-negotiation.
    • +

Category: Resources/res_pjsip_session

ASTERISK-28953: res_pjsip_session: Preserve stream label
Reported by: Joshua C. Colp
    +
  • [6777ec0340] Joshua C. Colp -- res_pjsip_session: Preserve label on incoming re-INVITE.
    • +

Category: Resources/res_rtp_asterisk

ASTERISK-28939: res_rtp_asterisk: Don't have send/receive buffers on non-WebRTC
Reported by: Joshua C. Colp
    +
  • [744bfb049c] Joshua C. Colp -- res_rtp_asterisk: Don't assume setting retrans props means to enable.
    • +

Commits Not Associated with an Issue

[Back to Top]

This is a list of all changes that went into this release that did not reference a JIRA issue.

+ + + + +
RevisionAuthorSummary
ec8b3850d2Kevin HarwellRevert "core_unreal / core_local: Add multistream and re-negotiation."
0fe63c2f87George Josephapp_confbridge: Plug ref leak of bridge channel with send_events
10dad4ea3aKevin HarwellCompiler fixes for gcc 10

Diffstat Results

[Back to Top]

This is a summary of the changes to the source code that went into this release that was generated using the diffstat utility.

Makefile.rules                        |   19 ++++++++++---------
+addons/ooh323c/src/decode.c           |   10 +++++++---
+addons/ooh323c/src/ooSocket.c         |    2 +-
+addons/ooh323c/src/oochannels.c       |    2 +-
+apps/app_stack.c                      |   11 ++++++-----
+apps/app_voicemail.c                  |   28 ++++++++++++++++------------
+apps/confbridge/confbridge_manager.c  |    2 +-
+bridges/bridge_softmix.c              |    3 ++-
+include/asterisk/app.h                |    7 +++++--
+main/dns_srv.c                        |    6 ++++--
+main/http.c                           |    6 ++++--
+main/message.c                        |    2 +-
+main/pbx.c                            |    7 +++++--
+main/pbx_variables.c                  |    8 ++------
+main/stasis.c                         |   12 ++++++++----
+pbx/pbx_dundi.c                       |   15 +++++++--------
+res/parking/parking_bridge_features.c |   11 +++++++----
+res/res_pjsip_registrar.c             |   13 ++++++++-----
+res/res_pjsip_session.c               |   16 ++++++++++++++++
+res/res_rtp_asterisk.c                |   25 ++++++++++++++++++++++---
+tests/test_locale.c                   |   12 ++++++++----
+utils/astman.c                        |    2 +-
+utils/db1-ast/hash/ndbm.c             |    3 +--
+utils/extconf.c                       |   10 ++++++----
+utils/muted.c                         |    2 +-
+25 files changed, 150 insertions(+), 84 deletions(-)

\ No newline at end of file diff --git a/asterisk-certified-16.8-cert2-summary.txt b/asterisk-certified-16.8-cert3-summary.txt similarity index 51% rename from asterisk-certified-16.8-cert2-summary.txt rename to asterisk-certified-16.8-cert3-summary.txt index 3f83198a0c..47bfcf88bd 100644 --- a/asterisk-certified-16.8-cert2-summary.txt +++ b/asterisk-certified-16.8-cert3-summary.txt @@ -1,8 +1,8 @@ Release Summary - asterisk-certified/16.8-cert2 + asterisk-certified/16.8-cert3 - Date: 2020-04-30 + Date: 2020-06-19 @@ -32,7 +32,7 @@ to this release series. The data in this summary reflects changes that have been made since the - previous release, asterisk-certified/16.8-cert1. + previous release, asterisk-certified/16.8-cert2. ---------------------------------------------------------------------- @@ -50,10 +50,9 @@ this release. Coders Testers Reporters - 2 Joshua Colp 1 Joshua C. Colp - 2 Joshua C. Colp 1 Joeran Vinzens - 1 George Joseph 1 Alexander Traud - 1 Alexander Traud 1 Alexey Vasilyev + 4 Joshua C. Colp 4 Joshua C. Colp + 2 Kevin Harwell + 1 George Joseph ---------------------------------------------------------------------- @@ -66,44 +65,36 @@ Bug - Category: Core/BuildSystem - - ASTERISK-28824: BuildSystem: Search for Python/C API when possibly needed - only. - Reported by: Alexander Traud - * [eeab038b01] Alexander Traud -- BuildSystem: Search for Python/C API - when possibly needed only. - ASTERISK-27717: [patch] BuildSystem: In NetBSD, the Python Programming - Language is python-2.7. - Reported by: Alexander Traud - * [eeab038b01] Alexander Traud -- BuildSystem: Search for Python/C API - when possibly needed only. - - Category: Core/General - - ASTERISK-28839: Sporadic crashes with Segmentation fault - Reported by: Joeran Vinzens - * [02188feb32] Joshua C. Colp -- fax: Fix crashes in PJSIP - re-negotiation scenarios. - - Category: Tests/testsuite - - ASTERISK-27717: [patch] BuildSystem: In NetBSD, the Python Programming - Language is python-2.7. - Reported by: Alexander Traud - * [eeab038b01] Alexander Traud -- BuildSystem: Search for Python/C API - when possibly needed only. - - Category: pjproject/pjsip - - ASTERISK-28811: Crash occurs when fax session switches from T.38 to audio - Reported by: Alexey Vasilyev - * [02188feb32] Joshua C. Colp -- fax: Fix crashes in PJSIP - re-negotiation scenarios. - ASTERISK-28859: pjsip: Increase maximum candidate count + Category: Bridges/bridge_softmix + + ASTERISK-28944: bridge_softmix: Transitioning a stream from inactive -> + sendrecv/sendonly doesn't re-negotiation + Reported by: Joshua C. Colp + * [53a3be1891] Joshua C. Colp -- bridge_softmix: Add additional old + states for adding new source. + + Category: Channels/chan_local + + ASTERISK-28938: core_unreal / core_local: Add support for multistream and + re-negotiation + Reported by: Joshua C. Colp + * [2b5b9cd8e0] Joshua C. Colp -- core_unreal / core_local: Add + multistream and re-negotiation. + + Category: Resources/res_pjsip_session + + ASTERISK-28953: res_pjsip_session: Preserve stream label + Reported by: Joshua C. Colp + * [6777ec0340] Joshua C. Colp -- res_pjsip_session: Preserve label on + incoming re-INVITE. + + Category: Resources/res_rtp_asterisk + + ASTERISK-28939: res_rtp_asterisk: Don't have send/receive buffers on + non-WebRTC Reported by: Joshua C. Colp - * [05671db0c3] Joshua C. Colp -- pjsip: Increase maximum ICE candidate - count. + * [744bfb049c] Joshua C. Colp -- res_rtp_asterisk: Don't assume setting + retrans props means to enable. ---------------------------------------------------------------------- @@ -117,15 +108,13 @@ +------------------------------------------------------------------------+ | Revision | Author | Summary | |------------+---------------+-------------------------------------------| - | d3fae6d7a1 | George Joseph | app_voicemail: Add workaround for a gcc | - | | | 10 issue with -Wrestrict | + | ec8b3850d2 | Kevin Harwell | Revert "core_unreal / core_local: Add | + | | | multistream and re-negotiation." | |------------+---------------+-------------------------------------------| - | | | Revert "Revert "res_rtp_asterisk: Free | - | 40d379f59e | Joshua Colp | payload when error on insertion to data | - | | | buffer"" | + | 0fe63c2f87 | George Joseph | app_confbridge: Plug ref leak of bridge | + | | | channel with send_events | |------------+---------------+-------------------------------------------| - | 491f0bae7d | Joshua Colp | Revert "Revert "res_rtp_asterisk: Resolve | - | | | loop when receive buffer is flushed"" | + | 10dad4ea3a | Kevin Harwell | Compiler fixes for gcc 10 | +------------------------------------------------------------------------+ ---------------------------------------------------------------------- @@ -137,10 +126,29 @@ This is a summary of the changes to the source code that went into this release that was generated using the diffstat utility. - apps/app_voicemail.c | 13 +- - configure | 198 ++++++++++++++++++++++++++++++++++++++++- - configure.ac | 23 +++- - contrib/scripts/install_prereq | 18 +-- - main/data_buffer.c | 2 - res/res_pjsip_session.c | 14 ++ - 6 files changed, 242 insertions(+), 26 deletions(-) + Makefile.rules | 19 ++++++++++--------- + addons/ooh323c/src/decode.c | 10 +++++++--- + addons/ooh323c/src/ooSocket.c | 2 +- + addons/ooh323c/src/oochannels.c | 2 +- + apps/app_stack.c | 11 ++++++----- + apps/app_voicemail.c | 28 ++++++++++++++++------------ + apps/confbridge/confbridge_manager.c | 2 +- + bridges/bridge_softmix.c | 3 ++- + include/asterisk/app.h | 7 +++++-- + main/dns_srv.c | 6 ++++-- + main/http.c | 6 ++++-- + main/message.c | 2 +- + main/pbx.c | 7 +++++-- + main/pbx_variables.c | 8 ++------ + main/stasis.c | 12 ++++++++---- + pbx/pbx_dundi.c | 15 +++++++-------- + res/parking/parking_bridge_features.c | 11 +++++++---- + res/res_pjsip_registrar.c | 13 ++++++++----- + res/res_pjsip_session.c | 16 ++++++++++++++++ + res/res_rtp_asterisk.c | 25 ++++++++++++++++++++++--- + tests/test_locale.c | 12 ++++++++---- + utils/astman.c | 2 +- + utils/db1-ast/hash/ndbm.c | 3 +-- + utils/extconf.c | 10 ++++++---- + utils/muted.c | 2 +- + 25 files changed, 150 insertions(+), 84 deletions(-) -- 2.47.2