Skip to content
Snippets Groups Projects
  1. Aug 11, 2016
  2. Aug 10, 2016
    • Richard Mudgett's avatar
      res_srtp: Move SDP SRTP code from the core to res_srtp. · 41aba83f
      Richard Mudgett authored
      A patch made to the master branch (Now the 14 branch) inadvertently made
      libsrtp a required dependency in order to compile Asterisk.  Rather than
      create dummy defines to substitute for the defines supplied by libsrtp
      when libsrtp is not available, most of the code in sdp_srtp.c is moved
      into res_srtp.c.  This gets more code out of Asterisk's core that isn't
      used when SRTP is not available.  This also makes another inadvertent
      required dependency on libsrtp by Asterisk's core unlikely.
      
      ASTERISK-26253 #close
      Reported by: Ben Merrills
      
      Change-Id: I0a46cde81501c0405399c2588633ae32706d1ee7
      41aba83f
    • Alexei Gradinari's avatar
      pjsip: Fix deadlock with suspend taskprocessor on masquerade · 82087941
      Alexei Gradinari authored
      If both channels which should be masqueraded
      are in the same serializer:
      1st channel will be locked waiting condition 'complete'
      2nd channel will be locked waiting condition 'suspended'
      
      On heavy load system a chance that both channels will be in
      the same serializer 'pjsip/distibutor' is very high.
      
      To reproduce compile res_pjsip/pjsip_distributor.c with
      DISTRIBUTOR_POOL_SIZE=1
      
      Steps to reproduce:
      1. Party A calls Party B (bridged call 'AB')
      2. Party B places Party A on hold
      3. Party B calls Voicemail app (non-bridged call 'BV')
      4. Party B attended transfers Party A to voicemail using REFER.
      5. When asterisk masquerades calls 'AB' and 'BV',
         a deadlock is happened.
      
      This patch adds a suspension indicator to the taskprocessor.
      When a session suspends/unsuspends the serializer
      it sets the indicator to the appropriate state.
      The session checks the suspension indicator before
      suspend the serializer.
      
      ASTERISK-26145 #close
      
      Change-Id: Iaaebee60013a58c942ba47b1b4930a63e686663b
      82087941
    • Kevin Harwell's avatar
      alembic/sqlalchemy: auto increment only allowed on a single column · d4170df4
      Kevin Harwell authored
      The extensions table defined two columns (id and priority) as primary key
      autoincrement columns. However only one is allowed when defining the primary
      key.
      
      This patch removes the autoincrement attribute from the priority column since
      it does not need to be as such and really should not have been on there in the
      first place.
      
      This patch also removes 'context', 'exten', and 'priority' from the primary key
      index and creates a new combined unique contraint index on them.
      
      ASTERISK-26183 #close
      
      Change-Id: Ib9c712c612a4d7ec1edb0dcb77f1bae0905a470b
      d4170df4
    • zuul's avatar
      d78fe8fe
    • zuul's avatar
  3. Aug 09, 2016
    • zuul's avatar
      26921a55
    • Mark Michelson's avatar
      res_rtp_asterisk: Cache local RTCP address. · 8fe9f1f7
      Mark Michelson authored
      When an RTCP packet is sent or received, res_rtp_asterisk generates a
      Stasis event that contains the RTCP report as well as the local and
      remote addresses that the report pertains to.
      
      The addresses are determined using ast_find_ourip(). For the local
      address, this will typically result in a lookup of the hostname of the
      server, and then a DNS lookup of that hostname. If you do not have the
      host in /etc/hosts, then this results in a full DNS lookup, which can
      potentially block for some time.
      
      This is especially problematic when performing RTCP reads, since those
      are done on the same thread responsible for reading and writing media.
      
      This patch addresses the issue by performing a lookup of the local
      address when RTCP is allocated. We then use this cached local address
      for the Stasis events when necessary.
      
      ASTERISK-26280 #close
      Reported by Mark Michelson
      
      Change-Id: I3dd61882c2e57036f09f0c390cf38f7c87e9b556
      8fe9f1f7
    • zuul's avatar
    • Corey Farrell's avatar
      Produce friendly error when AST_MODULE_SELF_SYM is not defined. · 827457dc
      Corey Farrell authored
      Modules must define AST_MODULE_SELF_SYM to be used as the name of a
      generated function.  This produces a friendly error when it's not
      defined.
      
      ASTERISK-26278 #close
      
      Change-Id: Ib9d35a08104529c516d636771365e02c6e77a45b
      827457dc
  4. Aug 08, 2016
    • Alexei Gradinari's avatar
      res_pjsip_mwi: fix unsolicited mwi blocks PJSIP stack · 403b6357
      Alexei Gradinari authored
      The PJSIP taskprocessors could be overflowed on startup
      if there are many (thousands) realtime endpoints
      configured with unsolicited mwi.
      The PJSIP stack could be totally unresponsive for a few minutes
      after boot completed.
      
      This patch creates a separate PJSIP serializers pool for mwi
      and makes unsolicited mwi use serializers from this pool.
      This patch also adds 2 new global options to tune taskprocessor
      alert levels: 'mwi_tps_queue_high' and 'mwi_tps_queue_low'.
      
      This patch also adds new global option 'mwi_disable_initial_unsolicited'
      to disable sending unsolicited mwi to all endpoints on startup.
      If disabled then unsolicited mwi will start processing
      on next endpoint's contact update.
      
      ASTERISK-26230 #close
      
      Change-Id: I4c8ecb82c249eb887930980a800c9f87f28f861a
      403b6357
  5. Aug 05, 2016
    • Alexei Gradinari's avatar
      app_voicemail: Add taskprocessor alert level options. · 9042ad40
      Alexei Gradinari authored
      On heavy loaded system with IMAP or DB storage,
      'app_voicemail' taskprocessor queue could reach 500 scheduled tasks.
      It could happen when the IMAP or DB server dies or is unreachable.
      It could happen on startup when there are many (thousands)
      realtime endpoints configured with unsolicited mwi.
      If the taskprocessor queue reaches the high water level
      then the alert is triggered and pjsip stops processing new requests
      until the queue reaches the low water level to clear the alert.
      
      This patch adds 2 new 'general' configuration options
      to tune taskprocessor alert levels:
      'tps_queue_high' - Taskprocessor high water alert trigger level.
      'tps_queue_low' - Taskprocessor low water clear alert level
      
      ASTERISK-26229 #close
      
      Change-Id: I766294fbffedf64053c0d9ac0bedd3109f043ee8
      9042ad40
    • Joshua Colp's avatar
      res_pjsip_outbound_publish: Use a serializer shutdown group for unload. · 54869e48
      Joshua Colp authored
      This change replaces the custom unload process for the outbound
      publish module with the common serializer shutdown group.
      
      ASTERISK-25217 #close
      
      Change-Id: I280a0384d860c486202d87d2d674394cca77ffb6
      54869e48
  6. Aug 04, 2016
  7. Aug 03, 2016
  8. Aug 02, 2016
  9. Aug 01, 2016
  10. Jul 29, 2016
    • Mark Michelson's avatar
      Remove SILK payload mappings from Asterisk core. · 1cd79d6e
      Mark Michelson authored
      SILK is a bit of a hog when it comes to using up our limited number of
      dynamic payload types in the RTP engine. By freeing up four slots, it
      allows for other codecs to potentially take the place.
      
      Now, codec_silk.so will dynamically use the payload slots in the RTP
      engine when it loads.
      
      A better fix would be make RTP dynamic payload types actually
      dynamic. However, at this stage of Asterisk 14 development, this is a
      risky move that would be imprudent.
      
      Change-Id: I5774e09408f9a203db189529eabdc0d3f4c1e612
      1cd79d6e
    • zuul's avatar
      d462cb73
    • Joshua Colp's avatar
    • Joshua Colp's avatar
      astconfigparser: Handle case where line is simply a comment. · a7ae4844
      Joshua Colp authored
      Change-Id: I2dea5815363f4d787d709228a04f33baee383ef5
      a7ae4844
    • Joshua Colp's avatar
    • Corey Farrell's avatar
      pbx.c: Fix handling of '-' in extension name and callerid · 89a0a1eb
      Corey Farrell authored
      This adds a two strings to ast_exten.  name to go with exten and
      cidmatch_display to go with cidmatch.  The new fields contain input used
      to add the extension in the first place.  The existing fields now
      contain stripped input that excludes insignificant spaces and dashes.
      These stripped fields should always be used for comparisons.  The
      unstripped fields should normally be used for display, but displaying
      stripped values will not cause runtime errors.
      
      Note the actual string is only stored twice if it contains dashes.  If
      no dashes are found then both 'char *' fields point to the same memory.
      So this change has a minimum effect on memory usage.
      
      The existing functions ast_get_extension_name and
      ast_get_extension_cidmatch return unstripped values as they did before
      this change.  Other similar bugs likely still exist where unstripped
      extensions are saved outside pbx.c then passed back in.
      
      ASTERISK-26233 #close
      
      Change-Id: I6cd61ce57acc1570ca6cc14960c4c3b0a9eb837f
      89a0a1eb
  11. Jul 28, 2016
Loading