All of lore.kernel.org
 help / color / mirror / Atom feed
* pcm: ioplug: Limit transfer size to buffer boundary
       [not found] <1606476231002583981-webhooks-bot@alsa-project.org>
@ 2020-11-27 11:23 ` GitHub pull_request - opened
  0 siblings, 0 replies; only message in thread
From: GitHub pull_request - opened @ 2020-11-27 11:23 UTC (permalink / raw)
  To: alsa-devel

alsa-project/alsa-lib pull request #103 was opened from aditpape:

Commit 1714332719fc91507ca24dd3567e50d7094b3001 introduced 2nd transfer()
call to transfer all remaining available frames.
If the prior calculated avail value exceeds the buffer size a too large size value
is passed to the underlaying plugin and results in memory corruption if not blocked by plugin internally.
Avail values > buffer size can happen if e.g. xrun detection is disabled,
as avail is calculated by pure difference between hw and app position.
This patch limits 2nd transfer call to remaining rest of a buffer size.

Signed-off-by: Andreas Pape <apape@de.adit-jv.com>

Request URL   : https://github.com/alsa-project/alsa-lib/pull/103
Patch URL     : https://github.com/alsa-project/alsa-lib/pull/103.patch
Repository URL: https://github.com/alsa-project/alsa-lib

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2020-11-27 11:24 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <1606476231002583981-webhooks-bot@alsa-project.org>
2020-11-27 11:23 ` pcm: ioplug: Limit transfer size to buffer boundary GitHub pull_request - opened

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.