protect data read from pg_pathman's tables

funbringer · funbringer · commit 79e11d94a147 · 2018-06-21T17:43:39.000+03:00
diff --git a/src/init.c b/src/init.c
@@ -664,6 +664,7 @@ pathman_config_contains_relation(Oid relid, Datum *values, bool *isnull,
 		/* Extract data if necessary */
 		if (values && isnull)
 		{
+			htup = heap_copytuple(htup);
 			heap_deform_tuple(htup, RelationGetDescr(rel), values, isnull);
 
 			/* Perform checks for non-NULL columns */
@@ -778,6 +779,7 @@ read_pathman_params(Oid relid, Datum *values, bool *isnull)
 	if ((htup = heap_getnext(scan, ForwardScanDirection)) != NULL)
 	{
 		/* Extract data if necessary */
+		htup = heap_copytuple(htup);
 		heap_deform_tuple(htup, RelationGetDescr(rel), values, isnull);
 		row_found = true;
 

Original file line number	Diff line number	Diff line change
`@@ -664,6 +664,7 @@ pathman_config_contains_relation(Oid relid, Datum values, bool isnull,`
`664`	`664`	`/* Extract data if necessary */`
`665`	`665`	`if (values && isnull)`
`666`	`666`	`{`
	`667`	`+ htup = heap_copytuple(htup);`
`667`	`668`	`heap_deform_tuple(htup, RelationGetDescr(rel), values, isnull);`
`668`	`669`
`669`	`670`	`/* Perform checks for non-NULL columns */`
`@@ -778,6 +779,7 @@ read_pathman_params(Oid relid, Datum values, bool isnull)`
`778`	`779`	`if ((htup = heap_getnext(scan, ForwardScanDirection)) != NULL)`
`779`	`780`	`{`
`780`	`781`	`/* Extract data if necessary */`
	`782`	`+ htup = heap_copytuple(htup);`
`781`	`783`	`heap_deform_tuple(htup, RelationGetDescr(rel), values, isnull);`
`782`	`784`	`row_found = true;`
`783`	`785`