On Wed, Dec 01, 2004 at 09:41:49PM -0700, Michael Fuhr wrote:
> The warnings are coming from these two lines (around lines 448 and 449):
>
> free(comma_oids);
> free(comma_tables);
>
> Apparently the values of comma_oids and comma_tables are bogus.
> I haven't looked more closely yet to see why.
main() initializes my_opts->oids->num and my_opts->tables->num to 0.
sql_exec_searchtables() later sets comma_oids and comma_tables with
the following code:
comma_oids = get_comma_elts(opts->oids);
comma_tables = get_comma_elts(opts->tables);
get_comma_elts() starts with the following:
if (eary->num == 0)
return "";
sql_exec_searchtables() later tries to free the pointers to "".
Program fall down go boom.
--
Michael Fuhr
http://www.fuhr.org/~mfuhr/
