[asterisk-bugs] [Asterisk 0012600]: If binding as anonymous, no need to save password or use a default

Tue May 13 13:31:53 CDT 2008

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=12600 
====================================================================== 
Reported By:                suretec
Assigned To:                russell
====================================================================== 
Project:                    Asterisk
Issue ID:                   12600
Category:                   Resources/res_config_ldap
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     closed
Asterisk Version:           1.6.0-beta8 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 115506 
Disclaimer on File?:        N/A 
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             05-07-2008 10:18 CDT
Last Modified:              05-13-2008 13:31 CDT
====================================================================== 
Summary:                    If binding as anonymous, no need to save password or
use a default
Description: 
        if (!(s = ast_variable_retrieve(config, "_general", "user"))) {
                ast_log(LOG_WARNING, "No directory user found, anonymous
binding as default.\n");
                user[0] = '\0';
        } else
                ast_copy_string(user, s, sizeof(user));

        if (!(s = ast_variable_retrieve(config, "_general", "pass"))) {
                ast_log(LOG_WARNING, "No directory password found, using
'asterisk' as default.\n");
                ast_copy_string(pass, "asterisk", sizeof(pass) - 1);
        } else
                ast_copy_string(pass, s, sizeof(pass));
====================================================================== 

---------------------------------------------------------------------- 
 svnbot - 05-13-08 13:31  
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 115885

_U  team/group/newcdr/
U   team/group/newcdr/channels/chan_h323.c
U   team/group/newcdr/channels/chan_iax2.c
U   team/group/newcdr/channels/chan_sip.c
U   team/group/newcdr/contrib/scripts/asterisk.ldap-schema
U   team/group/newcdr/contrib/scripts/asterisk.ldif
_U  team/group/newcdr/include/asterisk/dlinkedlists.h
U   team/group/newcdr/main/sched.c
U   team/group/newcdr/res/res_config_ldap.c
U   team/group/newcdr/res/res_odbc.c
U   team/group/newcdr/sounds/Makefile

------------------------------------------------------------------------
r115885 | murf | 2008-05-13 13:31:15 -0500 (Tue, 13 May 2008) | 110 lines

Merged revisions
115507,115509,115513,115515,115518-115519,115521,115523,115525,115535,115537
via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

................
r115507 | file | 2008-05-07 07:41:25 -0600 (Wed, 07 May 2008) | 4 lines

Remove redundant header getting.
(closes issue http://bugs.digium.com/view.php?id=12597)
Reported by: hooi

................
r115509 | tilghman | 2008-05-07 07:49:15 -0600 (Wed, 07 May 2008) | 6
lines

Update typos in description fields
(closes issue http://bugs.digium.com/view.php?id=12598)
 Reported by: suretec
 Patches: 
       asterisk_schema_changes.patch uploaded by suretec (license 70)

................
r115513 | russell | 2008-05-07 11:28:19 -0600 (Wed, 07 May 2008) | 19
lines

Merged revisions 115512 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

................
r115512 | russell | 2008-05-07 11:24:09 -0500 (Wed, 07 May 2008) | 11
lines

Merged revisions 115511 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r115511 | russell | 2008-05-07 11:22:49 -0500 (Wed, 07 May 2008) | 3 lines

Remove remnants of dlinkedlists.  I didn't actually use them in the final
version
of my IAX2 improvements.

........

................

................
r115515 | russell | 2008-05-07 11:38:36 -0600 (Wed, 07 May 2008) | 2 lines

re-add dlinkedlists.h to trunk, oops!

................
r115518 | russell | 2008-05-07 12:17:43 -0600 (Wed, 07 May 2008) | 12
lines

Blocked revisions 115517 via svnmerge

........
r115517 | russell | 2008-05-07 13:17:19 -0500 (Wed, 07 May 2008) | 5 lines

Track peer references when stored in the sip_pvt struct as the peer
related to
a qualify ping or a subscription.  This fixes some realtime related
crashes.
(closes issue http://bugs.digium.com/view.php?id=12588)
(closes issue http://bugs.digium.com/view.php?id=12555)

........

................
r115519 | russell | 2008-05-07 12:24:51 -0600 (Wed, 07 May 2008) | 2 lines

Let chan_h323 build in dev mode

................
r115521 | russell | 2008-05-07 12:30:12 -0600 (Wed, 07 May 2008) | 7 lines

Use the default that the log output claims will be used for the basedn

(closes issue http://bugs.digium.com/view.php?id=12599)
Reported by: suretec
Patches:
      12599.patch uploaded by juggie (license 24)

................
r115523 | russell | 2008-05-07 12:33:50 -0600 (Wed, 07 May 2008) | 6 lines

Only save a password if a username exists.

(closes issue http://bugs.digium.com/view.php?id=12600)
Reported By: suretec
Patch by me

................
r115525 | tilghman | 2008-05-07 12:40:21 -0600 (Wed, 07 May 2008) | 2
lines

Don't free the object on destroy, as astobj2 takes care of that for you

................
r115535 | tilghman | 2008-05-07 14:22:09 -0600 (Wed, 07 May 2008) | 2
lines

Advance to next sounds release

................
r115537 | russell | 2008-05-07 15:11:33 -0600 (Wed, 07 May 2008) | 10
lines

Fix up a problem that was introduced into the scheduler when it was
converted
to use doubly linked lists.  The schedule() function had an optimization
that
had it try to guess which direction would be better for the traversal to
insert
the task into the scheduler queue.  However, if the code chose the path
where
it traversed the queue in reverse, and the result was that the task should
be
at the head of the queue, then the code would actually put it at the tail,
instead.

(Problem found by bbryant, debugged and fixed by bbryant and me)

................

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=115885 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
05-13-08 13:31  svnbot         Checkin                                      
05-13-08 13:31  svnbot         Note Added: 0086804                          
======================================================================