From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753196AbdFLVGz (ORCPT ); Mon, 12 Jun 2017 17:06:55 -0400 Received: from mail-ve1eur01on0119.outbound.protection.outlook.com ([104.47.1.119]:17780 "EHLO EUR01-VE1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752497AbdFLVGm (ORCPT ); Mon, 12 Jun 2017 17:06:42 -0400 Authentication-Results: zytor.com; dkim=none (message not signed) header.d=none;zytor.com; dmarc=none action=none header.from=virtuozzo.com; Date: Mon, 12 Jun 2017 14:06:11 -0700 From: Andrei Vagin To: tip-bot for Thomas Gleixner Cc: linux-tip-commits@vger.kernel.org, hpa@zytor.com, linux-kernel@vger.kernel.org, gorcunov@openvz.org, john.stultz@linaro.org, mingo@kernel.org, tglx@linutronix.de, peterz@infradead.org Subject: Re: [tip:timers/core] posix-timers: Zero out oldval itimerspec Message-ID: <20170612210610.GA23730@outlook.office365.com> References: <20170609201156.GB21491@outlook.office365.com> MIME-Version: 1.0 Content-Type: text/plain; charset=koi8-r Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.8.0 (2017-02-23) X-Originating-IP: [65.152.152.74] X-ClientProxiedBy: SG2PR0401CA0001.apcprd04.prod.outlook.com (2603:1096:3:1::11) To VI1PR08MB0750.eurprd08.prod.outlook.com (2a01:111:e400:5a04::12) X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: VI1PR08MB0750: X-MS-Office365-Filtering-Correlation-Id: 520d3686-c3b0-4d76-8c59-08d4b1d6ea2f X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001)(201703131423075)(201703031133081);SRVR:VI1PR08MB0750; X-Microsoft-Exchange-Diagnostics: 1;VI1PR08MB0750;3:GUb/2OqddLyJsjmqnjdrgdN6XUBSZqDjpcDIPY4T4Ot8FMd8+XlTVM32KavMP+x1agS3urDqRoLkhKwi0mFH6EwzOPV+rp1RKpHm9ypi22jXdP1vgpKCbbLjw/I0h2n6n+5QawlVe7rTE77VlGn5PVvvWEG8mGdTOX5ezY5BqhBtsW5AwVsubhYFayRrwZMXMrj3mGlFUVXbrjafiZLSzgZ33rhg1ZAy2+KdTFbbZpL8EbFpEHoaL8cIoJ0X2eusOQX/NatfxnnoWsT8ksdOz881xzc+IFbCx/jtx4/5CDc6PvT5MpaiTo3a+Mq4zHtDFcwcM5F/hO6CkDRbsRQveA==;25:TM08D6O8k0umMQTpK62ZDsBCWl7eJUkB8rXsVcl3fl/TuPKplDAcw5YM5DGXmkINhYO+6btjoH14X6l/tO1DksZCEUCosdwgh9yFtyvYf3lkRerHqxySeo3vxIti6vxttFLhOjnqleI1uGfHfpVfsqcNLRkgk9yVePZvuMg+glIZoGROQP3sbWwPeJziLq39PDt5MyLYH4hNxYc7c9eeDFTDPTGdUlNYxgVvrSd3UdZEl1vhvCxkTwJxba6wwvLiF5Z+4LJs7e5K1v+VRcIZDszCf4NYXTn/49Ub1L147wVVRkpUIYmxK04t4gc5gKVNG2Eq5JLruxhX7pjQxtDQj98uTLMRzcby6ZTg26TEEyOn/UuouQASC+Ls2gF3H4WkKVRRvhgpZVkibWeAgVlov3TbYUoQMxxHok15764LB/QFwcJy5hazc8g9x0P4fPplyxckBale5kIEV0O4wQp6xu/xldYVf2apQia4U3raZwY= X-Microsoft-Exchange-Diagnostics: 1;VI1PR08MB0750;31:GyLAhJ37Oy/8agzJC+fdxbxYHwA7TR5VJIYvw5XmUS8rItrSoZ0PSZiGBzG0ocrLA6B49l/jguPw08GYQIcNzf6lO6f3A4Hel1Wcp6MnITsR0bJvx+byMkLlhiogOw3krbuDmjyp4odmvtcXfrGJBySD/F9ssDm4QfhjLxx1iUcru6b5A1x0XwGzQAAVuNF8Jw+ueKhRBees34VTgyJHDgu7fTLjRfRt5kStflfMrBA=;20:k7LwZvPcBM7LOWlzc0TQ/+U8JFN6lMZJVhLx6nrYUPLP1LloApRzHMIkb5tEU+mgafjaISKxB+SJKToi81HzytJTmw03ema19a/smxNJol3VPwacKB+kAcyzoonrz/ty6D+rP28RmPHrutineN0Nu/9CE56kx0y+1S6w2WVYNgyKGn0LsFldbcZMPAuwo+ydQ9umWg4PVOzJ+TnewvoN7V0o63xFcRLDmyY3nfgsrWQH8riaFoub/Kwj4F1fJvgiJ7E2ugqjGTK2gSat0GaQRfExfTfoov3JNOcIghbpW9Fn7ieE/m0wMmMmAzCEm94qmUHRiM0XTAmnhPQbbbo2czsRZhWhCHH4mnOKmzvOWFjbgDewU6dlIzA+SxX7aqDBzNjxQWdXDh81aCDZHfnxnWw469v4s8/kzx7Qr5mRI9w= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(277809949798613)(42068640409301)(84791874153150)(13630703328958); X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(601004)(2401047)(8121501046)(5005006)(93006095)(93001095)(100000703101)(100105400095)(10201501046)(3002001)(6041248)(20161123564025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123560025)(20161123562025)(20161123558100)(20161123555025)(6072148)(100000704101)(100105200095)(100000705101)(100105500095);SRVR:VI1PR08MB0750;BCL:0;PCL:0;RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095);SRVR:VI1PR08MB0750; X-Microsoft-Exchange-Diagnostics: =?koi8-r?Q?1;VI1PR08MB0750;4:pPnyWUhjvxwr4BpiugjG70KZcASajSv1LVdZyxOAgbfs?= =?koi8-r?Q?T4TFJ8p9pNP3AJXZdYa3u4ZJrP0RixEozOoELQOPD14kNsnRg3YlGmiu4wGy96?= =?koi8-r?Q?QKCORo+p9khfaVvcKE+8gcqMGRkLER05xhJcnixMHfi8uOnaFbFromqFgsA2kN?= =?koi8-r?Q?t6J8oGOrbUIs1Jz7Aa2vSEuTSZ9vlmwSjhL2iPmZaIZAfuOwMVYiaE+AnY1q9r?= =?koi8-r?Q?CfeHEpcbu6oMmFvoS+rHIlBGOg2nNSoMJyxyt6oQgh4Fso3kDJuHKA5t6tc/JR?= =?koi8-r?Q?JTXpUdHA30u9I1QaWcLbxci9SCiwvgUi33z0bKAcHXW2Me03kjghcYyp8jHlZl?= =?koi8-r?Q?w/+B21WQe2AQFMep+AsqyXzp0j8VO+/GQhWwcIhsVkAAYC49iEDOBO7SrAeTuK?= =?koi8-r?Q?7uWMGvh/V4R68CkKdm5lStuc0O63549Oy55i3MYS9Aps1Wflu4e176H1l3/Loz?= =?koi8-r?Q?kE3k3d12OK2xqsrKg39dbR0+S187TLIzh/O036uv6VAFO6jjUnurAiA8VmgLUZ?= =?koi8-r?Q?B4n/HmY8qAuc9fQVJhLifbA3JjeIAgZWkpBMR2UQ+YDMQ1kZdxnWly+VOyzUX0?= =?koi8-r?Q?Q/B1Qg1bady7t9xq/gQEGxIrAZ9HFyXqz/KVLztsmBkGG9p3A0k2dtQDBVXnqY?= =?koi8-r?Q?OJqT1KRLI3YVkeeDzdBZiWGOTLVnXzeTOPO/sVzJnDX07T/iv9vtdvVbKYuVTK?= =?koi8-r?Q?y8tTJKGsYzbkR6GUGgrfbx0P5zLDJzPjGMKwVGivTq/UTe420eej7EPedA97Lb?= =?koi8-r?Q?Og+RxmotUdtSngkZkReuWFmjUsXpwDK/DJZ2i0WqZFVt9oIrwiCrS41OkfCbGc?= =?koi8-r?Q?KWNBkSAqwxoRwYHGVZ5X6It5k7/hbXuIOzYQuHsTCiW5hV7Jbg1QH8cO1fpv7O?= =?koi8-r?Q?KOb9mZOiV6V6IVceY7p5C3/vSo+yueCycwmyfSLCTIiSB6LNHD1tazPm9mtJUQ?= =?koi8-r?Q?R3lEyjgU6l2wqqMJ+gwSfdAoXhS6IWaNovJftfY+dZPsLhmTtpDwQ9wPGcwli7?= =?koi8-r?Q?L+DaGZ4fqe+rnA7h7Da/j9T6Kp6vlKLH9Aw4KqdpvvMCbBgxUsdvV1l1URLk6w?= =?koi8-r?Q?PSWTpFW02emlwlOjYNiEw2KRSjANF+xyMyWGfrptZ4ALH100nXUwJI1C7k3OXo?= =?koi8-r?Q?9NMe3D71QfbKbC7akAI/nHW843GlUMLTPAJ4klxw8Zq9p1tVhQzd1RZu6C8+Y+?= =?koi8-r?Q?Ra+Hxk9xmwx3W+QjoBluD735iYoWBAKQqSXUsJ0aOh1KE0PFglm/VILUZ9wXYs?= =?koi8-r?Q?eWg5VsiuPDHWiGnfyw=3D=3D?= X-Forefront-PRVS: 03361FCC43 X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10019020)(4630300001)(6009001)(39400400002)(39830400002)(39450400003)(39410400002)(24454002)(47776003)(7736002)(305945005)(5660300001)(575784001)(478600001)(66066001)(1720100001)(38730400002)(110136004)(45080400002)(6246003)(53936002)(2906002)(54356999)(33656002)(76176999)(6506006)(25786009)(189998001)(6306002)(9686003)(50986999)(966005)(55016002)(4326008)(1076002)(3846002)(53416004)(42186005)(6666003)(229853002)(86362001)(81166006)(8676002)(23686003)(6916009)(2950100002)(18370500001);DIR:OUT;SFP:1102;SCL:1;SRVR:VI1PR08MB0750;H:outlook.office365.com;FPR:;SPF:None;MLV:sfv;LANG:en; X-Microsoft-Exchange-Diagnostics: =?koi8-r?Q?1;VI1PR08MB0750;23:ZOO6bW4OJtarKZC0oJOFh2zlDUMHu9RW0Vme4/zo8/9?= =?koi8-r?Q?KEDZyDkOlP3yJMlqfDtI6OQUK6ClFiiIUyr6rPplb3VSvOLJcS8MevArwHG3+R?= =?koi8-r?Q?/F7kGqNjGER8KYRBnAE2BlTwiQhrm0eIuZI1bklE2mrAUvA2o5JnnBonQkP17J?= =?koi8-r?Q?jLEQf6IX23Bxs/63wuFS2VStk4+vVPLOSZpYhkqsm0QSGMPbox1u6gEKivV1XP?= =?koi8-r?Q?bcZEZm3herB30XN+HJNqnu+Nw8R5b09yahJjepvODcpY1rjRibnTuxikrnHEr2?= =?koi8-r?Q?RnByKFjYXWZuM67Znm42wdJ3JRDHvWrNOhHzRH2+12hq65LvfbNMG5oeTbhhGK?= =?koi8-r?Q?dEKR/qKrukBkJiHrG6BBxjm9bH2ZWlGxRbJDeg/Ta9t58aUrGy4bXDOPMVqerV?= =?koi8-r?Q?Qz/lf04dtJEu6w/XGfBt89dXmtR5wlXBy34xx+UIlEpgZrTS1O/+o7wc9WAwiJ?= =?koi8-r?Q?85DU1cSOWnMYnkc9QU2rjCxdZ4jccP+7tUr3wNr/3Bies2JjnPzQOwuoba3XFm?= =?koi8-r?Q?bci6FY1A4fmpzC4ksR9/rE31q2r8/+axB1m3bwW7quOM2AVUd+lUp05lyZw0zI?= =?koi8-r?Q?ddwMHSSEwYUuwVjSLa6veyWZbjabok5NDIWjp174GCHr1onmt/mL8kG/aDENNh?= =?koi8-r?Q?db1ZaWCJvDkeqXmG4ZIqH4zWKBgHBJrw6+Lly2q4Aazgc6/hOxB1mV0uJ7tPor?= =?koi8-r?Q?SGjnMxr97wk4mMDHe/MErInd3JhQtK5lK0tPtiC/LoQ9BBIyQ3P1J2UTrnmSMh?= =?koi8-r?Q?C9hK+DxjGNkng4NHUX1R5DzjPTnFDQMqyN+vSOEQR2Fr8rq/pvWP0hG54EQJ32?= =?koi8-r?Q?XDMQP5k1GH2dMXAOhxGsyXHtTuagdnG04ufT77mn7P7ztdgG+FzQz7Jwuk/Nr6?= =?koi8-r?Q?7d3TknN+4o1dgSXd9HmiuOgPYCpL8lbD8ywqHUZH2B/F18oWbUzSYT6sJzolgK?= =?koi8-r?Q?zggyCFlSSud5RtuhfeX5brs2n53N6fVA4Q98yfh8ntF3slNtkr/gYYooRV0p59?= =?koi8-r?Q?OmsG0J64VP+VpqwuH7ELiTtLsZu7jGaYVoM6xV08M5ht+l/CK6a5oib58u3q+l?= =?koi8-r?Q?FSVAvrdPY0uTfhfYpdFst/C2p7Q+04REEU+xSXnWvWr1jlC026/xuC2?= X-Microsoft-Exchange-Diagnostics: 1;VI1PR08MB0750;6:HVpLAL4MPhogMFldCjpcwIbw5f5EfB+AK9MJtNrF5OVPvVdV8K2kL+o0OCgoTCamipKtoymcSF9Zjgy0JhfaGAVPrYQgOoqAqGcEmVqEmaRsC1XVfsCprkYTFcxKPM7WGN6bk8naTInCmInPfBrFSZh6NDz4C9FsUXVYFzAUjjKC6FUwODDRzrpD0zyBUNUUJFeoxd1oQPI3p+pMnItUFlnEIVvH8UOBdiRK9Y33/12ggqlGrdlW5qAW6PRLZGSCGfONGuLxM8MtTTvHWmUmzMExiHumyeeuHswRf3QC3Id8fcWdjqd4e9vGSEHPJCwHGxyvoJfLGrjhzqF3HBkqyYuMdvs1QwS0IGn7oXo16VmICo/xeqZghrwoDwxYYhdTo/Vj1p5JRv/5aLbwIUZUeBwzb7syfJgjpy4aDMg3SKrYZSt1kl5wOD/EcA0IWWariBPGEEvuccNJEg9HoNOt3UWpIlVpi33K4C7KxRJfaUvnL5gR1WcXsQSjbbnX4dlL/aCsTVK2yxB6xHzMg/5Pxg== X-Microsoft-Exchange-Diagnostics: 1;VI1PR08MB0750;5:KFgba1LsiOGvXQVhwWMp59YSaOxk0rIVHdkbRUcXJiJ/AcuefofMvBpcGy1iqRlpJMhivOzKRTGDLqk21MBf/zMKmhyWyU50EPny8HNlkVZXGHH4+wMyekIIxoU0goNGkYky1lsrCw1J9qkmefbtc0siXhiPcguusFn+ZqiN9igKn6PZhIOlCo3jfB9WrC6O4u4zAGrKqeYgRJP8E2O/sJtvnfFwWcAEL8gA16PP9xhaLDtKOtgL0qfPI2CsX9hhhAJUMOuxb9jSA9MAtlNkgcZjdBZslReuiyaYqV+ZL7h8S5R0qCmpPFv+MzjrfjSxcW6l9is+qnB//DDDQFoX6v9tTkg8WtaMLfXL3Y4qkmjngpenCV+lVpbuZJE7lCgs0y1UYblRsM2zleSTjddTJK9MsNgm7VeKPsK1cTppoHKDi5J0IFFtSdEJI1w57gaJatZb4G+WW+wwnZKuvRjbmkikDBMJKp0QAej791keSdF0b7M5WaNrVx49UwbMw3E0;24:VQQe3scN1E1it9vCJ94SMG9n/borsuAVKSnZGFKK7qR8JFt8vFKapDJ0Yg3P+Zw1+lTkowTxuEDoCriCmbZU5LDJ74VTmpnY/4D/CsomGZY= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1;VI1PR08MB0750;7:LXoH7tQv3vdl8X3lwdg8gRiqQnjkYtQfyDMxkXIw5C9Q1Te+kv+AXgtaLKME3bNqzW1ueuGb17x2gAkZaHFm6MMF5KwNIrNoVTH7umtZk6Pi7I8ZIwv5e2qJpUPRTNQiOwI4Y6RyyghO5sMmtxg3lau/G4WlN5Z6HF4EXh+F8Sce2QaC/piWiVYz1xcfaRlh6k5uae9rtPDhMwTqYVxkn889ebLexPFlZi+4VlQjBx9odV70mcMU8bQi/gMFEh6DNaBGbUTFnXRBlDnt/XuIeKGbMf28oOJE7MKGiXkRsRC0BinJcKzunKiuoIukyj3gUMfvsBpOvTShOqlIPEE9xQ==;20:V73XpZ8vdSssKyZrGtPWVhK+YtViKeOIaPZrsSi5bHmY2QHzCGAzF0O09boaJpn08bjvexpfzM5d0Knjds6NQ7VucZ5JcwwOguMXaw6AbF3AGEE4W8k1hs7duMEfcYwkEzcd5bjZektThCf3+pLE1AtSYSIelCFOviPMbpOxPoM= X-OriginatorOrg: virtuozzo.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Jun 2017 21:06:33.6652 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR08MB0750 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Jun 12, 2017 at 12:13:15PM -0700, tip-bot for Thomas Gleixner wrote: > Commit-ID: 5c7a3a3d20a4e175304c0e23809e3d70be8fed8a > Gitweb: http://git.kernel.org/tip/5c7a3a3d20a4e175304c0e23809e3d70be8fed8a > Author: Thomas Gleixner > AuthorDate: Mon, 12 Jun 2017 19:44:09 +0200 > Committer: Thomas Gleixner > CommitDate: Mon, 12 Jun 2017 21:07:40 +0200 > > posix-timers: Zero out oldval itimerspec > > The recent posix timer rework moved the clearing of the itimerspec to the > real syscall implementation, but forgot that the kclock->timer_get() is > used by timer_settime() as well. That results in an uninitialized variable > and bogus values returned to user space. > > Add the missing memset to timer_settime(). > > Fixes: eabdec043853 ("posix-timers: Zero settings value in common code") > Reported-by: Andrei Vagin > Signed-off-by: Thomas Gleixner > Cc: Peter Zijlstra > Cc: John Stultz > Cc: Cyrill Gorcunov > Link: http://lkml.kernel.org/r/20170609201156.GB21491@outlook.office365.com > --- > kernel/time/posix-timers.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/kernel/time/posix-timers.c b/kernel/time/posix-timers.c > index b53a0b5..88517dc 100644 > --- a/kernel/time/posix-timers.c > +++ b/kernel/time/posix-timers.c > @@ -828,6 +828,8 @@ SYSCALL_DEFINE4(timer_settime, timer_t, timer_id, int, flags, > if (!timespec64_valid(&new_spec64.it_interval) || > !timespec64_valid(&new_spec64.it_value)) > return -EINVAL; > + if (rtn) > + memset(rtn, 0, sizeof(*rtn)); Maybe we need to call memset after "retry:"? common_timer_get() is called at the begining of common_timer_set(), then common_timer_set() can return TIMER_RETRY. common_timer_get() will be called again and some fields of rtn which have been touched first time will not be touched. At the end, rtn will contain data from two executions of common_timer_get(). Thanks, Andrei > retry: > timr = lock_timer(timer_id, &flag); > if (!timr)