Thanks for catching that, I will fix it.
Teresa
On Fri, May 9, 2014 at 7:49 PM, Andi Kleen <a...@firstfloor.org> wrote:
> Teresa Johnson <tejohn...@google.com> writes:
>>
>> Passes regression tests. Ok for google branches?
>
> + {
> + char parameter[1000];
> + sprintf (parameter, "%s=%ld",
> + curr_parm->macro_name, curr_parm->value);
> + cpp_define (parse_in, parameter);
>
> Buffer overflow. Use at least snprintf, but better some dynamic
> allocation.
>
> -Andi
>
> --
> a...@linux.intel.com -- Speaking for myself only
--
Teresa Johnson | Software Engineer | tejohn...@google.com | 408-460-2413
