Fixed processing SVDRP client responses in case the caller doesn't want the actual response strings

3 files changed, 8 insertions, 1 deletions

diff --git a/CONTRIBUTORS b/CONTRIBUTORS
index a906532d..f837b493 100644
--- a/CONTRIBUTORS
+++ b/CONTRIBUTORS
@@ -2845,6 +2845,8 @@ Johann Friedrichs <johann.friedrichs@web.de>
  for fixing handling timers during the change from DST to winter time
  for fixing a possible crash with plugins that retrieve player information after a
  replay has been stopped, but before the replay control has been destroyed
+ for reporting a problem in processing SVDRP client responses in case the caller doesn't
+ want the actual response strings
 
 Timo Helkio <timolavi@mbnet.fi>
  for reporting a hangup when replaying a TS recording with subtitles activated
diff --git a/HISTORY b/HISTORY
index 6c101c4b..2869b795 100644
--- a/HISTORY
+++ b/HISTORY
@@ -9320,3 +9320,5 @@ Video Disk Recorder Revision History
 2018-03-19: Version 2.4.0
 
 - Updated the Ukrainian OSD texts (thanks to Yarema Aka Knedlyk).
+- Fixed processing SVDRP client responses in case the caller doesn't want the actual
+  response strings (reported by Johann Friedrichs).
diff --git a/svdrp.c b/svdrp.c
index 4f74c8fd..0f56d10f 100644
--- a/svdrp.c
+++ b/svdrp.c
@@ -10,7 +10,7 @@
  * and interact with the Video Disk Recorder - or write a full featured
  * graphical interface that sits on top of an SVDRP connection.
  *
- * $Id: svdrp.c 4.36 2018/03/18 10:43:53 kls Exp $
+ * $Id: svdrp.c 4.37 2018/03/19 12:16:33 kls Exp $
  */
 
 #include "svdrp.h"
@@ -480,8 +480,11 @@ bool cSVDRPClient::Process(cStringList *Response)
 
 bool cSVDRPClient::Execute(const char *Command, cStringList *Response)
 {
+  cStringList Dummy;
   if (Response)
      Response->Clear();
+  else
+     Response = &Dummy;
   return Send(Command) && Process(Response);
 }