[Asterisk-video] Another crashed when callout

[Sergio Garcia]

I've added more length checkings to the code, test new version.

Thanxs
Sergio


---------- Original Message ----------------------------------
From: Reza Fatahillah <ezhot_95 at yahoo.com>
Reply-To: Development discussion of video media support in Asterisk<asterisk-video at lists.digium.com>
Date:  Thu, 4 Oct 2007 22:39:03 -0700 (PDT)

>Hi Sergio,
>
>I have changed as written.
>Then i got another crash. This time i just made 4
>calls.
>
>Here are another bt full:
>------------------------
>#0  0x00ab7481 in create_ast_frame (frame=0x9124930,
>vt=0xb7826740) at app_h324m.c:187
>        header = 15 '\017'
>        mode = 15 '\017'
>        stuf = 4
>        bs = 4294967295
>        mark = 0
>        i = 4294967265
>        found = 0
>        len = 153014336
>        send = (struct ast_frame *) 0x9121dd0
>        framedata = (unsigned char *) 0x9225480 "\017"
>        framelength = 1
>        __PRETTY_FUNCTION__ = "create_ast_frame"
>#1  0x00ab8a7d in app_h324m_call (chan=0x8e706f0,
>data=0xb782af60) at app_h324m.c:995
>        f = (struct ast_frame *) 0x8b536b4
>        send = (struct ast_frame *) 0x9200e90
>        u = (struct ast_module_user *) 0x8f0df38
>        pak = {framedata = 0x91d83c8
>"<<§ò\221\225þo\206!x\033êÅy\221Ì5¡]_\016ÆïfÙ\220·0\023Pè|",
>offset = 0x91d83e9 "", framelength = 33, num = 1, max
>= 1}
>        vt = {tv = {tv_sec = 0, tv_usec = 0}, tvnext =
>{tv_sec = 0, tv_usec = 0}, samples = 0, first = 1
>'\001',
>  buffer =
>"é\024\b¤\021±\b\000\000\000\000\000\000\000\000\000\000s
>\000\000\000\000ÿÿÿÿ", '\0' <repeats 32 times>,
>"\001\000\000\000\000\000\000\000½\204\025\b\002\00030\000\000\000\000peB\000ô\037N\000\fh\202·Øg\202·}vB\000\fh\202·$\210\034\tÕ\210\034\t\000\000\000\000Õ\210\034\tô\037N\000²\000\000\000peB\000ô\037N\000<h\202·\bh\202·}vB\000<h\202·$\210\034\tÕ\210\034\t\000\000\000\000Õ\210\034\tô\037N\000²\000\000\000$\210\034\t(i\202·y'B\000<h\202·ô\037N\000²\000\000\000$\210\034\t(i\202·\215'B\000<h\202·<r\024\b\bm\202"...,
>bufferLength = 0}
>        frame = (void *) 0x9124930
>        input = 0x0
>        reason = 0
>        ms = -1
>        channels = {0x8e706f0, 0x8e710f8}
>        pseudo = (struct ast_channel *) 0x8e710f8
>        where = (struct ast_channel *) 0x8e710f8
>        __PRETTY_FUNCTION__ = "app_h324m_call"
>        id = (void *) 0x91ed040
>#2  0x080c39af in pbx_exec (c=0x8e706f0,
>app=0x8b8f900, data=0xb782af60) at pbx.c:532
>        res = -1216188944
>        saved_c_appl = 0x0
>        saved_c_data = 0x0
>#3  0x080c744d in pbx_extension_helper (c=0x8e706f0,
>con=0x0, context=0x8e70918 "sipout", exten=0x8e70968
>"08039744253", priority=2, label=0x0,
>    callerid=0x8f6a720 "1005", action=E_SPAWN) at
>pbx.c:1833
>        e = (struct ast_exten *) 0x8b4f9a8
>        app = (struct ast_app *) 0x8b8f900
>        res = 8
>        q = {incstack = {0x0 <repeats 128 times>},
>stacklen = 0, status = 5, swo = 0x0, data = 0x0,
>foundcontext = 0x8e70918 "sipout"}
>        passdata = "_X. at threegvideo", '\0' <repeats
>8176 times>
>        matching_action = 0
>        __PRETTY_FUNCTION__ = "pbx_extension_helper"
>#4  0x080c8819 in ast_spawn_extension (c=0x8e706f0,
>context=0x8e70918 "sipout", exten=0x8e70968
>"08039744253", priority=2, callerid=0x8f6a720 "1005")
>    at pbx.c:2288
>
>--- Sergio Garcia Murillo
><sergio.garcia at fontventa.com> wrote:
>
>> I think I've found the problem, but don't have a
>> server rigth now available
>> for compiling and commiting it (mine crashed a week
>> ago and still didn't got
>> time to buy new one).
>> Could you test to change lines 133,135
>> 
>> int i=0
>> int len;
>> 
>> for:
>> 
>> unsigned int i=0
>> unsigned int len;
>> 
>> and append before line 215
>> 
>> if(framelength>2)
>> 
>> 
>> BR
>> Sergio
>
>
>
>       
>____________________________________________________________________________________
>Moody friends. Drama queens. Your life? Nope! - their life, your story. Play Sims Stories at Yahoo! Games.
>http://sims.yahoo.com/  
>
>_______________________________________________
>--Bandwidth and Colocation Provided by http://www.api-digital.com--
>
>asterisk-video mailing list
>To UNSUBSCRIBE or update options visit:
>   http://lists.digium.com/mailman/listinfo/asterisk-video
>