Re: [patch] kernel events layer

Robert Love
2004-07-24T02:46:59Z

On Sat, 2004-07-24 at 00:32 +0300, Dan Aloni wrote:

> IMHO you either should not assume anything about the length of the object
> string, _or_ do the complete safe string assembly e.g:
>
> len += snprintf(buffer, PAGE_SIZE, "From: %s\nSignal: %s\n",
> object, signal);
>

Fair enough. I guess what we want, exactly, is:

len = snprintf(buffer, PAGE_SIZE, "From: %s\n", object);
len += snprintf(&buffer[len], PAGE_SIZE - len "Signal: %s\n", signal);

I will add that to the next revision.

Robert Love