[vdr] [Patch] crashes with old recordings without info file

Mika Laitio lamikr at pilppa.org
Sun Apr 3 14:26:03 CEST 2011

I wonder who is actually calling cRecordingInfo::Read(FILE *f) with
> a NULL pointer? In VDR's own code all calls to that function are
> made sure to get a non-NULL pointer:

Yes, the real reason is in the vdrrip plugin that I cloned from

It has this kind of code which always assumes that info file always exist.

void cMovie::setLengthVDR() {
         char *infoFile = NULL;
         asprintf(&infoFile, "%s%s", Dir, OldRecording ? "/info.vdr" : 
         dsyslog ("[vdrrip] reading recording info %s ",infoFile);
         cRecordingInfo *crec = new cRecordingInfo(Dir);
         FILE *f = fopen(infoFile, "r");
         if (crec->Read(f)) {

I just thought that it would be good to fix that in vdr code also.
But I think putting the metod now private will also quarantee NULL 
pointer check :-)


More information about the vdr mailing list