Summary:ASTERISK-02117: [patch] Fail to get redirecting number
Reporter:rluque (rluque)Labels:
Date Opened:2004-07-27 02:04:24Date Closed:2004-09-25 02:09:42
Versions:Frequency of
Environment:Attachments:( 0) q931.c.diff
( 1) q931.c.patch
( 2) q931.c.patch
Description:When you extract redirecting number from ISDN Information Element, sometimes fails if previous fields are not present.


In order to solve this bug, you should check every field before redirecting number field.
Comments:By: Olle Johansson (oej) 2004-07-27 04:31:14

* Always attach patch in a .txt file, not within the bug report
* When coding, follow the coding guidelines (doc/CODING-GUIDELINES in your
 source directory
* Please follow report guidelines on http://www.digium.com/bugtracker.html

We are missing the following info:
-- System you are using?
-- Version of asterisk, zaptel, libpri

edited on: 07-27-04 04:15

By: Olle Johansson (oej) 2004-07-27 04:32:32

Also, please confirm that you've sent in a disclaimer accoring to the guidelines. Thank you for contributing to Asterisk!


By: rluque (rluque) 2004-07-28 03:21:13

I'm sorry, I will follow your advice next time:

System configuration:
Dell PowerEdge 700 P4-3.2, 1Gb, 80Gb SATA.
Debian Sarge, kernel 2.6.5
Telephony hardware Junghanns QuadBRI ISDN card.
asterisk, libpri, zaptel: lastest CVS (2004/07/23)
q931.c : CVS v. 1.83

By: Olle Johansson (oej) 2004-08-08 15:34:22

Does this patch still work with current CVS head?

By: Olle Johansson (oej) 2004-08-08 15:35:09

Dislaimer sent?

By: Olle Johansson (oej) 2004-08-14 13:14:15

We need an update on the disclaimer. Without a disclaimer, the patch will not move forward. if you have any questions, find a bug marshal on the irc channel.

By: Brian West (bkw918) 2004-08-22 23:08:42

Can we get an update here?  We are pushing for 1.0 we need faster turn around for bugs reported.


By: Paul Cadach (pcadach) 2004-08-23 22:19:19

I have a disclaimer, patch is updated to follow my "strategy" to be compliant with Q.931 clause 4.5.1.

By: Brian West (bkw918) 2004-08-24 10:54:21

YAY :) Thanks

By: Paul Cadach (pcadach) 2004-08-24 12:34:20

Sorry, forget close paren after switch() statement... Could it be added manually to not re-generate diff-file?

By: Mark Spencer (markster) 2004-08-25 01:21:04

Fixed in CVS