Improved cControl::Launch() to keep 'control' from pointing to uninitialized memory

This commit is contained in:
Klaus Schmidinger 2007-07-20 15:26:13 +02:00
parent a8eaa2538c
commit 214c12289f
3 changed files with 7 additions and 2 deletions

View File

@ -1015,6 +1015,8 @@ Rolf Ahrenberg <rahrenbe@cc.hut.fi>
the plugins
for increasing the maximum number of DVB devices to 8
for suggesting to change the parameter "OSD font" to "Default font" in "Setup/OSD"
for improving cControl::Launch() to keep 'control' from pointing to uninitialized
memory
Ralf Klueber <ralf.klueber@vodafone.com>
for reporting a bug in cutting a recording if there is only a single editing mark

View File

@ -5288,3 +5288,5 @@ Video Disk Recorder Revision History
- Fixed a typo in the function name of cOsd::SetOsdPosition() and added a range
check to it (thanks to Christoph Haubrich).
- Updated the Finnish OSD texts (thanks to Rolf Ahrenberg).
- Improved cControl::Launch() to keep 'control' from pointing to uninitialized
memory (thanks to Rolf Ahrenberg).

View File

@ -4,7 +4,7 @@
* See the main source file 'vdr.c' for copyright information and
* how to reach the author.
*
* $Id: player.c 1.11 2006/01/06 11:30:07 kls Exp $
* $Id: player.c 1.12 2007/07/20 15:25:24 kls Exp $
*/
#include "player.h"
@ -69,8 +69,9 @@ cControl *cControl::Control(void)
void cControl::Launch(cControl *Control)
{
cMutexLock MutexLock(&mutex);
delete control;
cControl *c = control; // keeps control from pointing to uninitialized memory
control = Control;
delete c;
}
void cControl::Attach(void)