(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@xxxxxxxxxx> Reviewd-by: Takashi Sakamoto <o-takashi@xxxxxxxxxxxxx> Tested-by: Takashi Sakamoto <o-takashi@xxxxxxxxxxxxx> > 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; -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html