[ucb-issues] [Issue 106478] ucb: ftpurl.cxx: memory l eak

2010-01-14 Thread tkr
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=106478


User tkr changed the following:

What|Old value |New value

  Status|RESOLVED  |VERIFIED





--- Additional comments from t...@openoffice.org Thu Jan 14 09:45:13 + 
2010 ---
verified by code review

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: issues-unsubscr...@ucb.openoffice.org
For additional commands, e-mail: issues-h...@ucb.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[ucb-issues] [Issue 106478] ucb: ftpurl.cxx: memory l eak

2010-01-11 Thread kso
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=106478


User kso changed the following:

What|Old value |New value

 Assigned to|kso   |tkr





--- Additional comments from k...@openoffice.org Mon Jan 11 10:39:19 + 
2010 ---
tkr: Please verify (code review).

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: issues-unsubscr...@ucb.openoffice.org
For additional commands, e-mail: issues-h...@ucb.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[ucb-issues] [Issue 106478] ucb: ftpurl.cxx: memory l eak

2009-11-05 Thread cmc
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=106478


User cmc changed the following:

What|Old value |New value

OtherIssuesDependingOnTh|  |106602
  is|  |





-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: issues-unsubscr...@ucb.openoffice.org
For additional commands, e-mail: issues-h...@ucb.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[ucb-issues] [Issue 106478] ucb: ftpurl.cxx: memory l eak

2009-11-03 Thread kso
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=106478


User kso changed the following:

What|Old value |New value

  Status|STARTED   |RESOLVED

  Resolution|  |FIXED





--- Additional comments from k...@openoffice.org Tue Nov  3 13:00:24 + 
2009 ---
Applied path to CWS kso42. Thanks.

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: issues-unsubscr...@ucb.openoffice.org
For additional commands, e-mail: issues-h...@ucb.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[ucb-issues] [Issue 106478] ucb: ftpurl.cxx: memory l eak

2009-10-30 Thread dcb314
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=106478
 Issue #|106478
 Summary|ucb: ftpurl.cxx: memory leak
   Component|ucb
 Version|DEV300m63
Platform|All
 URL|
  OS/Version|Linux
  Status|UNCONFIRMED
   Status whiteboard|
Keywords|
  Resolution|
  Issue type|DEFECT
Priority|P3
Subcomponent|code
 Assigned to|mhu
 Reported by|dcb314





--- Additional comments from dcb...@openoffice.org Fri Oct 30 10:58:26 
+ 2009 ---
For source code file ucb/source/ucp/ftp/ftpurl.cxx,
I notice the following code

char *buffer = new char[1+aIdent.getLength()];

const char* p2 = aIdent.getStr();

rtl::OString lower = aIdent.toAsciiLowerCase();
if(lower.getLength()  6 ||
   strncmp(ftp://,lower.getStr(),6))
throw malformed_exception();

If the throw is taken, then buffer is a memory leak. Suggest new code

const char* p2 = aIdent.getStr();

rtl::OString lower = aIdent.toAsciiLowerCase();
if(lower.getLength()  6 ||
   strncmp(ftp://,lower.getStr(),6))
throw malformed_exception();

   char *buffer = new char[1+aIdent.getLength()];

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: issues-unsubscr...@ucb.openoffice.org
For additional commands, e-mail: issues-h...@ucb.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[ucb-issues] [Issue 106478] ucb: ftpurl.cxx: memory l eak

2009-10-30 Thread cmc
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=106478


User cmc changed the following:

What|Old value |New value

 Attachment is patch|  |Created an attachment (id=
|  |65796)
just move allocs af
|  |ter throw






--- Additional comments from c...@openoffice.org Fri Oct 30 11:10:27 + 
2009 ---
Created an attachment (id=65796)
just move allocs after throw


-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: issues-unsubscr...@ucb.openoffice.org
For additional commands, e-mail: issues-h...@ucb.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[ucb-issues] [Issue 106478] ucb: ftpurl.cxx: memory l eak

2009-10-30 Thread cmc
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=106478


User cmc changed the following:

What|Old value |New value

 Assigned to|mhu   |kso

  Ever confirmed|  |1

  Status|UNCONFIRMED   |NEW

  Issue type|DEFECT|PATCH





--- Additional comments from c...@openoffice.org Fri Oct 30 11:14:16 + 
2009 ---
confirmed. Straight forward patch is probably sufficient, rather than reaching
for boost::scoped_array or std::vector

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: issues-unsubscr...@ucb.openoffice.org
For additional commands, e-mail: issues-h...@ucb.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[ucb-issues] [Issue 106478] ucb: ftpurl.cxx: memory l eak

2009-10-30 Thread kso
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=106478


User kso changed the following:

What|Old value |New value

  Status|NEW   |STARTED

Target milestone|---   |OOo 3.3





--- Additional comments from k...@openoffice.org Fri Oct 30 11:31:16 + 
2009 ---
.

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: issues-unsubscr...@ucb.openoffice.org
For additional commands, e-mail: issues-h...@ucb.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org