linux-kernel - Re: adjtimex system man page EINVAL

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <f5a7b3810906040414me72f8b6h583ff3c05b99693@mail.gmail.com>
Date:	Thu, 4 Jun 2009 16:44:03 +0530
From:	naresh kamboju <naresh.kernel@...il.com>
To:	mtk.manpages@...il.com
Cc:	ziegler@...-freiburg.de, akpm@...ux-foundation.org,
	johnstul@...ibm.com, zippel@...ux-m68k.org,
	linux-kernel@...r.kernel.org, saji.kumar@...ro.com
Subject: Re: adjtimex system man page EINVAL

Dear Michael Kerrisk,

After comparing the changes between 2.6.25 and 2.6.29 kernels
As per my understanding EINVAL error check is removed with
timex.offset after 2.6.25 kernels and above.

To make sure the offset value following lines are added in
linux-2.6.29/kernel/time/ntp.c +80


+       /*
+        * Scale the phase adjustment and
+        * clamp to the operating range.
+        */
+       offset = min(offset, MAXPHASE);
+       offset = max(offset, -MAXPHASE);
+


I have attached patch as adjtimex_man.patch


Best wishes,
Naresh Kamboju

Signed-off-by: Naresh Kamboju < naresh.kernel@...il.com >
diff -Naurb man-pages-3.21/man2/adjtimex.2 man-pages-3.21_mod/man2/adjtimex.2
--- man-pages-3.21/man2/adjtimex.2      2009-04-15 21:35:32.000000000 +0530
+++ man-pages-3.21_mod/man2/adjtimex.2  2009-06-04 14:19:34.000000000 +0530
@@ -118,7 +118,8 @@
 .B EINVAL
 An attempt is made to set
 .I buf.offset
-to a value outside the range \-131071 to +131071,
+to a value outside the range \-131071 to +131071 applicable upto 2.6.25
+kernels, EINVAL error check with buf.offset been removed from 2.6.25 and above,
 or to set
 .I buf.status
 to a value other than those listed above,





On Tue, Jun 2, 2009 at 10:37 AM, Michael Kerrisk
<mtk.manpages@...glemail.com> wrote:
> Naresh,
>
> On Fri, May 22, 2009 at 1:17 PM, naresh kamboju <naresh.kernel@...il.com> wrote:
>> Hi,
>>
>> Regarding adjtimex system man page,
>>
>> Since there are modifications noticed in
>> /kerel/time/ntp.c
>> /include/linux/timex.h
>>
>> compared with /kerel/time/ntp.c 2.6.23 and 2.6.29 and above i have
>> attached adjtimex_2.6.23_vs_2.6.29.log : line number 330.
>>
>> In adjtimex (2) man page under the EINVAL Error check section changes
>> are required w.r.t buf.offset.
>> Could you conform and update the man page.
>
> Could you please summarize in a couple of sentence what the actual
> changes are that you think are required.
>
> Cheers,
>
> Michael
>
>
> --
> Michael Kerrisk
> Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
> git://git.kernel.org/pub/scm/docs/man-pages/man-pages.git
> man-pages online: http://www.kernel.org/doc/man-pages/online_pages.html
> Found a bug? http://www.kernel.org/doc/man-pages/reporting_bugs.html
>

Download attachment "adjtimex_man.patch" of type "application/octet-stream" (572 bytes)