Issue 7453

Summary: OOO_STABLE_1_PORTS/X11: vcl print code incorrectly deallocates driver data
Product: porting Reporter: fa <dan>
Component: codeAssignee: ed
Status: CLOSED DUPLICATE QA Contact: issues@porting <issues>
Severity: Trivial    
Priority: P2 CC: issues
Version: OOo 1.0.1Keywords: merge_pending
Target Milestone: OOo 1.0.4   
Hardware: All   
OS: All   
Issue Type: PATCH Latest Confirmation in: ---
Developer Difficulty: ---
Issue Depends on:    
Issue Blocks: 8607    
Attachments:
Description Flags
cd to SRC_ROOT/vcl, patch -p0 < /path/to/patchfile, rebuild vcl none

Description fa 2002-09-02 22:15:25 UTC 
Patch should explain it all.  The data is rtl_freeMemory()d and in creation the member variable is always new'ed, so it should be delete'd instead.

This caused a repeatable SEGFAULT on MacOS X printing.

Dan
Comment 1 fa 2002-09-02 22:16:06 UTC 
Created attachment 2689 [details]
cd to SRC_ROOT/vcl, patch -p0 < /path/to/patchfile, rebuild vcl
Comment 2 fa 2002-09-02 22:16:40 UTC 
Requesting approval for the 083102 patch.

Dan
Comment 3 fa 2002-09-03 15:39:37 UTC 
Approved by Sander...  Ed, commit to OOO_STABLE_1_PORTS?

---------------------------------------
Date: Tue, 3 Sep 2002 14:18:46 +0100 
(BST)                                      
From: Sander Vesik 
<Sander.Vesik@Sun.COM>                                       
Reply-To: 
dev@porting.openoffice.org                                           
 
To: 
dev@porting.openoffice.org                                           
       
Subject: Re: [porting-dev] Approval request for 
7453                            
           
On Mon, 2 Sep 2002, Dan Williams 
wrote:                                         
> Approval request for issue 
7453.                                              
>                                                                    
> http://www.openoffice.org/issues/show_bug.cgi?
id=7453                         
>                                                                    
> free() versus delete 
problem.                                                 
           
approved                                                             

> 
Dan
Comment 4 ed 2002-10-12 14:25:12 UTC 
Committed to PORTS.  Added keyword merge_pending
Comment 5 khendricks 2003-04-17 01:18:45 UTC 
Hi Dan, 
 
I think this should be an array delete and not a non-array delete  
(at least on OOo 1.0.3 it is an array delete). 
 
Kevin
Comment 6 fa 2003-04-17 02:22:44 UTC 
Kevin,

Correct, it should be.  This issue (and the patch) were superceded by Issue 8012, 
which includes the correct delete[] code.

http://www.openoffice.org/project/www/issues/show_bug.cgi?id=8012

I'm going to mark this issue as Resolved Fixed.

Dan
Comment 7 fa 2003-04-17 02:23:20 UTC 
Superceded by Issue 8012.

*** This issue has been marked as a duplicate of 8012 ***
Comment 8 Martin Hollmichel 2004-03-25 07:39:18 UTC 
close issue.