Re: Forbid use of LF and CR characters in database and role names - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: Forbid use of LF and CR characters in database and role names
Date
Msg-id CAB7nPqRi4w-q-8ECxSTv3RzpAnJ_2u=kb14O0WdXDTALmpAcJg@mail.gmail.com
Whole thread Raw
In response to Re: Forbid use of LF and CR characters in database and role names  ("Ideriha, Takeshi" <ideriha.takeshi@jp.fujitsu.com>)
Responses Re: Forbid use of LF and CR characters in database and role names  (Michael Paquier <michael.paquier@gmail.com>)
List pgsql-hackers
On Fri, Nov 25, 2016 at 4:02 PM, Ideriha, Takeshi
<ideriha.takeshi@jp.fujitsu.com> wrote:
> I applied your fixed patch and new one, and confirmed the applied source passed the tests successfully. And I also
checkedmanually the error messages were emitted successfully when cr/lf are included in dbname or rolename or
data_directory.
>
> AFAICT, this patch satisfies the concept discussed before. So I’ve switched this patch “Ready for Committer”.

Thanks for the review, Ideriha-san. (See you next week perhaps?)
--
Michael



pgsql-hackers by date:

Previous
From: Tomas Vondra
Date:
Subject: confusing checkpoint_flush_after / bgwriter_flush_after
Next
From: Amos Bird
Date:
Subject: make default TABLESPACE belong to target table.