From patchwork Sun May 21 16:32:23 2006
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
X-Patchwork-Submitter: Tobias Grimm <listaccount@e-tobi.net>
X-Patchwork-Id: 12312
Received: from moutng.kundenserver.de ([212.227.126.187])
	by www.linuxtv.org with esmtp (Exim 4.50) id 1Fhqr8-0008MA-O0
	for vdr@linuxtv.org; Sun, 21 May 2006 18:32:38 +0200
Received: from [84.184.25.27] (helo=[192.168.30.10])
	by mrelayeu.kundenserver.de (node=mrelayeu4) with ESMTP (Nemesis),
	id 0ML21M-1Fhqqt3rn2-0003pS; Sun, 21 May 2006 18:32:36 +0200
Message-ID: <44709617.20804@e-tobi.net>
Date: Sun, 21 May 2006 18:32:23 +0200
From: Tobias Grimm <listaccount@e-tobi.net>
User-Agent: Debian Thunderbird 1.0.7 (X11/20051017)
X-Accept-Language: en-us, en
MIME-Version: 1.0
To: VDR Mailing List <vdr@linuxtv.org>
X-Provags-ID: kundenserver.de abuse@kundenserver.de
	login:3a660adf8439ddc7be7d699eb4083112
Subject: [vdr] Some small bug in vdr 1.4.0
X-BeenThere: vdr@linuxtv.org
X-Mailman-Version: 2.1.5
Precedence: list
Reply-To: VDR Mailing List <vdr@linuxtv.org>
List-Id: VDR Mailing List <vdr.linuxtv.org>
List-Unsubscribe: <http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr>,
	<mailto:vdr-request@linuxtv.org?subject=unsubscribe>
List-Archive: <http://www.linuxtv.org/pipermail/vdr>
List-Post: <mailto:vdr@linuxtv.org>
List-Help: <mailto:vdr-request@linuxtv.org?subject=help>
List-Subscribe: <http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr>,
	<mailto:vdr-request@linuxtv.org?subject=subscribe>
X-List-Received-Date: Sun, 21 May 2006 16:32:39 -0000
Status: O
X-Status: 
X-Keywords: 
X-UID: 9550

Hi!

While checking some plugins with valgrind, I stumbled across this minor
bug in vdr:

The skinDescriptions array allocated in cMenuSetupOsd should be deleted
with delete[] instead of delete.

bye,

Tobias

--- vdr-1.4.0.orig/menu.c
+++ vdr-1.4.0/menu.c
@@ -2131,7 +2131,7 @@
 cMenuSetupOSD::~cMenuSetupOSD()
 {
   cFont::SetCode(I18nCharSets()[Setup.OSDLanguage]);
-  delete skinDescriptions;
+  delete[] skinDescriptions;
 }
 
 void cMenuSetupOSD::Set(void)