Summary:ASTERISK-03404: segmentaion fault in app_meetme.c
Reporter:constfilin (constfilin)Labels:
Date Opened:2005-02-01 00:20:36.000-0600Date Closed:2008-01-15 15:23:57.000-0600
Versions:Frequency of
Environment:Attachments:( 0) app_meetme.diff
Description:The current app_meetme.c goes only halfway to try working when a pseudo
zap channel can't be created. In this case "build_conf" initializes
conf->fd and leave conf->chan equal NULL.

Then in conf_run annoucement about user joining or leaving the conference
are still played on conf->chan, which is NULL. This creates segmentation

The attached patch checks for conf->chan being not NULL before passing it
to ast_streamfile().

This is not a full solution to haveing meetme work when Zap pseudo channel
is not available but is it better than having a segmentation fault.

Would be great to have this in CVS.


I ran cvs diff on the leatest version of app_meetme.c (1.75)
Comments:By: Mark Spencer (markster) 2005-02-01 00:47:52.000-0600

Fixed in CVS head.  Next time please use cvs diff -u format, and if you remember to mention your disclaimer you'll get an extra +1 of karma!  Thanks!

By: Russell Bryant (russell) 2005-02-06 21:16:38.000-0600

feature not in 1.0

By: Digium Subversion (svnbot) 2008-01-15 15:23:57.000-0600

Repository: asterisk
Revision: 4942

U   trunk/apps/app_meetme.c

r4942 | markster | 2008-01-15 15:23:56 -0600 (Tue, 15 Jan 2008) | 2 lines

Fix potential segfault (bug ASTERISK-3404)