linux-kernel - Re: [PATCH] ALSA: sound: Increase max size of components field

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: <20251028004252.GA553484@workstation.local>
Date: Tue, 28 Oct 2025 09:42:52 +0900
From: Takashi Sakamoto <o-takashi@...amocchi.jp>
To: Maciej Strozek <mstrozek@...nsource.cirrus.com>
Cc: Takashi Iwai <tiwai@...e.de>, Jaroslav Kysela <perex@...ex.cz>,
	alsa-devel@...a-project.org, patches@...nsource.cirrus.com,
	linux-sound@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] ALSA: sound: Increase max size of components field

Hi,

On Mon, Oct 27, 2025 at 11:50:32AM +0000, Maciej Strozek wrote:
> diff --git a/include/uapi/sound/asound.h b/include/uapi/sound/asound.h
> index c302698fb685..7d53f6da59e2 100644
> --- a/include/uapi/sound/asound.h
> +++ b/include/uapi/sound/asound.h
> @@ -1069,7 +1069,12 @@ struct snd_ctl_card_info {
>         unsigned char longname[80];
>         unsigned char reserved_[16];
>         unsigned char mixername[80];
> -       unsigned char components[256];
> +       unsigned char components[128];
> +};

Any approach to change the result of 'sizeof(struct snd_ctl_card_info)'
breaks ABI to userspace application. This is the reason that Jaroslav
addresses to the new PCM ioctl command. Unless, the userspace
application build with the UAPI header provided by the existing version
of Linux kernel does not work as expected in your version of Linux
kernel, in respect to the sound subsystem.


Regards

Takashi Sakamoto