<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 21 April 2015 at 16:45, Arnd Bergmann <span dir="ltr"><<a href="mailto:arnd@arndb.de" target="_blank">arnd@arndb.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tuesday 21 April 2015 16:36:13 Baolin Wang wrote:<br>
> On 21 April 2015 at 04:48, Thomas Gleixner <<a href="mailto:tglx@linutronix.de">tglx@linutronix.de</a>> wrote:<br>
><br>
> > On Mon, 20 Apr 2015, Baolin Wang wrote:<br>
> > >  /* Set clock_realtime */<br>
> > >  static int posix_clock_realtime_set(const clockid_t which_clock,<br>
> > > -                                 const struct timespec *tp)<br>
> > > +                                 const struct timespec64 *tp)<br>
> > >  {<br>
> > > -     return do_sys_settimeofday(tp, NULL);<br>
> > > +     struct timespec ts = timespec64_to_timespec(*tp);<br>
> > > +<br>
> > > +     return do_sys_settimeofday(&ts, NULL);<br>
> ><br>
> > Sigh. No. We first provide a proper function for this, which takes a<br>
> > timespec64, i.e. do_sys_settimeofday64() instead of having this<br>
> > wrapper mess all over the place.<br>
> ><br>
><br>
> Thanks for your comments,but if use do_sys_settimeofday64() here that<br>
> will  introduce<br>
> a security bug: do_sys_settimeofday contains a capability<br>
> check that normally prevents non-root users from setting the time.<br>
><br>
> With your change, any user can set the system time.<br>
<br>
</span>He was asking for a new do_sys_settimeofday64 function to be added,<br>
not using the low-level do_settimeofday64.<br>
<span class="HOEnZb"><font color="#888888"><br>
        Arnd<br>
</font></span></blockquote></div><br>Sorry for the misunderstand, i'll fix that in next patch. Thanks.<br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div>Baolin.wang<br></div>Best Regards<br></div></div>
</div></div>