[Freeswitch-users] Memory leak in mod_nibblebill or in ODBC core?

[Sergey Okhapkin]

It's already on jira - FSMOD-48. Valgrind with --leak-check=full finds 
nothing.

Memory kept growing and growing, I had to restart FS when RSS reached 500M to 
avoid swap.

On Monday 10 May 2010, Anthony Minessale wrote:
> Why does this thread continue on the list instead of being opened in JIRA
> Can you see how this issue can be forgotten because it's on the mailing
>  list that gets 200 emails a day?
> When you show the code in the body of an email like that, then I have to
> spend a minuted wondering where in the code you were talking about because
> it's not in the form of a unified diff.
> 
> I appreciate that you do not normally have to deal with organized software
> management but I must ask that you please spend a little time to learn our
> customs and use JIRA and patches to communicate bug reports.  I am glad you
> are helping to find problems but we really need to stay organized to make
>  it forward.
> 
> I redid the function in latest GIT.  It did look like it was doing a few
> things wrong but they probably would have surfaced in valgrind if you were
> running it in full leak check mode.  When you say it goes to 300M, does it
> keep going from there because there is a bit of memory that will be pooled
> as you start to use more advanced features.  Did you ever watch it to see
> how high it will go?
> 
> On Sun, May 9, 2010 at 4:01 PM, Sergey Okhapkin 
<sos at sokhapkin.dyndns.org>wrote:
> > Shouldn't label "end:" be BEFORE 
> > switch_odbc_statement_handle_free(&stmt); ?
> > I think it should...
> >
> >
> > static switch_status_t bill_event(
> > ....
> >
> >       if (switch_odbc_handle_exec(globals.master_odbc, sql, &stmt, NULL)
> > != SWITCH_ODBC_SUCCESS) {
> >                char *err_str;
> >                err_str =
> > switch_odbc_handle_get_error(globals.master_odbc, stmt);
> >                switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_ERROR,
> > "ERR:
> > [%s]\n[%s]\n", sql, switch_str_nil(err_str));
> >                switch_safe_free(err_str);
> >        } else {
> >                /* TODO: Failover to a flat/text file if DB is unavailable
> > */
> >
> >                goto end;
> >        }
> >
> >        switch_odbc_statement_handle_free(&stmt);
> >
> > end:
> >
> > On Sunday 09 May 2010, Brian West wrote:
> > > What are you calling significant memory usage?
> > >
> > > /b
> > >
> > > On May 9, 2010, at 2:10 PM, Sergey Okhapkin wrote:
> > > > Valgrind output shows no significant leaks. This was discussed
> > > > already
> >
> > in
> >
> > > > this thread a month ago.
> > > >
> > > > On Sunday 09 May 2010, Brian West wrote:
> > > >> Please use valgrind and see where its leaking then open a jira.
> > > >>
> > > >> Nobody has been able to reproduce this in a lab nor provide any
> >
> > details
> >
> > > >> to assist in finding the issue... All I have seen is people
> >
> > complaining
> >
> > > >> about it and not doing what they should debugging the issue and
> > > >> reporting it.
> > > >>
> > > >> /b
> > >
> > > _______________________________________________
> > > FreeSWITCH-users mailing list
> > > FreeSWITCH-users at lists.freeswitch.org
> > > http://lists.freeswitch.org/mailman/listinfo/freeswitch-users
> > > UNSUBSCRIBE:http://lists.freeswitch.org/mailman/options/freeswitch-user
> > >s http://www.freeswitch.org
> >
> > _______________________________________________
> > FreeSWITCH-users mailing list
> > FreeSWITCH-users at lists.freeswitch.org
> > http://lists.freeswitch.org/mailman/listinfo/freeswitch-users
> > UNSUBSCRIBE:http://lists.freeswitch.org/mailman/options/freeswitch-users
> > http://www.freeswitch.org
>