]> git.ipfire.org Git - thirdparty/kernel/linux.git/commit
ALSA: seq: avoid past-the-end iterator in snd_seq_create_port()
authorMaoyi Xie <maoyixie.tju@gmail.com>
Mon, 18 May 2026 19:40:23 +0000 (03:40 +0800)
committerTakashi Iwai <tiwai@suse.de>
Tue, 19 May 2026 05:39:06 +0000 (07:39 +0200)
commit92b62b7416af11fcfaab7373b15a32a471500bab
tree8787cf9e39f6c4d05fd2a51d9864a59a5f8bcdea
parent34ed2395613b23f8645e320abdcab6d688dc7a80
ALSA: seq: avoid past-the-end iterator in snd_seq_create_port()

snd_seq_create_port() walks client->ports_list_head looking for
the ordered insertion point and on loop fall-through passes
&p->list to list_add_tail():

    list_for_each_entry(p, &client->ports_list_head, list) {
            if (p->addr.port == port) {
                    kfree(new_port);
                    return -EBUSY;
            }
            if (p->addr.port > num)
                    break;
            ...
    }
    list_add_tail(&new_port->list, &p->list);

When the loop walks all entries without break (e.g., the new
port sorts last), p is past-the-end. &p->list aliases
&client->ports_list_head (the list head) via container_of offset
cancellation, so the insert lands at the list tail. That is the
intended behaviour, but the access is undefined per C11 even
though it works in practice.

Track an explicit insert_before pointer initialised to the list
head and overwritten to &p->list only when the loop breaks
early. The observable behaviour is unchanged.

Fixes: 9244b2c3079f ("[ALSA] alsa core: convert to list_for_each_entry*")
Signed-off-by: Maoyi Xie <maoyixie.tju@gmail.com>
Link: https://patch.msgid.link/20260518194023.1667857-3-maoyixie.tju@gmail.com
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/core/seq/seq_ports.c