Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1180674 > unrolled thread
| Started by | Takashi Iwai <tiwai@suse.de> |
|---|---|
| First post | 2015-07-09 14:30 +0200 |
| Last post | 2015-07-09 14:30 +0200 |
| Articles | 1 — 1 participant |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH] ALSA: hda: Delete an unnecessary check before the function call "kobject_put" Takashi Iwai <tiwai@suse.de> - 2015-07-09 14:30 +0200
| From | Takashi Iwai <tiwai@suse.de> |
|---|---|
| Date | 2015-07-09 14:30 +0200 |
| Subject | Re: [PATCH] ALSA: hda: Delete an unnecessary check before the function call "kobject_put" |
| Message-ID | <pKj7J-6i-29@gated-at.bofh.it> |
On Wed, 08 Jul 2015 21:31:57 +0200, SF Markus Elfring wrote: > > From: Markus Elfring <elfring@users.sourceforge.net> > Date: Wed, 8 Jul 2015 21:26:02 +0200 > > The kobject_put() function tests whether its argument is NULL and then > returns immediately. Thus the test around the call is not needed. > > This issue was detected by using the Coccinelle software. > > Signed-off-by: Markus Elfring <elfring@users.sourceforge.net> Applied, thanks. Takashi > --- > sound/hda/hdac_sysfs.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/sound/hda/hdac_sysfs.c b/sound/hda/hdac_sysfs.c > index 0a6ce3b..089b35f 100644 > --- a/sound/hda/hdac_sysfs.c > +++ b/sound/hda/hdac_sysfs.c > @@ -321,8 +321,7 @@ static void widget_tree_free(struct hdac_device *codec) > free_widget_node(*p, &widget_node_group); > kfree(tree->nodes); > } > - if (tree->root) > - kobject_put(tree->root); > + kobject_put(tree->root); > kfree(tree); > codec->widgets = NULL; > } > -- > 2.4.5 > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Back to top | Article view | linux.kernel
csiph-web