Ticket #2574 (closed Bugs: fixed)

Opened 4 years ago

Last modified 4 years ago

sip_additional.conf warnings with asterisk trunk/1.6

Reported by: pnlarsson Assigned to: pnlarsson
Priority: minor Milestone: 2.4
Component: Asterisk Configuration Version: 2.4-branch
Keywords: Cc:
Confirmation: Confirmed SVN Revision (if applicable):
Backend Engine: Asterisk 1.6 Backend Engine Version:

Description 

[Dec 19 22:16:57] WARNING[30127] frame.c: Cannot disallow unknown format ''
[Dec 19 22:16:57] WARNING[30127] chan_sip.c: Codec configuration errors found in line 17 : disallow =
[Dec 19 22:16:57] WARNING[30127] frame.c: Cannot allow unknown format ''
[Dec 19 22:16:57] WARNING[30127] chan_sip.c: Codec configuration errors found in line 23 : allow =

Attachments

sip_allow.diff (0.9 kB) - added by pnlarsson on 12/31/07 17:47:09.
allow.diff (2.3 kB) - added by pnlarsson on 01/06/08 13:20:42.
Doing it in PHP instead
allow2.diff (2.5 kB) - added by pnlarsson on 01/06/08 16:35:45.
In the switch statement instead - only >=1.4 are sending warnings about the empty allow

Change History

12/19/07 16:42:21 changed by p_lindheimer

  • confirmation changed from Unreviewed to Confirmed.
  • milestone changed from Cut Line to 2.4.

ok - they are just warnings, not errors. We can look at stripping out blank allow and disallow statements in core_conf class where these are generated, but since it's just a warning, I think I may ignore it. I'll leave in 2.4 milestone for now as a reminder.

12/31/07 17:47:09 changed by pnlarsson

  • attachment sip_allow.diff added.

12/31/07 17:50:57 changed by pnlarsson

A fix for this is attached - it's filtering out allow and disallow without data in the SQL. The question is if this is the way to go? Or should we do it later while we are going trough the result set?

01/06/08 13:20:42 changed by pnlarsson

  • attachment allow.diff added.

Doing it in PHP instead

01/06/08 16:35:45 changed by pnlarsson

  • attachment allow2.diff added.

In the switch statement instead - only >=1.4 are sending warnings about the empty allow

01/18/08 14:48:57 changed by pnlarsson

  • status changed from new to closed.
  • resolution set to fixed.

Fixed in r2574

01/18/08 14:50:13 changed by pnlarsson

Sorry r5605 is more correct