]> granicus.if.org Git - postgresql/commit
In AlterRole, make bypassrls an int
authorStephen Frost <sfrost@snowman.net>
Fri, 21 Aug 2015 12:22:22 +0000 (08:22 -0400)
committerStephen Frost <sfrost@snowman.net>
Fri, 21 Aug 2015 12:22:22 +0000 (08:22 -0400)
commit7ec8296e70f0f03cbdb3e0eb4f05ad5be0f810c8
tree04e60c9750122b4a7ef96a5e39b8ad0e8e858fdc
parent90a1d0aa76a7a95ff36f72cadd0d7aa0c3cb9b31
In AlterRole, make bypassrls an int

When reworking bypassrls in AlterRole to operate the same way the other
attribute handling is done, I missed that the variable was incorrectly a
bool rather than an int.  This meant that on platforms with an unsigned
char, we could end up with incorrect behavior during ALTER ROLE.

Pointed out by Andres thanks to tests he did changing our bool to be the
one from stdbool.h which showed this and a number of other issues.

Add regression tests to test CREATE/ALTER role for the various role
attributes.  Arrange to leave roles behind for testing pg_dumpall, but
none which have the LOGIN attribute.

Back-patch to 9.5 where the AlterRole bug exists.
src/backend/commands/user.c
src/test/regress/expected/roleattributes.out [new file with mode: 0644]
src/test/regress/parallel_schedule
src/test/regress/serial_schedule
src/test/regress/sql/roleattributes.sql [new file with mode: 0644]