Staging: bcm: PHSModule.c: Replaced member accessing with variable in free_phs_service()
authorMatthias Beyer <mail@beyermatthias.de>
Tue, 15 Jul 2014 07:43:05 +0000 (09:43 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 15 Jul 2014 15:09:48 +0000 (08:09 -0700)
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/bcm/PHSModule.c

index e3db972d5b35e90ac6f81306faddfa38965d5d50..d2aa9c911d1769b07a65013d217ca92a514082a7 100644 (file)
@@ -889,6 +889,8 @@ static void free_phs_serviceflow_rules(struct bcm_phs_table *psServiceFlowRulesT
 {
        int i, j;
        struct bcm_mini_adapter *Adapter = GET_BCM_ADAPTER(gblpnetdev);
+       struct bcm_phs_classifier_entry *curr_act_rules_list;
+       struct bcm_phs_classifier_entry *curr_old_rules_list;
 
        BCM_DEBUG_PRINT(Adapter, DBG_TYPE_OTHERS, PHS_DISPATCH, DBG_LVL_ALL,
                        "=======>\n");
@@ -902,26 +904,32 @@ static void free_phs_serviceflow_rules(struct bcm_phs_table *psServiceFlowRulesT
 
                        if (pstClassifierRulesTable) {
                                for (j = 0; j < MAX_PHSRULE_PER_SF; j++) {
-                                       if (pstClassifierRulesTable->stActivePhsRulesList[j].pstPhsRule) {
+                                       curr_act_rules_list =
+                                               &pstClassifierRulesTable->stActivePhsRulesList[j];
 
-                                               if (pstClassifierRulesTable->stActivePhsRulesList[j].pstPhsRule->u8RefCnt)
-                                                       pstClassifierRulesTable->stActivePhsRulesList[j].pstPhsRule->u8RefCnt--;
+                                       curr_old_rules_list =
+                                               &pstClassifierRulesTable->stOldPhsRulesList[j];
 
-                                               if (0 == pstClassifierRulesTable->stActivePhsRulesList[j].pstPhsRule->u8RefCnt)
-                                                       kfree(pstClassifierRulesTable->stActivePhsRulesList[j].pstPhsRule);
+                                       if (curr_act_rules_list->pstPhsRule) {
 
-                                               pstClassifierRulesTable->stActivePhsRulesList[j].pstPhsRule = NULL;
+                                               if (curr_act_rules_list->pstPhsRule->u8RefCnt)
+                                                       curr_act_rules_list->pstPhsRule->u8RefCnt--;
+
+                                               if (0 == curr_act_rules_list->pstPhsRule->u8RefCnt)
+                                                       kfree(curr_act_rules_list->pstPhsRule);
+
+                                               curr_act_rules_list->pstPhsRule = NULL;
                                        }
 
-                                       if (pstClassifierRulesTable->stOldPhsRulesList[j].pstPhsRule) {
+                                       if (curr_old_rules_list->pstPhsRule) {
 
-                                               if (pstClassifierRulesTable->stOldPhsRulesList[j].pstPhsRule->u8RefCnt)
-                                                       pstClassifierRulesTable->stOldPhsRulesList[j].pstPhsRule->u8RefCnt--;
+                                               if (curr_old_rules_list->pstPhsRule->u8RefCnt)
+                                                       curr_old_rules_list->pstPhsRule->u8RefCnt--;
 
-                                               if (0 == pstClassifierRulesTable->stOldPhsRulesList[j].pstPhsRule->u8RefCnt)
-                                                       kfree(pstClassifierRulesTable->stOldPhsRulesList[j].pstPhsRule);
+                                               if (0 == curr_old_rules_list->pstPhsRule->u8RefCnt)
+                                                       kfree(curr_old_rules_list->pstPhsRule);
 
-                                               pstClassifierRulesTable->stOldPhsRulesList[j].pstPhsRule = NULL;
+                                               curr_old_rules_list->pstPhsRule = NULL;
                                        }
                                }
                                kfree(pstClassifierRulesTable);