[alsa-devel] [patch 2/2] ALSA: fireworks: small leak on error path
There was a typo here so we return directly instead of freeing "hwinfo".
Signed-off-by: Dan Carpenter dan.carpenter@oracle.com
diff --git a/sound/firewire/fireworks/fireworks.c b/sound/firewire/fireworks/fireworks.c index 8e856cc..996fdc4 100644 --- a/sound/firewire/fireworks/fireworks.c +++ b/sound/firewire/fireworks/fireworks.c @@ -157,7 +157,7 @@ get_hardware_info(struct snd_efw *efw) /* Hardware metering. */ if (hwinfo->phys_in_grp_count > HWINFO_MAX_CAPS_GROUPS || hwinfo->phys_out_grp_count > HWINFO_MAX_CAPS_GROUPS) { - return -EIO; + err = -EIO; goto end; } efw->phys_in = hwinfo->phys_in;
(May 29 2014 01:42), Dan Carpenter wrote:
There was a typo here so we return directly instead of freeing "hwinfo".
Signed-off-by: Dan Carpenter dan.carpenter@oracle.com
Reviewd-by: Takashi Sakamoto o-takashi@sakamocchi.jp Tested-by: Takashi Sakamoto o-takashi@sakamocchi.jp
diff --git a/sound/firewire/fireworks/fireworks.c b/sound/firewire/fireworks/fireworks.c index 8e856cc..996fdc4 100644 --- a/sound/firewire/fireworks/fireworks.c +++ b/sound/firewire/fireworks/fireworks.c @@ -157,7 +157,7 @@ get_hardware_info(struct snd_efw *efw) /* Hardware metering. */ if (hwinfo->phys_in_grp_count > HWINFO_MAX_CAPS_GROUPS || hwinfo->phys_out_grp_count > HWINFO_MAX_CAPS_GROUPS) {
return -EIO;
goto end; } efw->phys_in = hwinfo->phys_in;err = -EIO;
At Wed, 28 May 2014 19:42:46 +0300, Dan Carpenter wrote:
There was a typo here so we return directly instead of freeing "hwinfo".
Signed-off-by: Dan Carpenter dan.carpenter@oracle.com
Applied, thanks.
Takashi
diff --git a/sound/firewire/fireworks/fireworks.c b/sound/firewire/fireworks/fireworks.c index 8e856cc..996fdc4 100644 --- a/sound/firewire/fireworks/fireworks.c +++ b/sound/firewire/fireworks/fireworks.c @@ -157,7 +157,7 @@ get_hardware_info(struct snd_efw *efw) /* Hardware metering. */ if (hwinfo->phys_in_grp_count > HWINFO_MAX_CAPS_GROUPS || hwinfo->phys_out_grp_count > HWINFO_MAX_CAPS_GROUPS) {
return -EIO;
goto end; } efw->phys_in = hwinfo->phys_in;err = -EIO;
participants (3)
-
Dan Carpenter
-
Takashi Iwai
-
Takashi Sakamoto