Ketan, What is the Coverity Impact Level for this defect? If it is Low then we can whitelist this defect and change.
Peter, This isn't a security fix. However, I see this is kind of code cleanup. -----Original Message----- From: Peter Zijlstra <[email protected]> Sent: Thursday, February 28, 2019 3:18 PM To: Ketan Patil <[email protected]> Cc: [email protected]; [email protected]; [email protected]; Sachin Nikam <[email protected]>; Bharat Nihalani <[email protected]>; Bo Yan <[email protected]>; Sai Gurrappadi <[email protected]>; Thierry Reding <[email protected]>; Timo Alho <[email protected]> Subject: Re: [PATCH] sched/cputime: Remove unnecessary assignment statement A: Because it messes up the order in which people normally read text. Q: Why is top-posting such a bad thing? A: Top-posting. Q: What is the most annoying thing in e-mail? On Thu, Feb 28, 2019 at 03:12:13PM +0530, Ketan Patil wrote: > The coverity tool has detected this issue as an unused value, since > the code assigns the value to utime variable and then after the jump, > the value of utime again gets updated, hence the previous value is not > at all useful and this patch removes that first assignment. Not a security issue then; just tell coverity to shut up.
