Svante Signell, on Tue 30 Aug 2016 23:38:19 +0200, wrote:
> On Tue, 2016-08-30 at 23:21 +0200, Samuel Thibault wrote:
> > Svante Signell, on Tue 30 Aug 2016 17:40:00 +0200, wrote:
> > > 
> > > Attached is an updated patch returning also EINVAL if abs(delta) is too
> > > large.
> > Never mix changes, please.
> > 
> > > 
> > > Whether the  adjtime()/RPC_host_adjust_time.c:___host_adjust_time() 
> > > survives
> > > a
> > > delta being NULL remains to test.
> > Well, there is no need to test::)
> > 
> > *(time_value_t *) delta,
> > 
> > will just crash.
> 
> How probable is this kind of (misuse) of this function?

? It's not misuse, it's even documented in the manual.

> Nevertheless, maybe we could use another dummy struct timespec (or the
> one already available)?

No. Please think about what values you would put in the dummy struct,
and what consequence they will have.

Samuel

Reply via email to