Asterisk
  1. Asterisk
  2. ASTERISK-26579

codec_opus: Recursiveness when parsing fmtp line

    Details

    • Frequency of Occurrence:
      Constant

      Description

      recursive stack overflow

      sdp_fmtp_get (attributes=<optimized out>, name=0x7ff5071de32c "ptime", attr=0x7ff4fc0548f8) at res_format_attr_opus.c:120
      #6450 0x00007ff5071dc957 in sdp_fmtp_get (attr=<optimized out>, name=<optimized out>, attributes=<optimized out>) at res_format_attr_opus.c:120
      

      sdp is

      v=0
      o=- 975991180 975991180 IN IP4 xxx
      s=Asterisk
      c=IN IP4 xxx
      t=0 0
      m=audio 25524 RTP/AVP 107 8 0 4 9 101
      a=rtpmap:107 opus/48000/2
      a=rtpmap:8 PCMA/8000
      a=rtpmap:0 PCMU/8000
      a=rtpmap:4 G723/8000
      a=rtpmap:9 G722/8000
      a=rtpmap:101 telephone-event/8000
      a=fmtp:101 0-16
      a=ptime:20
      a=maxptime:20
      
      1. res_format_attr_opus.diff
        1 kB
        Jørgen H
      2. res_format_attr_opus.diff
        1 kB
        Jørgen H

        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 4589 merged by Kevin Harwell:
          res_format_attr_opus: Fix crash when fmtp contains spaces.

          https://gerrit.asterisk.org/4589

          Show
          Friendly Automation added a comment - Change 4589 merged by Kevin Harwell: res_format_attr_opus: Fix crash when fmtp contains spaces. https://gerrit.asterisk.org/4589
          Hide
          Friendly Automation added a comment -

          Change 4578 merged by Kevin Harwell:
          res_format_attr_opus: Fix crash when fmtp contains spaces.

          https://gerrit.asterisk.org/4578

          Show
          Friendly Automation added a comment - Change 4578 merged by Kevin Harwell: res_format_attr_opus: Fix crash when fmtp contains spaces. https://gerrit.asterisk.org/4578
          Hide
          Friendly Automation added a comment -

          Change 4588 merged by Kevin Harwell:
          res_format_attr_opus: Fix crash when fmtp contains spaces.

          https://gerrit.asterisk.org/4588

          Show
          Friendly Automation added a comment - Change 4588 merged by Kevin Harwell: res_format_attr_opus: Fix crash when fmtp contains spaces. https://gerrit.asterisk.org/4588
          Hide
          Friendly Automation added a comment -

          Change 4581 merged by Kevin Harwell:
          res_format_attr_opus: Fix crash when fmtp contains spaces.

          https://gerrit.asterisk.org/4581

          Show
          Friendly Automation added a comment - Change 4581 merged by Kevin Harwell: res_format_attr_opus: Fix crash when fmtp contains spaces. https://gerrit.asterisk.org/4581
          Hide
          Friendly Automation added a comment -

          Change 4577 merged by Kevin Harwell:
          codecs/opus/fmtp_with_spaces: Add test for fmtp parsing with spaces.

          https://gerrit.asterisk.org/4577

          Show
          Friendly Automation added a comment - Change 4577 merged by Kevin Harwell: codecs/opus/fmtp_with_spaces: Add test for fmtp parsing with spaces. https://gerrit.asterisk.org/4577

            People

            • Watchers:
              7 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: