Use the FREE_AND_SET_NULL macro where appropriate

src/battle_anim_utility_funcs.c

@@ -594,8 +594,7 @@ static void StatsChangeAnimation_Step3(u8 taskId)
         if (gTasks[taskId].data[6] == 1)
             gSprites[gTasks[taskId].data[7]].oam.priority++;
 
-        Free(sAnimStatsChangeData);
+        FREE_AND_SET_NULL(sAnimStatsChangeData);
-        sAnimStatsChangeData = NULL;
         DestroyAnimVisualTask(taskId);
         break;
     }

src/battle_main.c

@@ -1498,8 +1498,7 @@ static void CB2_PreInitMultiBattle(void)
                 gBattleTypeFlags = *savedBattleTypeFlags;
                 gMain.savedCallback = *savedCallback;
                 SetMainCallback2(CB2_InitBattleInternal);
-                Free(sMultiPartnerPartyBuffer);
+                FREE_AND_SET_NULL(sMultiPartnerPartyBuffer);
-                sMultiPartnerPartyBuffer = NULL;
             }
         }
         else if (gReceivedRemoteLinkPlayers == 0)
@@ -1507,8 +1506,7 @@ static void CB2_PreInitMultiBattle(void)
             gBattleTypeFlags = *savedBattleTypeFlags;
             gMain.savedCallback = *savedCallback;
             SetMainCallback2(CB2_InitBattleInternal);
-            Free(sMultiPartnerPartyBuffer);
+            FREE_AND_SET_NULL(sMultiPartnerPartyBuffer);
-            sMultiPartnerPartyBuffer = NULL;
         }
         break;
     }
@@ -1544,8 +1542,7 @@ static void CB2_PreInitIngamePlayerPartnerBattle(void)
             gBattleTypeFlags = *savedBattleTypeFlags;
             gMain.savedCallback = *savedCallback;
             SetMainCallback2(CB2_InitBattleInternal);
-            Free(sMultiPartnerPartyBuffer);
+            FREE_AND_SET_NULL(sMultiPartnerPartyBuffer);
-            sMultiPartnerPartyBuffer = NULL;
         }
         break;
     }

src/evolution_scene.c

@@ -816,8 +816,7 @@ static void Task_EvolutionScene(u8 taskId)
 
             DestroyTask(taskId);
             FreeMonSpritesGfx();
-            Free(sEvoStructPtr);
+            FREE_AND_SET_NULL(sEvoStructPtr);
-            sEvoStructPtr = NULL;
             FreeAllWindowBuffers();
             SetMainCallback2(gCB2_AfterEvolution);
         }
@@ -1223,8 +1222,7 @@ static void Task_TradeEvolutionScene(u8 taskId)
         if (!IsTextPrinterActive(0))
         {
             DestroyTask(taskId);
-            Free(sEvoStructPtr);
+            FREE_AND_SET_NULL(sEvoStructPtr);
-            sEvoStructPtr = NULL;
             gTextFlags.useAlternateDownArrow = 0;
             SetMainCallback2(gCB2_AfterEvolution);
         }

src/mystery_gift_client.c

@@ -44,8 +44,7 @@ u32 MysteryGiftClient_Run(u16 * endVal)
     {
         *endVal = sClient->param;
         MysteryGiftClient_Free(sClient);
-        Free(sClient);
+        FREE_AND_SET_NULL(sClient);
-        sClient = NULL;
     }
     return result;
 }

src/mystery_gift_server.c

@@ -44,8 +44,7 @@ u32 MysterGiftServer_Run(u16 * endVal)
     {
         *endVal = sServer->param;
         MysteryGiftServer_Free(sServer);
-        Free(sServer);
+        FREE_AND_SET_NULL(sServer);
-        sServer = NULL;
     }
     return result;
 }

src/mystery_gift_view.c

@@ -207,8 +207,7 @@ void WonderCard_Destroy(void)
     if (sWonderCardData != NULL)
     {
         *sWonderCardData = (struct WonderCardData){};
-        Free(sWonderCardData);
+        FREE_AND_SET_NULL(sWonderCardData);
-        sWonderCardData = NULL;
     }
 }
 
@@ -660,8 +659,7 @@ void WonderNews_Destroy(void)
     if (sWonderNewsData != NULL)
     {
         *sWonderNewsData = (struct WonderNewsData){};
-        Free(sWonderNewsData);
+        FREE_AND_SET_NULL(sWonderNewsData);
-        sWonderNewsData = NULL;
     }
 }