staging: bcm: Remove NULL check before kfree
authorIlia Mirkin <imirkin@alum.mit.edu>
Sun, 13 Mar 2011 05:28:55 +0000 (00:28 -0500)
committerGreg Kroah-Hartman <gregkh@suse.de>
Mon, 14 Mar 2011 18:57:30 +0000 (11:57 -0700)
This patch was generated by the following semantic patch:
// <smpl>
@@ expression E; @@
- if (E != NULL) { kfree(E); }
+ kfree(E);

@@ expression E; @@
- if (E != NULL) { kfree(E); E = NULL; }
+ kfree(E);
+ E = NULL;
// </smpl>

Signed-off-by: Ilia Mirkin <imirkin@alum.mit.edu>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/bcm/CmHost.c

index 017b471..9be184f 100644 (file)
@@ -974,11 +974,7 @@ static VOID CopyToAdapter( register PMINI_ADAPTER Adapter,         /**<Pointer to the A
                        !(psfLocalSet->u8RequesttransmissionPolicy &
                                MASK_DISABLE_HEADER_SUPPRESSION);
 
-       if(Adapter->PackInfo[uiSearchRuleIndex].pstSFIndication)
-       {
-               kfree(Adapter->PackInfo[uiSearchRuleIndex].pstSFIndication);
-               Adapter->PackInfo[uiSearchRuleIndex].pstSFIndication = NULL;
-       }
+       kfree(Adapter->PackInfo[uiSearchRuleIndex].pstSFIndication);
        Adapter->PackInfo[uiSearchRuleIndex].pstSFIndication = pstAddIndication;
 
        //Re Sort the SF list in PackInfo according to Traffic Priority
@@ -1971,10 +1967,7 @@ INT AllocAdapterDsxBuffer(PMINI_ADAPTER Adapter)
 
 INT FreeAdapterDsxBuffer(PMINI_ADAPTER Adapter)
 {
-       if(Adapter->caDsxReqResp)
-       {
-               kfree(Adapter->caDsxReqResp);
-       }
+       kfree(Adapter->caDsxReqResp);
        return 0;
 
 }