Re: [PATCH] 2.4.19 do_adjtimex parameter checking

From: Ulrich Windl (Ulrich.Windl@rz.uni-regensburg.de)
Date: Mon Mar 25 2002 - 02:45:04 EST


On 24 Mar 2002, at 5:14, Paul Gortmaker wrote:

> Okay then, the bug still stands, but you want a different fix really.
> The user that reported it to me was infact setting multiple bit
> combos, which you indicate as taboo (but currently allowed). So
> when multiple bit combos are given, we can either do:
>
> a) return -EINVAL if more than ADJ_OFFSET_SINGLESHOT is set
> b) clear/ignore any bits above and beyond ADJ_OFFSET_SINGLESHOT.
>
> As you are a time guru, please indicate which is preferable and I will
> bounce Rusty an appropriate patch.

Thanks for the flowers,

my preference is EINVAL of course. Silently adjusting parameters is bad
IMHO.

Regards,
Ulrich

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Sun Mar 31 2002 - 22:00:09 EST