Re: Possible bug in plpgsql/src/gram.y

Also, can someone tell my why we use malloc in plpgsql?


> In this bit of code in src/pl/plpgsql/src/gram.y in the current CVS
> sources, curname_def is defined as PLpgSQL_expr * but it is is
> allocated the space required for a PLpgSQL_var.  This looks like a
> bug.
> 
> Ian
> 
>                 | decl_varname K_CURSOR decl_cursor_args decl_is_from K_SELECT decl_cursor_query
>                     {
>                         PLpgSQL_var *new;
>                         PLpgSQL_expr *curname_def;
>                         char        buf[1024];
>                         char        *cp1;
>                         char        *cp2;
> 
>                         plpgsql_ns_pop();
> 
>                         new = malloc(sizeof(PLpgSQL_var));
>                         memset(new, 0, sizeof(PLpgSQL_var));
> 
>                         curname_def = malloc(sizeof(PLpgSQL_var));
>                         memset(curname_def, 0, sizeof(PLpgSQL_var));
> 
> ---------------------------(end of broadcast)---------------------------
> TIP 3: if posting/reading through Usenet, please send an appropriate
> subscribe-nomail command to majordomo@postgresql.org so that your
> message can get through to the mailing list cleanly
> 

--  Bruce Momjian                        |  http://candle.pha.pa.us pgman@candle.pha.pa.us               |  (610)
853-3000+  If your life is a hard drive,     |  830 Blythe Avenue +  Christ can be your backup.        |  Drexel Hill,
Pennsylvania19026