Don't uselessly escape a string that doesn't need escaping

Per gripe from Ian Barwick

Co-authored-by: Ian Barwick 
Discussion: https://postgr.es/m/CABvVfJWNnNKb8cHsTLhkTsvL1+G6BVcV+57+w1JZ61p8YGPdWQ@mail.gmail.com

diff --git a/src/bin/pg_basebackup/pg_basebackup.c b/src/bin/pg_basebackup/pg_basebackup.c
index 33c1d4efe3697b73a89d32b64341ed33ed57d636..19ebf6d1a57b07312c65a2ab5733880da96d7708 100644 (file)
--- a/src/bin/pg_basebackup/pg_basebackup.c
+++ b/src/bin/pg_basebackup/pg_basebackup.c
@@ -1650,9 +1650,9 @@ GenerateRecoveryConf(PGconn *conn)
 
    if (replication_slot)
    {
-       escaped = escape_quotes(replication_slot);
-       appendPQExpBuffer(recoveryconfcontents, "primary_slot_name = '%s'\n", replication_slot);
-       free(escaped);
+       /* unescaped: ReplicationSlotValidateName allows [a-z0-9_] only */
+       appendPQExpBuffer(recoveryconfcontents, "primary_slot_name = '%s'\n",
+                         replication_slot);
    }
 
    if (PQExpBufferBroken(recoveryconfcontents) ||