Re: [PATCH] Optionally sanitize globals in user-defined sections

[Yury Gribov <y.gribov@samsung.com>]

On 04/19/2015 06:11 PM, Jakub Jelinek wrote:
> On Sun, Apr 19, 2015 at 10:54:57AM +0300, Yury Gribov wrote:
>> On 04/17/2015 08:29 PM, Andi Kleen wrote:
>>> Yury Gribov <y.gribov@samsung.com> writes:
>>>> +
>>>> +static bool
>>>> +section_sanitized_p (const char *sec)
>>>> +{
>>>> +  if (!sanitized_sections)
>>>> +    return false;
>>>> +  size_t len = strlen (sec);
>>>> +  const char *p = sanitized_sections;
>>>> +  while ((p = strstr (p, sec)))
>>>> +    {
>>>> +      if ((p == sanitized_sections || p[-1] == ',')
>>>> +	  && (p[len] == 0 || p[len] == ','))
>>>> +	return true;
>>>
>>> No wildcard support? That may be a long option in some cases.
>>
>> Right. Do you think * will be enough or we also need ? and [a-f] syntax?
>
> libiberty contains and gcc build utilities already use fnmatch, so you
> should just use that (with carefully chosen FNM_* options).

Hi all,

Here is an new patch which adds support for wildcards in 
-fsanitize-file:///home/ygribov/user-section-2.diff
sections.  This also adds a test which I forgot to svn-add last time 
(shame on me).

Bootstrapped and regtested on x64.  Ok to commit?

-Y