[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: delete (instead of delete[]) used with new[]

From: Stefan Kueng <tortoisesvn_at_gmail.com>
Date: Mon, 08 Dec 2008 20:15:35 +0100

Shannon Barber wrote:
> I recently have run several memory analysis tools on my one of my
> projects and some corruption errors were detected in TortoiseSVN.dll.
>
> The corruption is detected when a file-open dialog is closed.
>
> Since Tortoise integrates with the shell, this is arguable an important
> bug since every application is corrupting its free-store when open file
> dialogs are closed.
>
>
>
> There appears to be (at least) two locations where memory is allocated
> using new[] but is freed using delete.
>
> A review of the code searching for new[]’s and their corresponding
> delete or delete[] might be the most effective way to fix this bug.
>
> I have also included undecorated stack traces below (in case they help
> at all).
>
> (I searched the Flyspray task list and the ChangeLog.txt and saw no
> mention of delete/new new[] nor delete regarding memory allocation.)

Should be fixed already in trunk:
http://nightlybuilds.tortoisesvn.net/latest/changelog.html#r14336

Stefan

-- 
       ___
  oo  // \\      "De Chelonian Mobile"
 (_,\/ \_/ \     TortoiseSVN
   \ \_/_\_/>    The coolest Interface to (Sub)Version Control
   /_/   \_\     http://tortoisesvn.net
------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=4061&dsMessageId=981244
To unsubscribe from this discussion, e-mail: [users-unsubscribe_at_tortoisesvn.tigris.org].

Received on 2008-12-08 20:15:56 CET

This is an archived mail posted to the TortoiseSVN Users mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.