RepoMirrors/haproxy
1
0
Fork 0
mirror of http://git.haproxy.org/git/haproxy.git/ synced 2024-12-20 10:40:13 +00:00
Code Issues Packages Projects Releases Wiki Activity

BUG/MINOR: tools: also accept '+' as a valid character in an identifier

Browse Source 
The function is_idchar() was added by commit 36f586b ("MINOR: tools:
add is_idchar() to tell if a char may belong to an identifier") to
ease matching of sample fetch/converter names. But it lacked support
for the '+' character used in "base32+src" and "url32+src". A quick
way to figure the list of supported sample fetch+converter names is
to issue the following command:

   git grep '"[^"]*",.*SMP_T_.*SMP_USE_'|cut -f2 -d'"'|sort -u

No more entry is reported once searching for characters not covered
by is_idchar().

No backport is needed.
This commit is contained in:
Willy Tarreau 2020-02-17 06:34:11 +01:00
parent 07e1e3c93e
commit a71667c07d
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
include/common/standard.h
View File

@ -373,13 +373,13 @@ extern const char *invalid_domainchar(const char *name);
extern const char *invalid_prefix_char(const char *name);
/* returns true if <c> is an identifier character, that is, a digit, a letter,
* or '-', '_', ':' or '.'. This is usable for proxy names, server names, ACL
* names, sample fetch names, and converter names.
* or '-', '+', '_', ':' or '.'. This is usable for proxy names, server names,
* ACL names, sample fetch names, and converter names.
*/
static inline int is_idchar(char c)
{
return isalnum((int)(unsigned char)c) ||
c == '.' || c == '_' || c == '-' || c == ':';
c == '.' || c == '_' || c == '-' || c == '+' || c == ':';
}
/*