EldoS | Feel safer!

Software components for data protection, secure storage and transfer

RequestAttributes - Mem Leak when file does not exist

Also by EldoS: CallbackRegistry
A component to monitor and control Windows registry access and create virtual registry keys.
#13931
Posted: 07/15/2010 10:08:51
by merkl (Standard support level)
Joined: 07/15/2010
Posts: 3

When using

TElSimpleSFTPClient->RequestAttributes(File, bool, attrobj);

and File *does not exist*, then there left a Memory Leak.
When File exists, it works like a charm!

We tryed CleanupRegisteredGlobalObjects(), does not help. We using Borland C++ Builder 2009 with CodeGurad enabled.

Are there any aditional object to free, when the requested file does not exist?


Minimalistic sample project with a view line of code, just

SftpClient->Open();
SftpClient->RequestAttributes(),
SftpClient->close()

which shows the issue, I can send upon request to you.
#13933
Posted: 07/15/2010 10:13:42
by merkl (Standard support level)
Joined: 07/15/2010
Posts: 3

Added screenshot of CodeGuard output and hope this may help.


#13934
Posted: 07/15/2010 10:32:21
by Ken Ivanov (EldoS Corp.)

Thank you for the report. We will investigate the issue and get back to you as soon as possible.
#13936
Posted: 07/15/2010 10:48:17
by Mykola Olshevsky (Basic support level)
Joined: 07/07/2005
Posts: 450

Hi. Are you sure that that's not your object(s)?
When file does not exist, RequestFileAttributes throws an exception, and in your code snippet you don't handle it.
#13946
Posted: 07/16/2010 09:03:07
by merkl (Standard support level)
Joined: 07/15/2010
Posts: 3

You are right! Me ashamed - never mind that this causes a mem leak.

Thanks for helping so quickly!
Also by EldoS: CallbackProcess
A component to control process creation and termination in Windows and .NET applications.

Reply

Statistics

Topic viewed 902 times

Number of guests: 1, registered members: 0, in total hidden: 0




|

Back to top

As of July 15, 2016 EldoS Corporation will operate as a division of /n software inc. For more information, please read the announcement.

Got it!