Re: [PATCH] sanitize paths used in regular expression

2016-01-15 Thread David Malcolm
On Thu, 2016-01-14 at 17:31 -0800, Mike Stump wrote: > On Jan 14, 2016, at 4:54 PM, Zachary T Welch wrote: > > This patch fixes a small problem when running 'make check' from a path > > that includes "++". When such paths get used as a regular expression, > > that

Re: [PATCH] sanitize paths used in regular expression

2016-01-15 Thread Mike Stump
On Jan 15, 2016, at 2:47 AM, David Malcolm wrote: > FWIW, I do something similar in multiline.exp's _build_multiline_regex, > which attempts to have a complete list of metacharacters (though I > believe some of these are not valid for POSIX filenames); Only ‘\’ and ‘\0’ are

[PATCH] sanitize paths used in regular expression

2016-01-14 Thread Zachary T Welch
This patch fixes a small problem when running 'make check' from a path that includes "++". When such paths get used as a regular expression, that sequence would cause a runtime error. That is prevented here by escaping that character. gcc/testsuite/lib/ * prune.exp

Re: [PATCH] sanitize paths used in regular expression

2016-01-14 Thread Mike Stump
On Jan 14, 2016, at 4:54 PM, Zachary T Welch wrote: > This patch fixes a small problem when running 'make check' from a path > that includes "++". When such paths get used as a regular expression, > that sequence would cause a runtime error. That is prevented here by >