Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:5 as permitted sender) client-ip=2620:137:e000::3:5;
Feedback-ID: i56a14606:Fastmail
User-Agent: Cyrus-JMAP/3.9.0-alpha0-1178-geeaf0069a7-fm-20231114.001-geeaf0069
MIME-Version: 1.0
Message-Id: <d180f06b-64b0-4885-9794-5127c297a0f0@app.fastmail.com>
In-Reply-To: <ZWzLvctpo1nNTMOo@google.com>
References: <20231127204206.3593559-1-zack@kde.org>
 <ZWzLvctpo1nNTMOo@google.com>
Date:   Sun, 03 Dec 2023 21:50:51 +0100
From:   "Arnd Bergmann" <arnd@arndb.de>
To:     "Dmitry Torokhov" <dmitry.torokhov@gmail.com>,
        "Zack Rusin" <zackr@vmware.com>
Cc:     linux-kernel@vger.kernel.org,
        "VMware Graphics Reviewers" <linux-graphics-maintainer@vmware.com>,
        "Robert Jarzmik" <robert.jarzmik@free.fr>,
        "Raul Rangel" <rrangel@chromium.org>, linux-input@vger.kernel.org,
        stable@vger.kernel.org
Subject: Re: [PATCH] input/vmmouse: Fix device name copies
Content-Type: text/plain;charset=utf-8
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Sun, Dec 3, 2023, at 19:41, Dmitry Torokhov wrote:
> On Mon, Nov 27, 2023 at 03:42:06PM -0500, Zack Rusin wrote:
>> From: Zack Rusin <zackr@vmware.com>
>>=20
>> Make sure vmmouse_data::phys can hold serio::phys (which is 32 bytes)
>> plus an extra string, extend it to 64.
>>=20
>> Fixes gcc13 warnings:
>> drivers/input/mouse/vmmouse.c: In function =E2=80=98vmmouse_init=E2=80=
=99:
>> drivers/input/mouse/vmmouse.c:455:53: warning: =E2=80=98/input1=E2=80=
=99 directive output may be truncated writing 7 bytes into a region of s=
ize between 1 and 32 [-Wformat-truncation=3D]
>>   455 |         snprintf(priv->phys, sizeof(priv->phys), "%s/input1",
>>       |                                                     ^~~~~~~
>> drivers/input/mouse/vmmouse.c:455:9: note: =E2=80=98snprintf=E2=80=99=
 output between 8 and 39 bytes into a destination of size 32
>>   455 |         snprintf(priv->phys, sizeof(priv->phys), "%s/input1",
>>       |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>>   456 |                  psmouse->ps2dev.serio->phys);
>>       |                  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>
> This simply wastes 32 bytes. It is perfectly fine to truncate phys
> (which does not happen in real life).
>
> -Wformat-truncation is disabled in normal builds, folks should stop
> using it with W=3D1 as well.

It does find real bugs, and we are fairly close to being able
to enable it by default once the remaining warnings are all
fixed.

It also doesn't waste any memory in this specific case since
vmmouse_data is currently at 168 bytes, which gets rounded
up to either 192 or 256 bytes anyway. I'd suggest using
the minimum size that is large enough though, in this case
39 bytes for the string I guess.

     Arnd