diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2022-02-23 11:10:46 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2022-02-23 11:10:46 -0500 |
| commit | bd74c4037c4ee268db46e983bcc0f1e0a9f7ab72 (patch) | |
| tree | c203ad19d9e9751bbf6fc91ed37662e2bc9714d0 /src | |
| parent | 0475a97f744d2fea3676b2e69405d20358eac07a (diff) | |
| download | postgresql-bd74c4037c4ee268db46e983bcc0f1e0a9f7ab72.tar.gz postgresql-bd74c4037c4ee268db46e983bcc0f1e0a9f7ab72.zip | |
Re-allow underscore as first character of custom GUC names.
Commit 3db826bd5 intended that valid_custom_variable_name's
rules for valid identifiers match those of scan.l. However,
I (tgl) had some kind of brain fade and put "_" in the wrong
list.
Fix by Japin Li, per bug #17415 from Daniel Polski.
Discussion: https://postgr.es/m/17415-ebdb683d7e09a51c@postgresql.org
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/utils/misc/guc.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/backend/utils/misc/guc.c b/src/backend/utils/misc/guc.c index e4afd07bfe1..bf7ec0d4666 100644 --- a/src/backend/utils/misc/guc.c +++ b/src/backend/utils/misc/guc.c @@ -5474,13 +5474,13 @@ valid_custom_variable_name(const char *name) name_start = true; } else if (strchr("ABCDEFGHIJKLMNOPQRSTUVWXYZ" - "abcdefghijklmnopqrstuvwxyz", *p) != NULL || + "abcdefghijklmnopqrstuvwxyz_", *p) != NULL || IS_HIGHBIT_SET(*p)) { /* okay as first or non-first character */ name_start = false; } - else if (!name_start && strchr("0123456789_$", *p) != NULL) + else if (!name_start && strchr("0123456789$", *p) != NULL) /* okay as non-first character */ ; else return false; |