db/simple: call ReturnSong() on mounted database

Fixes a memory leak, or an assertion failure in the debug build.

db/simple: call ReturnSong() on mounted database
30e06447 · Max Kellermann · 3ada4640 · 30e06447 · 30e06447
Commit 30e06447 authored Feb 22, 2019 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 0 deletions

NEWS NEWS +1 -0

SimpleDatabasePlugin.cxx src/db/plugins/simple/SimpleDatabasePlugin.cxx +1 -0

No files found.
--- a/NEWS
+++ b/NEWS
@@ -8,6 +8,7 @@ ver 0.21.5 (not yet released)
 * storage
  - udisks: fix "AlreadyMounted" error
  - udisks: use relative path from mount URI
+  - fix memory leak
 * input
  - buffer: fix crash bug when playing remote WAV file
 * tags

--- a/src/db/plugins/simple/SimpleDatabasePlugin.cxx
+++ b/src/db/plugins/simple/SimpleDatabasePlugin.cxx
@@ -218,6 +218,7 @@ SimpleDatabase::GetSong(const char *uri) const

 		prefixed_light_song =
 			new PrefixedLightSong(*song, r.directory->GetPath());
+		r.directory->mounted_database->ReturnSong(song);
 		return prefixed_light_song;
 	}