[systemd-devel] [PATCH] fix bogus boot error message

Kay Sievers kay.sievers at vrfy.org
Tue Jul 19 05:04:23 PDT 2011


On Tue, Jul 19, 2011 at 13:22, Kelly Anderson
<kelly at silka.with-linux.com> wrote:
> The following commit introduces a bogus error message on boot since
> localtime may be a negative number, i.e. GMT-7.

> The following patch just uses errno directly rather than using it indirectly
> through hwclock_apply_localtime_delta():

Ah, tm->tm_gmtoff can be negative.

> +       errno = 0;
>          min = hwclock_apply_localtime_delta();

We usually prefer to return the errno instead of mangling it at the
caller's side, and have the values that can be negative passed as
arguments. Mind switching that? Or let me know.

Thanks,
Kay


More information about the systemd-devel mailing list