Fixes 'Unchecked return value' 29/263229/3 accepted/tizen/6.5/unified/20211028.124248 accepted/tizen/unified/20210831.143834 submit/tizen/20210830.082951 submit/tizen_6.5/20211028.162401 tizen_6.5.m2_release
authorJaechul Lee <jcsing.lee@samsung.com>
Mon, 30 Aug 2021 02:08:00 +0000 (11:08 +0900)
committerJaechul Lee <jcsing.lee@samsung.com>
Mon, 30 Aug 2021 03:19:55 +0000 (12:19 +0900)
[Version] 0.13.13
[Issue Type] Coverity

Change-Id: Ibe7d20fc4dbc01986cd44e72827e2061924c4014
Signed-off-by: Jaechul Lee <jcsing.lee@samsung.com>
aec/loopback.c
packaging/libmm-sound.spec

index 3a6b63bc6ec5b4e5d639f2afe94d3f145fd2fb23..3d21ee272ccb64de53e10a0378dc7bdecca1ce0a 100644 (file)
@@ -805,8 +805,14 @@ static int __open_dev(loopback_t *loopback, int index, struct pollfd *pfd)
                goto fail;
        }
 
-       snd_pcm_hw_params_get_buffer_size(hw_params, &buffer_size);
-       snd_pcm_hw_params_get_buffer_time(hw_params, &buffer_time, NULL);
+       if ((err = snd_pcm_hw_params_get_buffer_size(hw_params, &buffer_size)) < 0) {
+               LOGE("%s: failed to get buffer size (%s)\n", name, snd_strerror(err));
+               goto fail;
+       }
+       if ((err = snd_pcm_hw_params_get_buffer_time(hw_params, &buffer_time, NULL)) < 0) {
+               LOGE("%s: failed to get buffer time (%s)\n", name, snd_strerror(err));
+               goto fail;
+       }
 
        unsigned int period_time = buffer_time / 4;
        if ((err = snd_pcm_hw_params_set_period_time_near(dev->handle, hw_params, &period_time, NULL))) {
index a2f1879766006bd3a43941a0bb3ab960264f6a44..1f3ba304bcb9720ceb4d79662398c22b0e17d1f1 100644 (file)
@@ -1,6 +1,6 @@
 Name:       libmm-sound
 Summary:    MMSound Package contains client lib and focus server binary
-Version:    0.13.12
+Version:    0.13.13
 Release:    0
 Group:      System/Libraries
 License:    Apache-2.0