]> granicus.if.org Git - postgresql/commit
> Uh, isn't the correct fix
authorBruce Momjian <bruce@momjian.us>
Mon, 15 Oct 2001 18:16:31 +0000 (18:16 +0000)
committerBruce Momjian <bruce@momjian.us>
Mon, 15 Oct 2001 18:16:31 +0000 (18:16 +0000)
commit394925b60dad238c3eb5f9b21b282b3d116bec62
treecb84295e03c019ff5b0b61f30b1c749fc6883d18
parentb261cb9fa19c44bf9c1688df05104aa6e8d94108
 > Uh, isn't the correct fix
 > ! $$ = cat_str(8, make_str("grant"), $2, make_str("on"), $4, $5,
 > make_str("to"), $7, $8);
 > ISTM your patch loses the opt_with_grant clause.  (Of course the
 > backend doesn't currently accept that clause anyway, but that's no
 > reason for ecpg to drop it.)

My patch doesn't loose the option, it's never been passed on anyway:

 opt_with_grant:  WITH GRANT OPTION
                                {
                                        mmerror(ET_ERROR, "WITH GRANT OPTION is not supported.  Only relation owners can
 set privileges");
                                 }
                | /*EMPTY*/
                ;

The existing code in ecpg/preproc/preproc.y to handle the WITH option
simply throws an error and aborts the processing... The patch below
prevents the segfault and also passes on the WITH option to the
backend, probably a better fix.

Lee Kindness
src/interfaces/ecpg/preproc/preproc.y