[Home]

Summary:ASTERISK-17186: [patch] Up to 6 [DEBUG] messages forced to console on every call
Reporter:Kirill Katsnelson (kkm)Labels:
Date Opened:2010-12-29 23:30:35.000-0600Date Closed:2011-02-04 10:55:43.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Channels/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 018556-kkm-tame-feral-debugs-r300039.diff
Description:These do not obey `core set debug N'

[2010-12-29 20:34:33.122] DEBUG[29223]: channel.c:6045 ast_set_owners_and_peers: setting peeraccount to XxxxxTest for SIP/dyn-XxxxxxTest-2314-00000005 from data on channel SIP/xxxxx-in-00000004
[2010-12-29 20:34:33.123] DEBUG[29223]: channel.c:6050 ast_set_owners_and_peers: setting peeraccount to XxxxxxTest for SIP/xxxxx-in-00000004 from data on channel SIP/dyn-XxxxxxTest-2314-00000005

And printed on every call.

First found in 1.8.1.

Patch follows.

****** STEPS TO REPRODUCE ******

Will print 0 to 6, depending on what CDR fields you populate.

****** ADDITIONAL INFORMATION ******

There are about 200 locations in the codebase where ast_log(LOG_DEBUG is used instead of ast_dbg(). The former does not obey to the CLI debug switch command, and the latter does.

The patch fixes only those introduced between 1.6 and 1.8 and flooding the console on every call.
Comments:By: Kirill Katsnelson (kkm) 2011-01-05 04:28:07.000-0600

Xref https://reviewboard.asterisk.org/r/1071/
By: Digium Subversion (svnbot) 2011-02-04 10:55:42.000-0600

Repository: asterisk
Revision: 306258

U   trunk/addons/chan_ooh323.c
U   trunk/apps/app_fax.c
U   trunk/apps/app_meetme.c
U   trunk/apps/app_rpt.c
U   trunk/apps/app_voicemail.c
U   trunk/cel/cel_radius.c
U   trunk/channels/chan_dahdi.c
U   trunk/channels/chan_gtalk.c
U   trunk/channels/chan_iax2.c
U   trunk/channels/chan_misdn.c
U   trunk/channels/chan_sip.c
U   trunk/channels/chan_skinny.c
U   trunk/channels/sig_pri.c
U   trunk/codecs/codec_ilbc.c
U   trunk/include/asterisk/channel.h
U   trunk/include/asterisk/sched.h
U   trunk/main/app.c
U   trunk/main/audiohook.c
U   trunk/main/channel.c
U   trunk/main/features.c
U   trunk/main/http.c
U   trunk/main/logger.c
U   trunk/main/manager.c
U   trunk/main/pbx.c
U   trunk/main/say.c
U   trunk/main/taskprocessor.c
U   trunk/main/xmldoc.c
U   trunk/pbx/pbx_loopback.c
U   trunk/pbx/pbx_spool.c
U   trunk/res/ais/evt.c
U   trunk/res/res_agi.c
U   trunk/res/res_config_ldap.c
U   trunk/res/res_http_post.c
U   trunk/res/res_jabber.c
U   trunk/res/res_musiconhold.c
U   trunk/res/res_odbc.c
U   trunk/res/res_rtp_asterisk.c
U   trunk/res/res_smdi.c
U   trunk/utils/extconf.c

------------------------------------------------------------------------
r306258 | pabelanger | 2011-02-04 10:55:41 -0600 (Fri, 04 Feb 2011) | 7 lines

Replace ast_log(LOG_DEBUG, ...) with ast_debug()

(closes issue ASTERISK-17186)
Reported by: kkm

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

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

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