Second patch listed at redhat is ok.

But I would either replace
#define WIDTH 80
by
#define WIDTH (sizeof(tmperr))

or
#define WIDTH 80
char tmperr[128];
by
#define WIDTH 128
char tmperr[WIDTH];

snprintf does add a \0 at tmperr[WIDTH]

WIDTH is a poor choice as a macro name IMHO.

Attachment: signature.asc
Description: This is a digitally signed message part.

Reply via email to