diff options
| author | xians@chromium.org <xians@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-09-15 03:59:35 +0000 |
|---|---|---|
| committer | xians@chromium.org <xians@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-09-15 03:59:35 +0000 |
| commit | 7d5ad4239974cad5843fb605488a8c833716e56c (patch) | |
| tree | 69f4238224ecc2495f5a1fdf7ca027e4aa9c434a /media/audio/linux/alsa_wrapper.cc | |
| parent | e1ece30375b913301c3a1c85abe13e3ef51a0f44 (diff) | |
| download | chromium_src-7d5ad4239974cad5843fb605488a8c833716e56c.zip chromium_src-7d5ad4239974cad5843fb605488a8c833716e56c.tar.gz chromium_src-7d5ad4239974cad5843fb605488a8c833716e56c.tar.bz2 | |
There is a complain from Valgrind about invalid memory access in snd_device_name_hint(-1, ..) // -1 means all cards.
It looks like the problem is actually because we use -1 to loop through all the soundcards.
And this patch will loop through the soundcard manually, which should resolve the Valgrind problem.
Bug=96207
Test=media_unittests with Valgrind
Review URL: http://codereview.chromium.org/7888011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101240 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'media/audio/linux/alsa_wrapper.cc')
| -rw-r--r-- | media/audio/linux/alsa_wrapper.cc | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/media/audio/linux/alsa_wrapper.cc b/media/audio/linux/alsa_wrapper.cc index 563bccb..69b703e 100644 --- a/media/audio/linux/alsa_wrapper.cc +++ b/media/audio/linux/alsa_wrapper.cc @@ -29,6 +29,10 @@ int AlsaWrapper::DeviceNameFreeHint(void** hints) { return snd_device_name_free_hint(hints); } +int AlsaWrapper::CardNext(int* rcard) { + return snd_card_next(rcard); +} + int AlsaWrapper::PcmClose(snd_pcm_t* handle) { return snd_pcm_close(handle); } |