fix: allow null to be passed to free functions

BinaryAlien · Dec 1, 2024 · f1e8050 · f1e8050
1 parent dc20bb1
commit f1e8050
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 0 deletions.
diff --git a/src/a2s/info.c b/src/a2s/info.c
@@ -123,6 +123,8 @@ A2S_INFO *ssq_info(SSQ_SERVER *server) {
 }
 
 void ssq_info_free(A2S_INFO *info) {
+    if (info == NULL)
+        return;
     free(info->name);
     free(info->map);
     free(info->folder);

diff --git a/src/a2s/player.c b/src/a2s/player.c
@@ -80,6 +80,8 @@ A2S_PLAYER *ssq_player(SSQ_SERVER *server, uint8_t *player_count) {
 }
 
 void ssq_player_free(A2S_PLAYER players[], uint8_t player_count) {
+    if (players == NULL)
+        return;
     for (uint8_t i = 0; i < player_count; ++i)
         free(players[i].name);
     free(players);

diff --git a/src/a2s/rules.c b/src/a2s/rules.c
@@ -78,6 +78,8 @@ A2S_RULES *ssq_rules(SSQ_SERVER *server, uint16_t *rule_count) {
 }
 
 void ssq_rules_free(A2S_RULES rules[], uint16_t rule_count) {
+    if (rules == NULL)
+        return;
     for (uint16_t i = 0; i < rule_count; ++i) {
         free(rules[i].name);
         free(rules[i].value);

diff --git a/src/server.c b/src/server.c
@@ -35,6 +35,8 @@ SSQ_SERVER *ssq_server_new(const char hostname[], uint16_t port) {
 }
 
 void ssq_server_free(SSQ_SERVER *server) {
+    if (server == NULL)
+        return;
     freeaddrinfo(server->addr_list);
     free(server);
 }