From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753996AbdLHPYI (ORCPT ); Fri, 8 Dec 2017 10:24:08 -0500 Received: from mail-qt0-f196.google.com ([209.85.216.196]:38545 "EHLO mail-qt0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753567AbdLHPYD (ORCPT ); Fri, 8 Dec 2017 10:24:03 -0500 X-Google-Smtp-Source: AGs4zMYfhDn1aGKwuDUnByZk/CmRmlbVEgrekmp6+9eSxe73pAHomO+hK2iSl4HRlxCteR5XUgrqZYN7nGlTXb+N8Ds= MIME-Version: 1.0 In-Reply-To: <20171208151114.ly7sqymznmeme4w4@rob-hp-laptop> References: <1512738783-17452-1-git-send-email-geert+renesas@glider.be> <20171208151114.ly7sqymznmeme4w4@rob-hp-laptop> From: Geert Uytterhoeven Date: Fri, 8 Dec 2017 16:24:02 +0100 X-Google-Sender-Auth: zvpBXzgklnRsvNzNG7RGRvlyvAg Message-ID: Subject: Re: [PATCH 0/2] of: overlay: Crash fix and improvement To: Rob Herring Cc: Geert Uytterhoeven , Pantelis Antoniou , Frank Rowand , "devicetree@vger.kernel.org" , Linux-Renesas , "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Rob, On Fri, Dec 8, 2017 at 4:11 PM, Rob Herring wrote: > On Fri, Dec 08, 2017 at 02:13:01PM +0100, Geert Uytterhoeven wrote: >> This patch series fixes memory corruption when applying overlays. >> >> I first noticed this when using OF configfs. After lots of failed >> debugging attempts, I bisected it to "of: overlay: add per overlay sysfs >> attributes", which is not upstream. But that was a red herring: that >> commit enlarged struct fragment to exactly 64-bytes, which just made it >> more likely to cause random corruption when writing beyond the end of an >> array of fragment structures. With the smaller structure size before, >> such writes usually ended up in the unused holes between allocated >> blocks, causing no harm. >> >> The first patch is the real fix, and applies to both v4.15-rc2 and Rob's >> for-next branch. >> The second patch is a small improvement, and applies to Rob's for-next >> branch only. >> >> I've updated my topic/overlays and topic/renesas-overlays branches at >> git://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git >> accordingly. >> >> Thanks! >> >> Geert Uytterhoeven (2): >> of: overlay: Fix out-of-bounds write in init_overlay_changeset() >> of: overlay: Make node skipping in init_overlay_changeset() clearer > > I've applied both and am updating my pull req to Linus. I hope that's > the end of it. If further fixes can't be reproduced with mainline, I'm > not going to be inclined to take them for 4.15. Tahnks! BTW, seems I accidentally used "--" instead of "---" as a separator, so the commit message https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git/commit/?h=dt/linus&id=efb72067c287cd6aba8eb434bd5bdc1ae0af6ed7 contains a few more lines than intended. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds