New posts are disabled while we improve the user experience.

You can browse the site, or for urgent issues, raise a query at MyAccount.

question

Upvotes
Accepted
irfan.butt avatar image
1 ●1 ●1 ●2

Is there a memory leak in readRDMFieldDictionary? rfa8.1.0.L1

Hello

I am using the C++ API on Linux rfa8.1.0.L1. I have created a very small simple program that looks like this:

::rfa::rdm::RDMFieldDictionary& foo = ::rfa::rdm::RDMFieldDictionary::create();
foo.destroy();

This is fine, but the moment that I add a call to readRDMFieldDictionary in between the calls to create() and destroy(), the program leaks memory according to LeakSanitiser, like so:

================================================================= ==13136==ERROR: LeakSanitizer: detected memory leaks 
Direct leak of 256 byte(s) in 1 object(s) allocated from:    #0 0x7fbd695c55a0 in __interceptor_realloc ../../../../gcc-8.2.0-src/libsanitizer/asan/asan_malloc_linux.cc:105    #1 0x42e230 in rfa::rdm::RDMFieldDictionaryInt::readRDMFieldDictionary(rfa::common::RFA_String const&) (/apps/a1628967/ECOMMFX-630/s2bx/build.waf/debug_asan/install/unittest/RFA_Leaks_UnitTest+0x42e230) 
Direct leak of 256 byte(s) in 1 object(s) allocated from:    #0 0x7fbd695c55a0 in __interceptor_realloc ../../../../gcc-8.2.0-src/libsanitizer/asan/asan_malloc_linux.cc:105    #1 0x42e253 in rfa::rdm::RDMFieldDictionaryInt::readRDMFieldDictionary(rfa::common::RFA_String const&) (/apps/a1628967/ECOMMFX-630/s2bx/build.waf/debug_asan/install/unittest/RFA_Leaks_UnitTest+0x42e253) 
Direct leak of 256 byte(s) in 1 object(s) allocated from:    #0 0x7fbd695c55a0 in __interceptor_realloc ../../../../gcc-8.2.0-src/libsanitizer/asan/asan_malloc_linux.cc:105    #1 0x42e20d in rfa::rdm::RDMFieldDictionaryInt::readRDMFieldDictionary(rfa::common::RFA_String const&) (/apps/a1628967/ECOMMFX-630/s2bx/build.waf/debug_asan/install/unittest/RFA_Leaks_UnitTest+0x42e20d) 
SUMMARY: AddressSanitizer: 768 byte(s) leaked in 3 allocation(s).
c++elektron-data-dictionarymemory-leaks
icon clock
10 |1500

Up to 2 attachments (including images) can be used with a maximum of 512.0 KiB each and 1.0 MiB total.

4 Answers

· Write an Answer
Upvote
Accepted
Jirapongse avatar image
87.8k ●294 ●53 ●79

@irfan.butt

I could see the problem with the Visual Studio Profiler.

1641972605085.png

If you are an RDC named user, please submit this issue to the RDC support team via Contact Premium support to investigate and confirm the problem.



1641972605085.png (16.3 KiB)
icon clock
10 |1500

Up to 2 attachments (including images) can be used with a maximum of 512.0 KiB each and 1.0 MiB total.

irfan.butt avatar image irfan.butt
Hi there, thanks for investigating and confirming my findings. I'll raise this with someone in our organisation who has RDC named user access, and I'll also see if we are able to upgrade to the latest version of the library.

Regards

Irfan

irfan.butt avatar image irfan.butt
Also, @Jirapongse , which version of the RFA library did you find this leak in?
Jirapongse avatar image Jirapongse irfan.butt

@irfan.butt

I tested with RFA 8.2.2.L1.

irfan.butt avatar image irfan.butt Jirapongse
Hi @Jirapongse , thanks for letting me know that's most helpful. I'll see what I can do about raising this with your Premium Support team.


Regards


Irfan

Upvote
zoya faberov avatar image
32.2k ●41 ●11 ●20

Hello @irfan.butt ,

If you are designing or looking into a new realtime consumer or producer application, I would recommend to go with new, strategic and continuously improving Refinitiv Real-Time SDK C/C++ API suite, comprised of EMA - ease of use message tier and low-level ETA transport tier, Quickstart guide can be helpful in getting the first integration quickly and efficiently. Would first look into EMA option.

Article Choosing a Refinitiv Real-time Streaming API can be very helpful in making informed and up-to-date selection of realtime API.

If the issue is with an existent RFA application, please see Refinitiv Robust Foundation API (RFA) C++ and more details in the linked PCN, on re-branding, the existing applications will continue to work as is, however, it's important to know, that past the mandatory migration date, only re-branded versions continue to be fully supported , please also see this this previous discussion thread for the detailed relevant answer.

Hope this information is of help

icon clock
10 |1500

Up to 2 attachments (including images) can be used with a maximum of 512.0 KiB each and 1.0 MiB total.

Upvotes
irfan.butt avatar image
1 ●1 ●1 ●2

Hi @zoya faberov thanks for your prompt reply.


The issue is with an existing RFA application. We've only just noticed the leak because we've written a new unit test that is compiled with LeakSanitiser.

As a result of this, I simplified the code so that I now have a very simple test which calls create() and destroy().

The moment I call readRDMFieldDictionary(...) in between these 2 calls, LeakSanitiser reports the 3 leaks I have mentioned. The leaks are unrelated to the complexity of the dictionary, i.e. if the dictionary has 0 entries, I get 3 leaks, if it has 10 entries, I get 3 leaks, if it has 10s of entries, I still get 3 leaks.

I can try upgrading to a more recent version of the RFA library. How can I get a new version of the library? Has the interface changed between 8.1.0.L1 and more recent versions?

Regards

Irfan

icon clock
10 |1500

Up to 2 attachments (including images) can be used with a maximum of 512.0 KiB each and 1.0 MiB total.

Upvotes
zoya faberov avatar image
32.2k ●41 ●11 ●20

Hello @irfan.butt ,

Please download the latest rebranded version from Refinitiv Robust Foundation API (RFA) C++-> Downloads.

If your app was running for a while and there were no issues (i.e. excessive memory increases) , I would be cautious of a mis-report of an issue:

  • The reason being that RFA is a very mature API, has been around for long time, dictionary read has been in place, and all this time, any issue reported were fixed in the next releases.
  • Even if we consider that most would use the recommended download of the dictionary from infra (always up-to-date) still many would read local, and would have reported any leaks by now.

I would rebuild with the latest ( the interface should be the same) and see if you still observe the report.

If you continue seeing the report, I would let the app run for a while and see if you observe the leak, the increase in size and how much, without subscriptions/publications established, and go from there.

icon clock
10 |1500

Up to 2 attachments (including images) can be used with a maximum of 512.0 KiB each and 1.0 MiB total.

Write an Answer

Hint: Notify or tag a user in this post by typing @username.

Up to 2 attachments (including images) can be used with a maximum of 512.0 KiB each and 1.0 MiB total.