Asterisk 1.2.13 Stable

[Tzafrir Cohen]

On Mon, May 28, 2007 at 07:07:08AM +0200, Alain Degreffe wrote:
> Hi,
> 
>  
> 
> As I can see, the version used in the last stable have a bug very blocking
> for me and all people using the deprecated chan_modem.
> 
> The upstream is fixed : see bug 0009575 on asterisk.org. Without this fix,
> if I uncomment the Makefile under channels to enable chan_modem, the result
> is that when chan_modem is used, asterisk freeze. This bug is just a thread
> problem and just need to comment or suppress one line in the source code. 

Bug report:

  http://bugs.digium.com/view.php?id=9575

It does not include a patch, bt the fix seems to be:

  "removed pthread_join in restart_monitor() to make it like chan_zap 
  and other channel drivers"

  http://svn.digium.com/view/asterisk/branches/1.2/channels/chan_modem.c?r1=38904&r2=61776&makepatch=1&diff_format=h

A later patch to the same place (and to chan_zap.c) removed that '#if 0' 
block. But apart from that, there have been no later changes in 
chan_modem.c in 1.2.

In 1.4.... there is no chan_modem.c at all. chan_modem has been deprecated
and seem to have recieved very little attention.


As the channel is not built by default, you'd need to patch the source 
and rebuild the package anyway. So adding an extra patch shouldn't be
such big a deal. 

(OTOH, I really don't see how this breaks anything)

-- 
               Tzafrir Cohen       
icq#16849755                    jabber:tzafrir at jabber.org
+972-50-7952406           mailto:tzafrir.cohen at xorcom.com       
http://www.xorcom.com  iax:guest at local.xorcom.com/tzafrir