On Tue, Nov 22, 2016 at 5:55 PM, Ideriha, Takeshi
<ideriha.take...@jp.fujitsu.com> wrote:
> Here's a summary for what I tested  in RHEL7.0, details follow.

Thanks for the review.

> [Summary]
> 1. apply patch and make world
>   -> failed because </para> was mistakenly coded <para>.
>
> 2.correct this mistake and make check-world
>   -> got 1 failed test: "'pg_dumpall with \n\r in database name'"
>      because test script cannot createdb "foo\n\rbar"

The attached version addresses those problems. I have replaced the
test in src/bin/pg_dump/t/ by tests in src/bin/scripts/t/ to check if
the role name and database name with CR or LF fail to be created. I
have as well added a test for initdb when the data directory has an
incorrect character in 0002.
-- 
Michael

Attachment: 0002-Ensure-clean-up-of-data-directory-even-with-restrict.patch
Description: binary/octet-stream

Attachment: 0001-Forbid-newline-and-carriage-return-characters-in-dat.patch
Description: binary/octet-stream

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to