ChangeSet 1.2231.1.234, 2005/03/28 20:21:23-08:00, [EMAIL PROTECTED]

        [PATCH] dvb: vfree() checking cleanups
        
        vfree() checking cleanups.
        
        Signed-off by: James Lamanna <[EMAIL PROTECTED]>
        Signed-off-by: Domen Puncer <[EMAIL PROTECTED]>
        Signed-off-by: Johannes Stezenbach <[EMAIL PROTECTED]>
        Signed-off-by: Andrew Morton <[EMAIL PROTECTED]>
        Signed-off-by: Linus Torvalds <[EMAIL PROTECTED]>



 dvb-core/dmxdev.c         |   16 +++++-----------
 dvb-core/dvb_ca_en50221.c |    3 +--
 ttpci/budget-core.c       |    3 +--
 3 files changed, 7 insertions(+), 15 deletions(-)


diff -Nru a/drivers/media/dvb/dvb-core/dmxdev.c 
b/drivers/media/dvb/dvb-core/dmxdev.c
--- a/drivers/media/dvb/dvb-core/dmxdev.c       2005-03-28 22:00:27 -08:00
+++ b/drivers/media/dvb/dvb-core/dmxdev.c       2005-03-28 22:00:27 -08:00
@@ -304,8 +304,7 @@
        buf->size=size;
        buf->pwrite=buf->pread=0;
        spin_unlock_irq(&dmxdevfilter->dev->lock);
-       if (mem)
-               vfree(mem);
+       vfree(mem);
 
        if (buf->size) {
                mem=vmalloc(dmxdevfilter->buffer.size);
@@ -1129,15 +1128,10 @@
        dvb_unregister_device(dmxdev->dvbdev);
        dvb_unregister_device(dmxdev->dvr_dvbdev);
 
-       if (dmxdev->filter) {
-               vfree(dmxdev->filter);
-               dmxdev->filter=NULL;
-       }
-
-       if (dmxdev->dvr) {
-               vfree(dmxdev->dvr);
-               dmxdev->dvr=NULL;
-       }
+       vfree(dmxdev->filter);
+       dmxdev->filter=NULL;
+       vfree(dmxdev->dvr);
+       dmxdev->dvr=NULL;
        dmxdev->demux->close(dmxdev->demux);
 }
 EXPORT_SYMBOL(dvb_dmxdev_release);
diff -Nru a/drivers/media/dvb/dvb-core/dvb_ca_en50221.c 
b/drivers/media/dvb/dvb-core/dvb_ca_en50221.c
--- a/drivers/media/dvb/dvb-core/dvb_ca_en50221.c       2005-03-28 22:00:27 
-08:00
+++ b/drivers/media/dvb/dvb-core/dvb_ca_en50221.c       2005-03-28 22:00:27 
-08:00
@@ -804,8 +804,7 @@
        down_write(&ca->slot_info[slot].sem);
        ca->pub->slot_shutdown(ca->pub, slot);
        ca->slot_info[slot].slot_state = DVB_CA_SLOTSTATE_NONE;
-       if (ca->slot_info[slot].rx_buffer.data)
-               vfree(ca->slot_info[slot].rx_buffer.data);
+       vfree(ca->slot_info[slot].rx_buffer.data);
        ca->slot_info[slot].rx_buffer.data = NULL;
        up_write(&ca->slot_info[slot].sem);
 
diff -Nru a/drivers/media/dvb/ttpci/budget-core.c 
b/drivers/media/dvb/ttpci/budget-core.c
--- a/drivers/media/dvb/ttpci/budget-core.c     2005-03-28 22:00:27 -08:00
+++ b/drivers/media/dvb/ttpci/budget-core.c     2005-03-28 22:00:27 -08:00
@@ -415,8 +415,7 @@
 err:
        i2c_del_adapter(&budget->i2c_adap);
 
-       if (budget->grabbing)
-               vfree(budget->grabbing);
+       vfree(budget->grabbing);
 
        dvb_unregister_adapter(budget->dvb_adapter);
 
-
To unsubscribe from this list: send the line "unsubscribe bk-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to