Date: Wed, 7 Jun 2017 19:18:14 +0200
From: Seraphime Kirkovski <kirkseraph@gmail.com>
To: "Moore, Robert" <robert.moore@intel.com>
Cc: devel@acpica.org, afael.j.wysocki@intel.com,
        linux-acpi@vger.kernel.org, rafael.j.wysocki@intel.com,
        lv.zheng@intel.com, lenb@kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] acpi:  acpica:  dsutils: fixanoff-by-one index
Message-ID: <20170607171814.zjvrikbjonp5n3gd@macchiaveli>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <94F2FBAB4432B54E8AACC7DFDE6C92E37E5BF38C@ORSMSX110.amr.corp.intel.com>
User-Agent: NeoMutt/20170113 (1.7.2)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 517
Lines: 13

On Wed, Jun 07, 2017 at 03:14:46PM +0000, Moore, Robert wrote:
> I believe that the rationale for this is that at that point in the code, it is *guaranteed* that there is at least one operand; therefore the -1 would always be valid.
>
> In the end, we just deleted that call to 
> acpi_db_display_argument_object.
> 
> I don't know if this change has made it into Linux yet.
> 
The latest rc actually produces the UBSAN splat in my previous message.
So I suppose, I have some buggy hardware/firmware.

Thanks,
Seraph