From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joerg Vehlow Date: Fri, 15 Jan 2021 07:14:17 +0100 Subject: [LTP] LTP testcase(sysctl02) failed In-Reply-To: <0efaa481ffd24bc48fd41385159be66c@exch01.asrmicro.com> References: <0efaa481ffd24bc48fd41385159be66c@exch01.asrmicro.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: ltp@lists.linux.it Hi Gao, On 12/24/2020 6:56 AM, Gao Meitao????? wrote: > > after investigated it, this testcase is for bufffer overflow, I > checked source code (kernel 4.19), > > all of required patched had been applied, I dig into it and found new > patch applied as flow , that might make testcase failed > > so is this testcase out of date or something wrong wih my kernel ? > > who can help me figure this out? Thanks in advance. > I also saw this on 4.19. From the mailing list of the kernel, it looks like there was some confusion when they ported the patches back to 4.19. I do not find the mails on the mailing list anymore. But the commit 7f2923c from master is missing IIRC. Christian Brauner made two changes: 1. Prevent over and underflow by just keeping the old value (7f2923c together with 32a5ad9c). This introduced a kasan bug fixed in 9002b21 2. Return EINVAL, if an overflow or underflow of the new value is detected (e260ad01f) From what I remember Christian wanted the first change completely backported to 4.19 (7f2923c, 32a5ad9c and 9002b21) and he wanted to omit the change done in e260ad01f, because that changed the behavior of the interface towards the user. But when the changed were backported to 4.19, the commit 32a5ad9c, 9002b21 and e260ad01f were backported. Since 7f2923c is the commit, that really enables over- and underflow detection, this is not backported and all the other commit (including e260ad01f) have no effect. At the time I found this, I had no time to find out how and where to report this. @Cyril I guess you may know how and who to report this to? J?rg