Commit 68110661 authored by Takashi Iwai's avatar Takashi Iwai

ALSA: ctxfi - Fix uninitialized error checks

Fix a few uninitialized error checks that were introduced recently
mistakenlly during the clean-up:
  sound/pci/ctxfi/ctamixer.c: In function ‘get_amixer_rsc’:
  sound/pci/ctxfi/ctamixer.c:261: warning: ‘err’ may be used uninitialized in this function
  sound/pci/ctxfi/ctamixer.c: In function ‘get_sum_rsc’:
  sound/pci/ctxfi/ctamixer.c:415: warning: ‘err’ may be used uninitialized in this function
  sound/pci/ctxfi/ctsrc.c: In function ‘get_srcimp_rsc’:
  sound/pci/ctxfi/ctsrc.c:742: warning: ‘err’ may be used uninitialized in this function
Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent 55fe27f7
...@@ -242,13 +242,12 @@ static int get_amixer_rsc(struct amixer_mgr *mgr, ...@@ -242,13 +242,12 @@ static int get_amixer_rsc(struct amixer_mgr *mgr,
/* Allocate mem for amixer resource */ /* Allocate mem for amixer resource */
amixer = kzalloc(sizeof(*amixer), GFP_KERNEL); amixer = kzalloc(sizeof(*amixer), GFP_KERNEL);
if (NULL == amixer) { if (!amixer)
err = -ENOMEM; return -ENOMEM;
return err;
}
/* Check whether there are sufficient /* Check whether there are sufficient
* amixer resources to meet request. */ * amixer resources to meet request. */
err = 0;
spin_lock_irqsave(&mgr->mgr_lock, flags); spin_lock_irqsave(&mgr->mgr_lock, flags);
for (i = 0; i < desc->msr; i++) { for (i = 0; i < desc->msr; i++) {
err = mgr_get_resource(&mgr->mgr, 1, &idx); err = mgr_get_resource(&mgr->mgr, 1, &idx);
...@@ -397,12 +396,11 @@ static int get_sum_rsc(struct sum_mgr *mgr, ...@@ -397,12 +396,11 @@ static int get_sum_rsc(struct sum_mgr *mgr,
/* Allocate mem for sum resource */ /* Allocate mem for sum resource */
sum = kzalloc(sizeof(*sum), GFP_KERNEL); sum = kzalloc(sizeof(*sum), GFP_KERNEL);
if (NULL == sum) { if (!sum)
err = -ENOMEM; return -ENOMEM;
return err;
}
/* Check whether there are sufficient sum resources to meet request. */ /* Check whether there are sufficient sum resources to meet request. */
err = 0;
spin_lock_irqsave(&mgr->mgr_lock, flags); spin_lock_irqsave(&mgr->mgr_lock, flags);
for (i = 0; i < desc->msr; i++) { for (i = 0; i < desc->msr; i++) {
err = mgr_get_resource(&mgr->mgr, 1, &idx); err = mgr_get_resource(&mgr->mgr, 1, &idx);
......
...@@ -724,12 +724,11 @@ static int get_srcimp_rsc(struct srcimp_mgr *mgr, ...@@ -724,12 +724,11 @@ static int get_srcimp_rsc(struct srcimp_mgr *mgr,
/* Allocate mem for SRCIMP resource */ /* Allocate mem for SRCIMP resource */
srcimp = kzalloc(sizeof(*srcimp), GFP_KERNEL); srcimp = kzalloc(sizeof(*srcimp), GFP_KERNEL);
if (NULL == srcimp) { if (!srcimp)
err = -ENOMEM; return -ENOMEM;
return err;
}
/* Check whether there are sufficient SRCIMP resources. */ /* Check whether there are sufficient SRCIMP resources. */
err = 0;
spin_lock_irqsave(&mgr->mgr_lock, flags); spin_lock_irqsave(&mgr->mgr_lock, flags);
for (i = 0; i < desc->msr; i++) { for (i = 0; i < desc->msr; i++) {
err = mgr_get_resource(&mgr->mgr, 1, &idx); err = mgr_get_resource(&mgr->mgr, 1, &idx);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment