Summary: | ASTERISK-02117: [patch] Fail to get redirecting number | ||
Reporter: | rluque (rluque) | Labels: | |
Date Opened: | 2004-07-27 02:04:24 | Date Closed: | 2004-09-25 02:09:42 |
Priority: | Major | Regression? | No |
Status: | Closed/Complete | Components: | Core/General |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
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. ****** ADDITIONAL INFORMATION ****** 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! /O 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. Thanks, Brian 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 |