November 2024 - Alsa-devel - mailman.alsa-project.org

[PATCH 0/6] Add generic AMD Soundwire machine driver for Legacy(No
by Vijendar Mukunda 14 Nov '24

14 Nov '24

This patch series add SoundWire machines for RT711, RT714, RT1316 and RT722 codecs for ACP 6.3 platform. Also, it adds a generic SoundWire machine driver code for legacy(No DSP) stack. Vijendar Mukunda (6): ASoC: amd: acp: add rt722 based soundwire machines ASoC: amd: acp: add RT711, RT714 & RT1316 support for acp 6.3 platform ASoC: amd: ps: add soundwire machines for acp6.3 platform ASoC: amd: acp: move get_acp63_cpu_pin_id() to common file ASoC: amd: acp: add soundwire machine driver for legacy stack ASoC: amd: ps: fix the pcm device numbering for acp 6.3 platform sound/soc/amd/acp/Kconfig | 29 ++ sound/soc/amd/acp/Makefile | 4 + sound/soc/amd/acp/acp-sdw-legacy-mach.c | 486 +++++++++++++++++++++++ sound/soc/amd/acp/acp-sdw-mach-common.c | 64 +++ sound/soc/amd/acp/acp-sdw-sof-mach.c | 49 +-- sound/soc/amd/acp/amd-acp63-acpi-match.c | 54 +++ sound/soc/amd/acp/soc_amd_sdw_common.h | 4 + sound/soc/amd/mach-config.h | 1 + sound/soc/amd/ps/pci-ps.c | 1 + sound/soc/amd/ps/ps-sdw-dma.c | 2 + 10 files changed, 646 insertions(+), 48 deletions(-) create mode 100644 sound/soc/amd/acp/acp-sdw-legacy-mach.c create mode 100644 sound/soc/amd/acp/acp-sdw-mach-common.c -- 2.34.1

2 7

[GIT PULL] ASoC fixes for v6.12-rc7
by Mark Brown 14 Nov '24

14 Nov '24

The following changes since commit 48b86532c10128cf50c854a90c2d5b1410f4012d: ASoC: SOF: sof-client-probes-ipc4: Set param_size extension bits (2024-11-07 15:35:52 +0000) are available in the Git repository at: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git tags/asoc-fix-v6.12-rc7 for you to fetch changes up to 2ae6da569e34e1d26c5275442d17ffd75fd343b3: ASoC: max9768: Fix event generation for playback mute (2024-11-12 13:16:37 +0000) ---------------------------------------------------------------- ASoC: Fixes for v6.12 Some last updates for v6.12, one quirk plus a couple of fixes. One is a minor fix for a relatively obscure driver and the other is a relatively important fix for boot hangs with some audio graph based cards. ---------------------------------------------------------------- Deep Harsora (1): ASoC: intel: sof_sdw: add quirk for Dell SKU John Watts (1): ASoC: audio-graph-card2: Purge absent supplies for device tree nodes Mark Brown (1): ASoC: max9768: Fix event generation for playback mute sound/soc/codecs/max9768.c | 11 +++++++++-- sound/soc/generic/audio-graph-card2.c | 3 +++ sound/soc/intel/boards/sof_sdw.c | 8 ++++++++ 3 files changed, 20 insertions(+), 2 deletions(-)

2 1

src/Versions.in.in: Update *_tempo_base name
by GitHub pull_request - edited 14 Nov '24

14 Nov '24

alsa-project/alsa-lib pull request #421 was edited from nvinson: Change @SYMBOL_PREFIX@snd_has_tempo_base to @SYMBOL_PREFIX@snd_has_queue_tempo_base. Starting with version 1.2.13, alsa-lib fails to link with ld.lld-19 due to "version script assignment of 'ALSA_1.2.13' to symbol 'snd_seq_has_tempo_base' failed: symbol not defined". Per commit 769d1db1b0a213a39c7e59c0d1d724e7f45b1ac3 the correct name for the symbol is @SYMBOL_PREFIX@snd_has_queue_tempo_base; therefore, update src/Vesions.in.in to match. Fixes bug #420 Fixes Gentoo bug 943399 (https://bugs.gentoo.org/943399) Request URL : https://github.com/alsa-project/alsa-lib/pull/421 Patch URL : https://github.com/alsa-project/alsa-lib/pull/421.patch Repository URL: https://github.com/alsa-project/alsa-lib

1 0

src/Versions.in.in: Update *_tempo_base name
by GitHub pull_request - opened 14 Nov '24

14 Nov '24

alsa-project/alsa-lib pull request #421 was opened from nvinson: Change @SYMBOL_PREFIX@snd_has_tempo_base to @SYMBOL_PREFIX@snd_has_queue_tempo_base. Starting with version 1.2.13, alsa-lib fails to link with ld.lld-19 due to "version script assignment of 'ALSA_1.2.13' to symbol 'snd_seq_has_tempo_base' failed: symbol not defined". Per commit 769d1db1b0a213a39c7e59c0d1d724e7f45b1ac3 the correct name for the symbol is @SYMBOL_PREFIX@snd_has_queue_tempo_base; therefore, update src/Vesions.in.in to match. Fixes bug 420 Fixes Gentoo bug 943399 (https://bugs.gentoo.org/943399) Request URL : https://github.com/alsa-project/alsa-lib/pull/421 Patch URL : https://github.com/alsa-project/alsa-lib/pull/421.patch Repository URL: https://github.com/alsa-project/alsa-lib

1 0

alsa-lib-1.2.13: ld.lld: error: version script assignment of 'ALSA_1.2.13' to symbol 'snd_seq_has_tempo_base' failed: symbol not defined
by GitHub issues - opened 14 Nov '24

14 Nov '24

alsa-project/alsa-lib issue #420 was opened from nvinson: libasound.so fails to link with ld.lld with error ```ld.lld: error: version script assignment of 'ALSA_1.2.13' to symbol 'snd_seq_has_tempo_base' failed: symbol not defined``` Bug: [Gentoo bug 943399](https://bugs.gentoo.org/9433399) Issue URL : https://github.com/alsa-project/alsa-lib/issues/420 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

udevd[142]: GOTO 'alsa_restore_std' has no matching label in: '/usr/lib/udev/rules.d/90-alsa-restore.rules'
by GitHub issues - opened 14 Nov '24

14 Nov '24

alsa-project/alsa-utils issue #280 was opened from sxzzsf: There is no 'alsa_restore_std' label in the udev rule file, is it misspelled alsa_restore_go? Issue URL : https://github.com/alsa-project/alsa-utils/issues/280 Repository URL: https://github.com/alsa-project/alsa-utils

1 0

UCM2: Intel: sof-hda-dsp: Fix handling of empty sys_vendor string
by GitHub pull_request - opened 13 Nov '24

13 Nov '24

alsa-project/alsa-ucm-conf pull request #463 was opened from singalsu: The mistake in UCM syntax caused in alsaucm start error: ALSA lib ucm_cond.c:367:(if_eval) unknown If.Condition.Type Fixes: 13022a97711d ("sof-hda-dsp: Fix the case where sysfs dmi sys_vendor attribute is not set") Request URL : https://github.com/alsa-project/alsa-ucm-conf/pull/463 Patch URL : https://github.com/alsa-project/alsa-ucm-conf/pull/463.patch Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

memory leakage detected in alsa-lib-1.2.9 in Linux-5.10.24
by GitHub issues - edited 13 Nov '24

13 Nov '24

alsa-project/alsa-lib issue #419 was edited from wangtao13: I wrote an ALSA application using alsa-lib-1.2.9, the main logic of the application is ``` static int init_pcm_play(snd_pcm_t **playback_handle...) { snd_pcm_hw_params_t *hw_params; snd_pcm_format_t format; snd_pcm_uframes_t frames, periodsize; int ret; //1. openPCM， if (0 > snd_pcm_open(playback_handle, "default", SND_PCM_STREAM_PLAYBACK, 0)) { return -1; } //2. snd_pcm_hw_params_t if (0 > snd_pcm_hw_params_malloc (&hw_params)) { return -1; } //3. hw_params if (0 > snd_pcm_hw_params_any (*playback_handle, hw_params)) { return -1; } //4. if (0 > snd_pcm_hw_params_set_access (*playback_handle, hw_params, SND_PCM_ACCESS_RW_INTERLEAVED)) { return -1; } //5. SND_PCM_FORMAT_U8,8 if (8 == bits_per_sample) { format = SND_PCM_FORMAT_U8; } if (16 == bits_per_sample) { format = SND_PCM_FORMAT_S16_LE; } if (0 > snd_pcm_hw_params_set_format (*playback_handle, hw_params, format)) { return -1; } //6. if (0 > snd_pcm_hw_params_set_rate_near (*playback_handle, hw_params, &rate, 0)) { return -1; } //7. if (0 > snd_pcm_hw_params_set_channels(*playback_handle, hw_params, channels)) { return -1; } frames = 64; periodsize = frames; ret = snd_pcm_hw_params_set_buffer_size_near(*playback_handle, hw_params, &periodsize); if (ret < 0) { printf("Unable to set buffer size %li : %s\n", frames * 2, snd_strerror(ret)); } periodsize /= channels; ret = snd_pcm_hw_params_set_period_size_near(*playback_handle, hw_params, &periodsize, 0); if (ret < 0) { printf("Unable to set period size %li : %s\n", periodsize, snd_strerror(ret)); } //8. set hw_params if (0 > snd_pcm_hw_params (*playback_handle, hw_params)) { return -1; } snd_pcm_uframes_t temp; snd_pcm_hw_params_get_buffer_size(hw_params, &temp); printf("Got buffer size: %ld\n", temp); snd_pcm_hw_params_get_period_size(hw_params, &temp, 0); printf("Got period size: %ld\n", temp); snd_pcm_hw_params_free(hw_params); ret = snd_pcm_prepare(playback_handle); if (ret < 0) { printf("snd_pcm_prepare err\n"); } return ret; } int sound_play(playback_handle, ...) { rc = snd_pcm_writei(playback_handle, buf, buflen); if (rc < 0) { if (rc == -EAGAIN) { usleep(1000); return -1; } snd_pcm_prepare(playback_handle); return -1; } else if (rc == 0) { usleep(10 * 1000); } return 0; } int app_exit(....) { snd_pcm_close(*playback_handle); snd_config_update_free_global(); } #define TEST_LOOPS 2 /* > 1, can trigger memory leakage */ int main_test() { init_pcm_play(&playback_handle, ...., ..); for (int i=0; i<2; i++) { pcm_exit(); init_pcm_play(&playback_handle); sound_play(playback_handle) } return 0; } ``` ``` ==5176==ERROR: LeakSanitizer: detected memory leaks Direct leak of 12792 byte(s) in 39 object(s) allocated from: #0 0x77900d24 in __interceptor_calloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:95 #1 0x7626c458 (/usr/lib/libasound.so.2+0x4c458) Direct leak of 12 byte(s) in 1 object(s) allocated from: #0 0x77901ad8 in __interceptor_posix_memalign /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:155 #1 0x7758b3f4 in av_malloc (/usr/lib/libavutil.so.56+0x3b3f4) Indirect leak of 151552 byte(s) in 37 object(s) allocated from: #0 0x77900acc in __interceptor_malloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:86 #1 0x76281230 in snd_pcm_mmap (/usr/lib/libasound.so.2+0x61230) Indirect leak of 37392 byte(s) in 114 object(s) allocated from: #0 0x77900d24 in __interceptor_calloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:95 #1 0x7626c458 (/usr/lib/libasound.so.2+0x4c458) Indirect leak of 31200 byte(s) in 39 object(s) allocated from: #0 0x77900d24 in __interceptor_calloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:95 #1 0x762b91b8 in snd_pcm_softvol_open (/usr/lib/libasound.so.2+0x991b8) Indirect leak of 15808 byte(s) in 76 object(s) allocated from: #0 0x77900d24 in __interceptor_calloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:95 #1 0x76280d1c in snd_pcm_mmap (/usr/lib/libasound.so.2+0x60d1c) Indirect leak of 10032 byte(s) in 38 object(s) allocated from: #0 0x77900d24 in __interceptor_calloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:95 #1 0x762b4e28 (/usr/lib/libasound.so.2+0x94e28) Indirect leak of 5168 byte(s) in 38 object(s) allocated from: #0 0x77900acc in __interceptor_malloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:86 #1 0x7627e644 (/usr/lib/libasound.so.2+0x5e644) Indirect leak of 3344 byte(s) in 38 object(s) allocated from: #0 0x77900d24 in __interceptor_calloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:95 #1 0x7627e4e0 (/usr/lib/libasound.so.2+0x5e4e0) Indirect leak of 1976 byte(s) in 38 object(s) allocated from: #0 0x77900d24 in __interceptor_calloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:95 #1 0x762939ac in snd_pcm_plug_open (/usr/lib/libasound.so.2+0x739ac) Indirect leak of 1824 byte(s) in 76 object(s) allocated from: #0 0x77900d24 in __interceptor_calloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:95 #1 0x76280d3c in snd_pcm_mmap (/usr/lib/libasound.so.2+0x60d3c) Indirect leak of 1596 byte(s) in 38 object(s) allocated from: #0 0x7786d584 in __interceptor_strdup /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_interceptors.cc:405 #1 0x762c09b8 in snd_timer_hw_open (/usr/lib/libasound.so.2+0xa09b8) Indirect leak of 1560 byte(s) in 39 object(s) allocated from: #0 0x77900d24 in __interceptor_calloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:95 #1 0x762b7c58 (/usr/lib/libasound.so.2+0x97c58) Indirect leak of 1560 byte(s) in 39 object(s) allocated from: #0 0x77900d24 in __interceptor_calloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:95 #1 0x762512d4 (/usr/lib/libasound.so.2+0x312d4) Indirect leak of 1520 byte(s) in 38 object(s) allocated from: #0 0x77900d24 in __interceptor_calloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:95 #1 0x762c0978 in snd_timer_hw_open (/usr/lib/libasound.so.2+0xa0978) Indirect leak of 1419 byte(s) in 153 object(s) allocated from: #0 0x7786d584 in __interceptor_strdup /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_interceptors.cc:405 #1 0x7626c47c (/usr/lib/libasound.so.2+0x4c47c) Indirect leak of 468 byte(s) in 39 object(s) allocated from: #0 0x77900d24 in __interceptor_calloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:95 #1 0x762572e8 in snd_ctl_hw_open (/usr/lib/libasound.so.2+0x372e8) Indirect leak of 304 byte(s) in 76 object(s) allocated from: #0 0x779011a4 in __interceptor_realloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:105 #1 0x762690c4 (/usr/lib/libasound.so.2+0x490c4) Indirect leak of 273 byte(s) in 1 object(s) allocated from: #0 0x779011a4 in __interceptor_realloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:105 #1 0x77569f3c in av_buffer_realloc (/usr/lib/libavutil.so.56+0x19f3c) Indirect leak of 195 byte(s) in 39 object(s) allocated from: #0 0x7786d584 in __interceptor_strdup /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_interceptors.cc:405 #1 0x762512fc (/usr/lib/libasound.so.2+0x312fc) Indirect leak of 28 byte(s) in 1 object(s) allocated from: #0 0x77901ad8 in __interceptor_posix_memalign /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:155 #1 0x7758b3f4 in av_malloc (/usr/lib/libavutil.so.56+0x3b3f4) SUMMARY: AddressSanitizer: 280023 byte(s) leaked in 997 allocation(s). ``` If the `TEST_LOOPS` is defined as 1, there is NO memory leakage reported by ASAN. Issue URL : https://github.com/alsa-project/alsa-lib/issues/419 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

memory leakage detected in alsa-lib-1.2.9 in Linux-5.10.24
by GitHub issues - opened 13 Nov '24

13 Nov '24

alsa-project/alsa-lib issue #419 was opened from wangtao13: I wrote an ALSA application using alsa-lib-1.2.9, the main logic of the application is ``` static int init_pcm_play(snd_pcm_t **playback_handle...) { snd_pcm_hw_params_t *hw_params; snd_pcm_format_t format; snd_pcm_uframes_t frames, periodsize; int ret; //1. openPCM， if (0 > snd_pcm_open(playback_handle, "default", SND_PCM_STREAM_PLAYBACK, 0)) { return -1; } //2. snd_pcm_hw_params_t if (0 > snd_pcm_hw_params_malloc (&hw_params)) { return -1; } //3. hw_params if (0 > snd_pcm_hw_params_any (*playback_handle, hw_params)) { return -1; } //4. if (0 > snd_pcm_hw_params_set_access (*playback_handle, hw_params, SND_PCM_ACCESS_RW_INTERLEAVED)) { return -1; } //5. SND_PCM_FORMAT_U8,8 if (8 == bits_per_sample) { format = SND_PCM_FORMAT_U8; } if (16 == bits_per_sample) { format = SND_PCM_FORMAT_S16_LE; } if (0 > snd_pcm_hw_params_set_format (*playback_handle, hw_params, format)) { return -1; } //6. if (0 > snd_pcm_hw_params_set_rate_near (*playback_handle, hw_params, &rate, 0)) { return -1; } //7. if (0 > snd_pcm_hw_params_set_channels(*playback_handle, hw_params, channels)) { return -1; } frames = 64; periodsize = frames; ret = snd_pcm_hw_params_set_buffer_size_near(*playback_handle, hw_params, &periodsize); if (ret < 0) { printf("Unable to set buffer size %li : %s\n", frames * 2, snd_strerror(ret)); } periodsize /= channels; ret = snd_pcm_hw_params_set_period_size_near(*playback_handle, hw_params, &periodsize, 0); if (ret < 0) { printf("Unable to set period size %li : %s\n", periodsize, snd_strerror(ret)); } //8. set hw_params if (0 > snd_pcm_hw_params (*playback_handle, hw_params)) { return -1; } snd_pcm_uframes_t temp; snd_pcm_hw_params_get_buffer_size(hw_params, &temp); printf("Got buffer size: %ld\n", temp); snd_pcm_hw_params_get_period_size(hw_params, &temp, 0); printf("Got period size: %ld\n", temp); snd_pcm_hw_params_free(hw_params); ret = snd_pcm_prepare(playback_handle); if (ret < 0) { printf("snd_pcm_prepare err\n"); } return ret; } int sound_play(playback_handle, ...) { rc = snd_pcm_writei(playback_handle, buf, buflen); if (rc < 0) { if (rc == -EAGAIN) { usleep(1000); return -1; } snd_pcm_prepare(playback_handle); return -1; } else if (rc == 0) { usleep(10 * 1000); } return 0; } int app_exit(....) { snd_pcm_close(*playback_handle); snd_config_update_free_global(); } int main_test() { init_pcm_play(&playback_handle, ...., ..); for (int i=0; i<2; i++) { pcm_exit(); init_pcm_play(&playback_handle); sound_play(playback_handle) } return 0; } ``` ``` ==5176==ERROR: LeakSanitizer: detected memory leaks Direct leak of 12792 byte(s) in 39 object(s) allocated from: #0 0x77900d24 in __interceptor_calloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:95 #1 0x7626c458 (/usr/lib/libasound.so.2+0x4c458) Direct leak of 12 byte(s) in 1 object(s) allocated from: #0 0x77901ad8 in __interceptor_posix_memalign /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:155 #1 0x7758b3f4 in av_malloc (/usr/lib/libavutil.so.56+0x3b3f4) Indirect leak of 151552 byte(s) in 37 object(s) allocated from: #0 0x77900acc in __interceptor_malloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:86 #1 0x76281230 in snd_pcm_mmap (/usr/lib/libasound.so.2+0x61230) Indirect leak of 37392 byte(s) in 114 object(s) allocated from: #0 0x77900d24 in __interceptor_calloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:95 #1 0x7626c458 (/usr/lib/libasound.so.2+0x4c458) Indirect leak of 31200 byte(s) in 39 object(s) allocated from: #0 0x77900d24 in __interceptor_calloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:95 #1 0x762b91b8 in snd_pcm_softvol_open (/usr/lib/libasound.so.2+0x991b8) Indirect leak of 15808 byte(s) in 76 object(s) allocated from: #0 0x77900d24 in __interceptor_calloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:95 #1 0x76280d1c in snd_pcm_mmap (/usr/lib/libasound.so.2+0x60d1c) Indirect leak of 10032 byte(s) in 38 object(s) allocated from: #0 0x77900d24 in __interceptor_calloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:95 #1 0x762b4e28 (/usr/lib/libasound.so.2+0x94e28) Indirect leak of 5168 byte(s) in 38 object(s) allocated from: #0 0x77900acc in __interceptor_malloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:86 #1 0x7627e644 (/usr/lib/libasound.so.2+0x5e644) Indirect leak of 3344 byte(s) in 38 object(s) allocated from: #0 0x77900d24 in __interceptor_calloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:95 #1 0x7627e4e0 (/usr/lib/libasound.so.2+0x5e4e0) Indirect leak of 1976 byte(s) in 38 object(s) allocated from: #0 0x77900d24 in __interceptor_calloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:95 #1 0x762939ac in snd_pcm_plug_open (/usr/lib/libasound.so.2+0x739ac) Indirect leak of 1824 byte(s) in 76 object(s) allocated from: #0 0x77900d24 in __interceptor_calloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:95 #1 0x76280d3c in snd_pcm_mmap (/usr/lib/libasound.so.2+0x60d3c) Indirect leak of 1596 byte(s) in 38 object(s) allocated from: #0 0x7786d584 in __interceptor_strdup /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_interceptors.cc:405 #1 0x762c09b8 in snd_timer_hw_open (/usr/lib/libasound.so.2+0xa09b8) Indirect leak of 1560 byte(s) in 39 object(s) allocated from: #0 0x77900d24 in __interceptor_calloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:95 #1 0x762b7c58 (/usr/lib/libasound.so.2+0x97c58) Indirect leak of 1560 byte(s) in 39 object(s) allocated from: #0 0x77900d24 in __interceptor_calloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:95 #1 0x762512d4 (/usr/lib/libasound.so.2+0x312d4) Indirect leak of 1520 byte(s) in 38 object(s) allocated from: #0 0x77900d24 in __interceptor_calloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:95 #1 0x762c0978 in snd_timer_hw_open (/usr/lib/libasound.so.2+0xa0978) Indirect leak of 1419 byte(s) in 153 object(s) allocated from: #0 0x7786d584 in __interceptor_strdup /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_interceptors.cc:405 #1 0x7626c47c (/usr/lib/libasound.so.2+0x4c47c) Indirect leak of 468 byte(s) in 39 object(s) allocated from: #0 0x77900d24 in __interceptor_calloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:95 #1 0x762572e8 in snd_ctl_hw_open (/usr/lib/libasound.so.2+0x372e8) Indirect leak of 304 byte(s) in 76 object(s) allocated from: #0 0x779011a4 in __interceptor_realloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:105 #1 0x762690c4 (/usr/lib/libasound.so.2+0x490c4) Indirect leak of 273 byte(s) in 1 object(s) allocated from: #0 0x779011a4 in __interceptor_realloc /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:105 #1 0x77569f3c in av_buffer_realloc (/usr/lib/libavutil.so.56+0x19f3c) Indirect leak of 195 byte(s) in 39 object(s) allocated from: #0 0x7786d584 in __interceptor_strdup /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_interceptors.cc:405 #1 0x762512fc (/usr/lib/libasound.so.2+0x312fc) Indirect leak of 28 byte(s) in 1 object(s) allocated from: #0 0x77901ad8 in __interceptor_posix_memalign /home/users/at/work/arm-gnu-toolchain/arm-gcc/libsanitizer/asan/asan_malloc_linux.cc:155 #1 0x7758b3f4 in av_malloc (/usr/lib/libavutil.so.56+0x3b3f4) SUMMARY: AddressSanitizer: 280023 byte(s) leaked in 997 allocation(s). ``` Issue URL : https://github.com/alsa-project/alsa-lib/issues/419 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

[PATCH] Revert "aplay: fix S24_LE wav header"
by Shengjiu Wang 13 Nov '24

13 Nov '24

S24_LE means the lower 24bit in 32bit space has valid data. So previous code is correct. The commit causes regression, then revert it. This reverts commit 9cdcbe0e932b97fac88a6195f87647159a6c7194. Fixes: 9cdcbe0e932b ("aplay: fix S24_LE wav header") Signed-off-by: Shengjiu Wang <shengjiu.wang(a)nxp.com> --- aplay/aplay.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/aplay/aplay.c b/aplay/aplay.c index c58c2e9..d11e63a 100644 --- a/aplay/aplay.c +++ b/aplay/aplay.c @@ -2719,11 +2719,11 @@ static void begin_wave(int fd, size_t cnt) case SND_PCM_FORMAT_S16_LE: bits = 16; break; - case SND_PCM_FORMAT_S24_LE: /* S24_LE is 24 bits stored in 32 bit width with 8 bit padding */ case SND_PCM_FORMAT_S32_LE: - case SND_PCM_FORMAT_FLOAT_LE: + case SND_PCM_FORMAT_FLOAT_LE: bits = 32; break; + case SND_PCM_FORMAT_S24_LE: case SND_PCM_FORMAT_S24_3LE: bits = 24; break; -- 2.34.1

1 0