Skip to content
Snippets Groups Projects
Commit 48598fc0 authored by Rob Swindell's avatar Rob Swindell :speech_balloon:
Browse files

Fix memory leak (of extended description) when moving files

parent e2252f02
No related branches found
No related tags found
1 merge request!463MRC mods by Codefenix (2024-10-20)
Pipeline #2376 passed
......@@ -240,7 +240,9 @@ bool sbbs_t::movefile(smb_t* smb, file_t* f, int newdir)
}
newfile.dfield = NULL; // addfile() ends up realloc'ing dfield (in smb_addmsg)
if(!addfile(&cfg, newdir, &newfile, newfile.extdesc, newfile.metadata, /* client: */NULL))
bool result = addfile(&cfg, newdir, &newfile, newfile.extdesc, newfile.metadata, /* client: */NULL);
free(newfile.dfield);
if(!result)
return false;
if(!removefile(smb, f)) // Use ::removefile() here instead?
return false;
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment