fix: Fix memory leak in the error path of loading savedata.

[iphydf]

This change is 

[JFreegman]

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: complete! 1 of 1 approvals obtained

[codecov]

Codecov Report

Attention: 10 lines in your changes are missing coverage. Please review.

Comparison is base (32041a1) 74.41% compared to head (8235d9b) 74.49%.
Report is 2 commits behind head on master.

❗ Current head 8235d9b differs from pull request most recent head 94cf9d1. Consider uploading reports for the commit 94cf9d1 to get more accurate results

Files	Patch %	Lines
toxcore/tox.c	16.66%	10 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2424      +/-   ##
==========================================
+ Coverage   74.41%   74.49%   +0.08%     
==========================================
  Files          87       87              
  Lines       26235    26243       +8     
==========================================
+ Hits        19523    19551      +28     
+ Misses       6712     6692      -20     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[nurupo]

Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: complete! 2 of 1 approvals obtained

[nurupo]

Huh, how did this still use free() when all other parts of the code around it were changed to mem_delete()

[JFreegman]

Huh, how did this still use free() when all other parts of the code around it were changed to mem_delete()

This is a 2 year old PR. I forgot to incorporate some recent changes into it, such as that, and groupchats.

[nurupo]

Was confused how did iphy miss those free()s during his mem refactor, but they are new additions from from a recently merged old PR -- makes sense now.