Issue 22598 - Small fixes for memory leaks as reported by Valgrind
Summary: Small fixes for memory leaks as reported by Valgrind
Status: CLOSED FIXED
Alias: None
Product: porting
Classification: Code
Component: code (show other issues)
Version: OOo 1.1 RC5
Hardware: PC Linux, all
: P3 Trivial (vote)
Target Milestone: OOo 1.1.1
Assignee: philipp.lohmann
QA Contact: issues@porting
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-11-18 14:49 UTC by fa
Modified: 2004-02-26 17:50 UTC (History)
1 user (show)

See Also:
Issue Type: PATCH
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
cd vcl, patch -p0 < /path/to/patchfile (900 bytes, patch)
2003-11-18 14:50 UTC, fa 		no flags Details | Diff
patch -p0 < /path/to/patchfile (1.45 KB, patch)
2003-11-18 14:52 UTC, fa 		no flags Details | Diff
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description fa 2003-11-18 14:49:54 UTC 
A few small fixes.  The first two in vcl result from not freeing memory that's
returned by X11 calls even when the calls don't succeed.

The second one deals with a rare case, about ~100k leaked, when an object is not
cleaned up in LockHelper class.
Comment 1 fa 2003-11-18 14:50:31 UTC 
Created attachment 11373 [details]
cd vcl, patch -p0 < /path/to/patchfile
Comment 2 fa 2003-11-18 14:52:53 UTC 
Created attachment 11374 [details]
patch -p0 < /path/to/patchfile
Comment 3 jens-heiner.rechtien 2003-11-18 15:12:15 UTC 
Hi Philipp,

can you please commit these changes too one of your vcl CWSs?
valgrind01 is already in QA.

Thanks,
  Heiner
Comment 4 philipp.lohmann 2003-11-18 15:14:42 UTC 
will do
Comment 5 philipp.lohmann 2003-11-21 15:12:42 UTC 
fixed in CWS vcl7pp1r4 - there were lots of trees left in
wmadaptor.cxx where a property might not have been freed in an error
condition, i hope to have them squashed all.
Comment 6 philipp.lohmann 2003-12-12 14:22:22 UTC 
adjusted target
Comment 7 philipp.lohmann 2003-12-18 15:05:41 UTC 
verified in CWS vcl7pp1r4
Comment 8 philipp.lohmann 2003-12-18 15:12:27 UTC 
.
Comment 9 philipp.lohmann 2004-02-26 17:50:31 UTC 
integrated in SRC680.m27