[Home]

Summary:ASTERISK-02715: [patch] Little code cleanups for current CVS-HEAD
Reporter:Paul Cadach (pcadach)Labels:
Date Opened:2004-10-31 09:47:00.000-0600Date Closed:2008-01-15 15:12:11.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) codeclean.diff
Description:This patch is directed to:
1) remove compilation warnings on astman;
2) keep correctness linked list of channels at chan_zap (channels isn't removed from double-linked list when channel is destroyed);
3) possible memory overwrite condition at pbx_wilcalu (zeroing memory indexed  by read() result which could be negative);
4) missed mutex unlock at res_odbc.
Comments:By: Paul Cadach (pcadach) 2004-10-31 09:52:27.000-0600

Disclaimer is on the file.

By: Mark Spencer (markster) 2004-10-31 18:14:58.000-0600

Fixed in CVS except for the "astman" patch because that *introduces* the compile warnings in modern newt libraries.

By: Digium Subversion (svnbot) 2008-01-15 15:12:11.000-0600

Repository: asterisk
Revision: 4131

U   trunk/channels/chan_zap.c
U   trunk/pbx/pbx_wilcalu.c
U   trunk/res/res_odbc.c

------------------------------------------------------------------------
r4131 | markster | 2008-01-15 15:12:11 -0600 (Tue, 15 Jan 2008) | 2 lines

Misc code fixes (bug ASTERISK-2715)

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

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