linux-kernel - Re: [PATCH v3 3/3] iio: imu: st_lsm6dsx: add support for rotation sensor

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <693c13bcdda7cf871022075aa784a9d15de0a8ba.camel@baylibre.com>
Date: Tue, 20 Jan 2026 13:23:31 +0100
From: Francesco Lavra <flavra@...libre.com>
To: Andy Shevchenko <andy.shevchenko@...il.com>
Cc: Andy Shevchenko <andriy.shevchenko@...el.com>, Lorenzo Bianconi
	 <lorenzo@...nel.org>, Jonathan Cameron <jic23@...nel.org>, David Lechner
	 <dlechner@...libre.com>, Nuno Sá <nuno.sa@...log.com>, 
	Andy Shevchenko
	 <andy@...nel.org>, linux-iio@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3 3/3] iio: imu: st_lsm6dsx: add support for rotation
 sensor

On Tue, 2026-01-20 at 12:25 +0200, Andy Shevchenko wrote:
> On Tue, Jan 20, 2026 at 12:03 PM Francesco Lavra <flavra@...libre.com>
> wrote:
> > On Tue, 2026-01-20 at 11:36 +0200, Andy Shevchenko wrote:
> > > On Tue, Jan 20, 2026 at 10:28:15AM +0100, Francesco Lavra wrote:
> > > > On Mon, 2026-01-19 at 12:33 +0200, Andy Shevchenko wrote:
> > > > > On Mon, Jan 19, 2026 at 11:04:49AM +0100, Francesco Lavra wrote:
> 
> ...
> 
> > > > > > +       snprintf(sensor->name, sizeof(sensor->name), "%s_sf",
> > > > > > name);
> > > > > 
> > > > > Does GCC complain on this (`make W=1` build)?
> > > > > Since this can cut the string and we don't check the return
> > > > > value,
> > > > > the Q
> > > > > is:
> > > > > is this okay to have a reduced string?
> > > > 
> > > > gcc does not complain with W=1. sensor->name is appropriately sized
> > > > to
> > > > accommodate the longest possible name; if it wasn't, the string
> > > > would
> > > > be
> > > > cut in the accel and gyro IIO devices too (which use a longer
> > > > suffix
> > > > than
> > > > "_sf").
> > > 
> > > Right, the question is if compiler can prove that or not.
> > > 
> > > We have several patches in input subsystem to hide the warning by
> > > switching
> > > to scnprintf(), which I consider not the best approach, but still it
> > > depends
> > > if we care about cut or not. If we do, we should check for overflow.
> > 
> > A truncated string would result in a corrupted value in
> > /sys/bus/iio/devices/iio:deviceX/name, so I would say we do care if
> > it's
> > truncated. But I think the most appropriate check we could add in the
> > code
> > is a BUG_ON(); would that be acceptable?
> 
> No. BUG_ON() == panic == reboot (with "reboot_on_panic" or so). Mere
> drivers like this one should (almost) never use BUG() nor WARN(). Just
> add an appropriate check.

I'm struggling to find a suitable error code to be returned in case of
failure. What do you suggest? -ENOSPC? -EOVERFLOW?

Download attachment "signature.asc" of type "application/pgp-signature" (660 bytes)