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

Поиск
Список
Период
Сортировка
От Bruce Momjian
Тема Re: Possible bug in plpgsql/src/gram.y
Дата
Msg-id 200107120119.f6C1Jxt11746@candle.pha.pa.us
обсуждение исходный текст
Ответ на Possible bug in plpgsql/src/gram.y  (Ian Lance Taylor <ian@airs.com>)
Ответы Re: Possible bug in plpgsql/src/gram.y  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
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
 


В списке pgsql-hackers по дате отправления:

Предыдущее
От: Bruce Momjian
Дата:
Сообщение: Re: Possible bug in plpgsql/src/gram.y
Следующее
От: "Mikheev, Vadim"
Дата:
Сообщение: RE: Strangeness in xid allocation / snapshot setup