[Home]

Summary:ASTERISK-17878: [patch] Generating output using cdr_custom backend is corrupt because of pbx.c
Reporter:xyese (xyese)Labels:
Date Opened:2011-05-18 07:22:22Date Closed:2011-07-08 15:18:52
Priority:MinorRegression?
Status:Closed/CompleteComponents:Core/PBX
Versions:1.8.4 Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) pbx.c.diff.txt
Description:Then use cdr_custom with use this fields in cdr_custom.conf:
simple.csv => ${CDR(answer)},${CDR(src)},${CDR(dst)},${CDR(billsec)},${SHELL(echo -n test)}

Used dialplan sample:
exten => 100,1,Answer()
exten => 100,n,Echo()
exten => 200,1,Wait(100)
And make calls: 100 (answered call) and 200 (unanswered call)
Then in sample.csv I have this output:
4400,100,2011-05-18 08:49:22,3,3test
4400,200,4492,0,0test
Third and fifth field are corrupted. Fifth field append result to previous value. Third field copy previous value if result is null.
This is because in pbx.c string variable are not reseted in all cases.
With my patch output get corrected:
4400,100,2011-05-18 09:06:15,10,test
4400,200,,0,test

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

<inline patch removed by lmadsen>
Comments:By: xyese (xyese) 2011-05-18 07:27:56

I have mistaken in type config of cdr_custom.csv. Right config was:
simple.csv => ${CDR(src)},${CDR(dst)},${CDR(answer)},${CDR(billsec)},${SHELL(echo -n test)}


By: Leif Madsen (lmadsen) 2011-05-19 14:29:56

You must sign the license agreement prior to uploading patches. Thanks!
By: Matthew Nicholson (mnicholson) 2011-07-08 09:49:27.215-0500

This patch doesn't appear to have a license agreement associated with it. Please agree to the license agreement and upload a new patch.
By: David Woolley (davidw) 2011-07-12 06:04:52.373-0500

This is marked closed/fixed but appears to actually be in a feedback state!
By: Matthew Nicholson (mnicholson) 2011-07-12 08:04:33.136-0500

A fix has been committed.