Asterisk
  1. Asterisk
  2. ASTERISK-27550

[patch] bridge_softmix: Avoid warning about an uninitialized variable.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Severity: Major Major
    • Resolution: Fixed
    • Affects Version/s: 15.1.5
    • Target Release Version/s: 15.3.0, 16.0.0
    • Component/s: Bridges/bridge_softmix
    • Security Level: None
    • Labels:

      Description

      The compiler clang 5.0.1 reports:

      bridge_softmix.c:1407:56: warning: variable 'bridge_channel' is uninitialized when used here [-Wuninitialized]
                      check_binaural_position_change(bridge, softmix_data, bridge_channel);
                                                                           ^~~~~~~~~~~~~~
      bridge_softmix.c:1367:44: note: initialize the variable 'bridge_channel' to silence this warning
                      struct ast_bridge_channel *bridge_channel;
                                                               ^
                                                                = NULL

        Issue Links

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

          Hide
          Friendly Automation added a comment -

          Change 7821 merged by Jenkins2:
          bridge_softmix: Removed unused parameter from check_binaural_position_change(.).

          https://gerrit.asterisk.org/7821

          Show
          Friendly Automation added a comment - Change 7821 merged by Jenkins2: bridge_softmix: Removed unused parameter from check_binaural_position_change(.). https://gerrit.asterisk.org/7821
          Hide
          Friendly Automation added a comment -

          Change 7819 merged by Jenkins2:
          bridge_softmix: Removed unused parameter from check_binaural_position_change(.).

          https://gerrit.asterisk.org/7819

          Show
          Friendly Automation added a comment - Change 7819 merged by Jenkins2: bridge_softmix: Removed unused parameter from check_binaural_position_change(.). https://gerrit.asterisk.org/7819

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: