[vdr] [ANNOUNCE] VDR developer version 1.3.49 - Memleak

Tobias Grimm listaccount at e-tobi.net
Sat Apr 29 14:37:03 CEST 2006

Klaus Schmidinger wrote:

> Thanks - applied for version 1.4.0.

I'm afraid I've found another minor memory leak. Valgrind would really
be worth it's money, if it wouldn't be free :-)

When playing back a recording, the file handle gets closed, but not the
cUnbufferedFile object. I'm not sure why a distinction is made between
recording and not recording when closing the file, so I just use
CloseVideoFile() in both cases, which correctly frees the object. Hope
this is correct. See attached patch.


-------------- next part --------------
--- vdr-1.3.49.orig/recording.c
+++ vdr-1.3.49/recording.c
@@ -1436,7 +1436,7 @@
 void cFileName::Close(void)
   if (file) {
-     if ((record && CloseVideoFile(file) < 0) || (!record && file->Close() < 0))
+     if (CloseVideoFile(file))
      file = NULL;

More information about the vdr mailing list