Asterisk
  1. Asterisk
  2. ASTERISK-19941

Crash in res_config_ldap when used with realtime extensions

    Details

    • Type: Bug Bug
    • Status: Closed
    • Severity: Critical Critical
    • Resolution: Cannot Reproduce
    • Affects Version/s: 1.8.11.1, 11.5.1
    • Target Release Version/s: None
    • Security Level: None
    • Labels:
    • Environment:
      Asterisk Now 2.2 GUI and LDAP Imap voicemail registration

      Description

      Asterisk trying to modify entries in LDAP server DS 389.
      Also when I got sip user registered and I trying place call it just silence, I don't see any attempts to dial in logs.

      [2012-05-28 00:07:18] ERROR[25975]: res_config_ldap.c:1328 update_ldap: Couldn't modify 'name'='sbendersky', dn:uid=,ou=People,dc=,dc= because No such attribute
      – Registered SIP 'sbendersky' at 10.16.130.45:32899
      [2012-05-28 00:07:20] ERROR[26001]: res_config_ldap.c:1328 update_ldap: Couldn't modify 'name'='sbendersky', dn:uid=sbendersky,ou=People,dc=networklab,dc=ca because Object class violation
      – Unregistered SIP 'sbendersky'
      [2012-05-28 00:07:26] ERROR[26001]: res_config_ldap.c:1328 update_ldap: Couldn't modify 'name'='sbendersky', dn:uid=sbendersky,ou=People,dc=networklab,dc=ca because No such attribute

      Extension conf

      [users]
      switch => Realtime/@

      [voicemail]
      switch => Realtime/@

      [realtime-ldap]
      include => users
      include => voicemail

      Extconfig conf

      [settings]
      sipusers => ldap,"ou=People,dc=networklab,dc=ca",sip
      sippeers => ldap,"ou=People,dc=networklab,dc=ca",sip
      extensions => ldap,"ou=NetLabExtensions,dc=networklab,dc=ca",extensions
      voicemail => ldap,"ou=NetLabVoicemail,dc=networklab,dc=ca",voicemail

        Issue Links

          Activity

          Hide
          Matt Jordan added a comment -

          Patches cannot be accepted in-line. They must be attached to the issue in unified diff format after signing a license contributor agreement.

          Once a patch is attached to this issue, you can submit it for code review on Review Board.

          Please see https://wiki.asterisk.org/wiki/display/AST/Patch+Contribution+Process for more information.

          Show
          Matt Jordan added a comment - Patches cannot be accepted in-line. They must be attached to the issue in unified diff format after signing a license contributor agreement. Once a patch is attached to this issue, you can submit it for code review on Review Board. Please see https://wiki.asterisk.org/wiki/display/AST/Patch+Contribution+Process for more information.
          Hide
          Jeremy Lainé added a comment -

          Any progress on fixing this bug?

          Show
          Jeremy Lainé added a comment - Any progress on fixing this bug?
          Hide
          Chris Ridd added a comment -

          Looking at the trunk it would appear that the code was changed in r386731 to avoid this buggy behaviour:

          r386731 | file | 2013-04-27 13:01:29 +0100 (Sat, 27 Apr 2013) | 11 lines

          Add support for a realtime sorcery module.

          This change does the following:

          1. Adds the sorcery realtime module
          2. Adds unit tests for the sorcery realtime module
          3. Changes the realtime core to use an ast_variable list instead of variadic arguments
          4. Changes all realtime drivers to accept an ast_variable list

          Review: https://reviewboard.asterisk.org/r/2424/

          That change is in the version 12 branch but not the version 11 branch.

          Show
          Chris Ridd added a comment - Looking at the trunk it would appear that the code was changed in r386731 to avoid this buggy behaviour: r386731 | file | 2013-04-27 13:01:29 +0100 (Sat, 27 Apr 2013) | 11 lines Add support for a realtime sorcery module. This change does the following: 1. Adds the sorcery realtime module 2. Adds unit tests for the sorcery realtime module 3. Changes the realtime core to use an ast_variable list instead of variadic arguments 4. Changes all realtime drivers to accept an ast_variable list Review: https://reviewboard.asterisk.org/r/2424/ That change is in the version 12 branch but not the version 11 branch.
          Hide
          Sean Bright added a comment -

          Can anyone confirm whether or not this is an issue in Asterisk 13? I am not able to reproduce this.

          Show
          Sean Bright added a comment - Can anyone confirm whether or not this is an issue in Asterisk 13? I am not able to reproduce this.
          Hide
          Sean Bright added a comment -

          As pointed out by Chris Ridd, the segfault you were seeing was most likely caused by using {{va_list}}s inappropriately, and that code was replaced in April 2013 in r386731.

          If you experience this crash with Asterisk 13 or later, please feel free to re-open.

          Show
          Sean Bright added a comment - As pointed out by Chris Ridd, the segfault you were seeing was most likely caused by using {{va_list}}s inappropriately, and that code was replaced in April 2013 in r386731. If you experience this crash with Asterisk 13 or later, please feel free to re-open.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development