Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 4 Feb 2015 19:24:27 +0000 (UTC)
From:      Jimmy Olgeni <olgeni@FreeBSD.org>
To:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   svn commit: r378411 - in head/sysutils/ansible: . files
Message-ID:  <201502041924.t14JORqf010051@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: olgeni
Date: Wed Feb  4 19:24:26 2015
New Revision: 378411
URL: https://svnweb.freebsd.org/changeset/ports/378411
QAT: https://qat.redports.org/buildarchive/r378411/

Log:
  Fix quoting of function identifiers in postgresql_privs (issue 703
  in ansible/ansible-modules-core on GitHub).
  
  PR:		197325
  Submitted by:	olgeni
  Approved by:	maintainer

Added:
  head/sysutils/ansible/files/patch-lib_ansible_modules_core_database_postgresql_postgresql__privs.py   (contents, props changed)
Modified:
  head/sysutils/ansible/Makefile

Modified: head/sysutils/ansible/Makefile
==============================================================================
--- head/sysutils/ansible/Makefile	Wed Feb  4 19:15:22 2015	(r378410)
+++ head/sysutils/ansible/Makefile	Wed Feb  4 19:24:26 2015	(r378411)
@@ -3,7 +3,7 @@
 
 PORTNAME=	ansible
 PORTVERSION=	1.8.2
-PORTREVISION=	1
+PORTREVISION=	2
 CATEGORIES=	sysutils python
 MASTER_SITES=	http://releases.ansible.com/ansible/
 

Added: head/sysutils/ansible/files/patch-lib_ansible_modules_core_database_postgresql_postgresql__privs.py
==============================================================================
--- /dev/null	00:00:00 1970	(empty, because file is newly added)
+++ head/sysutils/ansible/files/patch-lib_ansible_modules_core_database_postgresql_postgresql__privs.py	Wed Feb  4 19:24:26 2015	(r378411)
@@ -0,0 +1,20 @@
+
+$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':



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201502041924.t14JORqf010051>