Re: [patch] kernel events layer

From: Robert Love
Date: Fri Jul 23 2004 - 21:48:07 EST

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

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at