Hi Paul,

> +static void directive_onoff(int onoff)
> +{
> +    if (directives_index >= sizeof(directives_stack)) {
> +     fprintf(stderr, "mhbuild: #on/off overflow, continuing\n");
> +     return;
> +    }
> +    directives_stack[++directives_index] = onoff;
> +}

I think the test is wrong.  If 31 >= 32 is false so
directives_stack[++31] will be written to.

Cheers, Ralph.

_______________________________________________
Nmh-workers mailing list
[email protected]
https://lists.nongnu.org/mailman/listinfo/nmh-workers

Reply via email to