[PATCH][OSS] Don't check pointers about to be vfree()'d for NULL

There's no need to check pointers for NULL before handing them to vfree().


Signed-off-by: Jesper Juhl <[email protected]>
---

 sound/oss/sequencer.c |   13 +++----------
 1 files changed, 3 insertions(+), 10 deletions(-)

--- linux-2.6.16-rc2-git1-orig/sound/oss/sequencer.c	2006-01-03 04:21:10.000000000 +0100
+++ linux-2.6.16-rc2-git1/sound/oss/sequencer.c	2006-02-04 20:51:49.000000000 +0100
@@ -1671,14 +1671,7 @@
 
 void sequencer_unload(void)
 {
-	if(queue)
-	{
-		vfree(queue);
-		queue=NULL;
-	}
-	if(iqueue)
-	{
-		vfree(iqueue);
-		iqueue=NULL;
-	}
+	vfree(queue);
+	vfree(iqueue);
+	queue = iqueue = NULL;
 }


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• Prev by Date: Re: athlon 64 dual core tsc out of sync
• Next by Date: Re: chroot in swsusp userland interface (was: Re: [Suspend2-devel] Re: [ 00/10] [Suspend2] Modules support.)
• Previous by thread: in crease ener gy
• Next by thread: [PATCH][OSS] no need to check vfree() argument for NULL
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]