src/crypto/heimdal/appl/push/push.c:270]: (style) Checking if unsigned variable 'from_line_length' is less than zero.
Source code is
from_line_length = snprintf (from_line, sizeof(from_line),
"From %s %s", "push", ctime(&now));
if (from_line_length < 0 || from_line_length > sizeof(from_line))
errx (1, "snprintf failed");
but variable from_line_length is unsigned and the return value
from snprintf is int.
Suggest sanity check return value from snprintf *before* storing in an unsigned variable.