Attachment #153191 for bug #197787

Lines 2-9 Link Here

(-)Makefile (-2 / +1 lines)
2	# $FreeBSD$	2	# $FreeBSD$
3		3
4	PORTNAME= ansible	4	PORTNAME= ansible
5	PORTVERSION= 1.8.2	5	PORTVERSION= 1.8.4
6	PORTREVISION= 3
7	CATEGORIES= sysutils python	6	CATEGORIES= sysutils python
8	MASTER_SITES= http://releases.ansible.com/ansible/	7	MASTER_SITES= http://releases.ansible.com/ansible/
9		8

Lines 1-2 Link Here

(-)distinfo (-2 / +2 lines)
1	SHA256 (ansible-1.8.2.tar.gz) = 2cd01ff4d67acb410bae0a9438956f6fc36d01babd956b0dc3b700620f2372b9	1	SHA256 (ansible-1.8.4.tar.gz) = d37c04b3abce9b036a6c8311fccb360c5cbc3ec895445f805243b0448d239ec1
2	SIZE (ansible-1.8.2.tar.gz) = 754435	2	SIZE (ansible-1.8.4.tar.gz) = 758916





$FreeBSD$

--- lib/ansible/modules/core/database/postgresql/postgresql_privs.py.orig
+++ lib/ansible/modules/core/database/postgresql/postgresql_privs.py
@@ -462,10 +462,13 @@
         if obj_type == 'group':
             set_what = ','.join(pg_quote_identifier(i, 'role') for i in obj_ids)
         else:
+            # function types are already quoted above
+            if obj_type != 'function':
+                obj_ids = [pg_quote_identifier(i, 'table') for i in obj_ids]
             # Note: obj_type has been checked against a set of string literals
             # and privs was escaped when it was parsed
             set_what = '%s ON %s %s' % (','.join(privs), obj_type,
-                                        ','.join(pg_quote_identifier(i, 'table') for i in obj_ids))
+                                        ','.join(obj_ids))
 
         # for_whom: SQL-fragment specifying for whom to set the above
         if roles == 'PUBLIC':

Return to bug 197787

Lines 1-20 Link Here

(-)files/patch-lib_ansible_modules_core_database_postgresql_postgresql__privs.py (-20 lines)
1
2	$FreeBSD$
3
4	--- lib/ansible/modules/core/database/postgresql/postgresql_privs.py.orig
5	+++ lib/ansible/modules/core/database/postgresql/postgresql_privs.py
6	@@ -462,10 +462,13 @@
7	if obj_type == 'group':
8	set_what = ','.join(pg_quote_identifier(i, 'role') for i in obj_ids)
9	else:
10	+ # function types are already quoted above
11	+ if obj_type != 'function':
12	+ obj_ids = [pg_quote_identifier(i, 'table') for i in obj_ids]
13	# Note: obj_type has been checked against a set of string literals
14	# and privs was escaped when it was parsed
15	set_what = '%s ON %s %s' % (','.join(privs), obj_type,
16	- ','.join(pg_quote_identifier(i, 'table') for i in obj_ids))
17	+ ','.join(obj_ids))
18
19	# for_whom: SQL-fragment specifying for whom to set the above
20	if roles == 'PUBLIC':