fixed crash when start recording with extrecmenu or remotetimers

HISTORY

@@ -150,4 +150,5 @@ Version 0.1.2
 
 Version 0.1.3
 
-- added {epgname} token for recordings in currentview and detailedview.
+- added {epgname} token for recordings in currentview and detailedview.
+- fixed crash when start recording with extrecmenu or remotetimers

displaychannel.c

@@ -2,6 +2,7 @@
 #include "libcore/timers.h"
 
 cSDDisplayChannel::cSDDisplayChannel(cTemplate *channelTemplate, bool WithInfo) {
+    channelView = NULL;
     if (firstDisplay) {
         firstDisplay = false;
         doOutput = false;
@@ -30,9 +31,8 @@ cSDDisplayChannel::cSDDisplayChannel(cTemplate *channelTemplate, bool WithInfo)
 }
 
 cSDDisplayChannel::~cSDDisplayChannel() {
-    if (!doOutput)
-        return;
-    delete channelView;
+    if (channelView)
+        delete channelView;
 }
 
 void cSDDisplayChannel::SetChannel(const cChannel *Channel, int Number) {

displaymenu.c

@@ -2,7 +2,8 @@
 #include "libcore/helpers.h"
 
 cSDDisplayMenu::cSDDisplayMenu(cTemplate *menuTemplate) {
-    textAreaFont = NULL; 
+    textAreaFont = NULL;
+    rootView = NULL;
     doOutput = true;
     state = vsInit;
     pluginMenu = -1;
@@ -21,9 +22,8 @@ cSDDisplayMenu::cSDDisplayMenu(cTemplate *menuTemplate) {
 }
 
 cSDDisplayMenu::~cSDDisplayMenu() {
-    if (!doOutput)
-        return;
-    delete rootView;
+    if (rootView)
+        delete rootView;
     if (textAreaFont)
         delete textAreaFont;
 }

displaymessage.c

@@ -1,6 +1,7 @@
 #include "displaymessage.h"
 
 cSDDisplayMessage::cSDDisplayMessage(cTemplate *messageTemplate) {
+    messageView = NULL;
     doOutput = true;
     initial = true;
     if (!messageTemplate) {
@@ -17,9 +18,8 @@ cSDDisplayMessage::cSDDisplayMessage(cTemplate *messageTemplate) {
 }
 
 cSDDisplayMessage::~cSDDisplayMessage() {
-    if (!doOutput)
-        return;
-    delete messageView;
+    if (messageView)
+        delete messageView;
 }
 
 void cSDDisplayMessage::SetMessage(eMessageType Type, const char *Text) {

displayreplay.c

@@ -5,6 +5,7 @@ cSDDisplayReplay::cSDDisplayReplay(cTemplate *replayTemplate, bool ModeOnly) {
     doOutput = true;
     initial = true;
     modeOnly = ModeOnly;
+    replayView = NULL;
     if (!replayTemplate) {
         doOutput = false;
         esyslog("skindesigner: displayReplay no valid template - aborting");
@@ -19,9 +20,8 @@ cSDDisplayReplay::cSDDisplayReplay(cTemplate *replayTemplate, bool ModeOnly) {
 }
 
 cSDDisplayReplay::~cSDDisplayReplay() {
-    if (!doOutput)
-        return;
-    delete replayView;
+    if (replayView)
+        delete replayView;
 }
 
 void cSDDisplayReplay::SetRecording(const cRecording *Recording) {

displaytracks.c

@@ -4,6 +4,7 @@
 cSDDisplayTracks::cSDDisplayTracks(cTemplate *audiotracksTemplate, const char *Title, int NumTracks, const char * const *Tracks) {
     initial = true;
     numTracks = NumTracks;
+    tracksView = NULL;
     doOutput = true;
     currentTrack = 0;
     menuTitle = Title;
@@ -28,9 +29,8 @@ cSDDisplayTracks::cSDDisplayTracks(cTemplate *audiotracksTemplate, const char *T
 }
 
 cSDDisplayTracks::~cSDDisplayTracks() {
-    if (!doOutput)
-        return;
-    delete tracksView;
+    if (tracksView)
+        delete tracksView;
 }
 
 void cSDDisplayTracks::SetTrack(int Index, const char * const *Tracks) {

displayvolume.c

@@ -4,6 +4,7 @@
 #include "libcore/helpers.h"
 
 cSDDisplayVolume::cSDDisplayVolume(cTemplate *volumeTemplate) {
+    volumeView = NULL;
     doOutput = true;
     initial = true;
     if (!volumeTemplate) {
@@ -20,9 +21,8 @@ cSDDisplayVolume::cSDDisplayVolume(cTemplate *volumeTemplate) {
 }
 
 cSDDisplayVolume::~cSDDisplayVolume() {
-    if (!doOutput)
-        return;
-    delete volumeView;
+    if (volumeView)
+        delete volumeView;
 }
 
 void cSDDisplayVolume::SetVolume(int Current, int Total, bool Mute) {

libcore/pixmapcontainer.c

@@ -44,7 +44,7 @@ cPixmapContainer::~cPixmapContainer(void) {
 
 bool cPixmapContainer::CreateOsd(int Left, int Top, int Width, int Height) {
     if (osd) {
-        return true;
+        return false;
     }
     cOsd *newOsd = cOsdProvider::NewOsd(Left, Top);
     if (newOsd) {