Multiple memory leak defects #120767

New Issue

Yin_Lu_Ming · 2024-04-18T05:31:47+02:00

Yin_Lu_Ming commented

2024-04-18 05:31:47 +02:00

Dear Blender developers,

I have identified a memory leak defect in the latest version of Blender. The detailed path of the defect is as follows:

1.In the file /blender/extern/mantaflow/helper/util/simpleimage.cpp, a pointer variable named "pic" is defined on line 155, and a dynamic memory region is allocated using the new operator. If the if statement on line 159 returns true, the program returns on line 162 without executing the release operation on the dynamic memory region pointed to by "pic" on line 203. This constitutes a memory leak defect, as illustrated in the diagram below:
https://github.com/LuMingYinDetect/blender_defects/blob/main/blender_4.png

Dear Blender developers, I have identified a memory leak defect in the latest version of Blender. The detailed path of the defect is as follows: 1.In the file /blender/extern/mantaflow/helper/util/simpleimage.cpp, a pointer variable named "pic" is defined on line 155, and a dynamic memory region is allocated using the new operator. If the if statement on line 159 returns true, the program returns on line 162 without executing the release operation on the dynamic memory region pointed to by "pic" on line 203. This constitutes a memory leak defect, as illustrated in the diagram below: https://github.com/LuMingYinDetect/blender_defects/blob/main/blender_4.png

Yin_Lu_Ming added the

labels 2024-04-18 05:31:48 +02:00

Pratik Borhade commented

2024-04-18 05:35:52 +02:00

Hi, thanks for finding the memory leak. But could you mention all of these in single report. I think creating new report for each case will just increase the noise 😅

😆 2

Yin_Lu_Ming commented

2024-04-18 09:12:43 +02:00

Hi, thanks for finding the memory leak. But could you mention all of these in single report. I think creating new report for each case will just increase the noise 😅

I apologize for any inconvenience caused! I have uploaded the HTML file containing all memory leak defects from the report for your reference!

> Hi, thanks for finding the memory leak. But could you mention all of these in single report. I think creating new report for each case will just increase the noise 😅 I apologize for any inconvenience caused! I have uploaded the HTML file containing all memory leak defects from the report for your reference!

indexer.cc.html

273 KiB

simpleimage.cpp.html

216 KiB

clip_buttons.cc.html

233 KiB

Cache.cpp.html

229 KiB

sequencer_preview.cc.html

213 KiB

Yin_Lu_Ming commented

2024-04-18 09:37:53 +02:00

Hi, thanks for finding the memory leak. But could you mention all of these in single report. I think creating new report for each case will just increase the noise 😅

Hello, in addition to the memory leak defect discovered earlier, I have found a new memory leak defect. Here are the details:

There is a memory leak defect at line 352 in the file /blender/source/blender/blentranslation/msgfmt/msgfmt.cc.

At line 292 of the file, a pointer variable named 'msgs_memarena' is defined, and a dynamic memory area is allocated through the function 'BLI_memarena_new'. When the 'if' statement at line 350 returns true, the program will return at line 352 without executing the release operation on the dynamic memory area pointed to by 'msgs_memarena' at line 438, thereby causing a memory leak defect, as shown in the attached image.

> Hi, thanks for finding the memory leak. But could you mention all of these in single report. I think creating new report for each case will just increase the noise 😅 Hello, in addition to the memory leak defect discovered earlier, I have found a new memory leak defect. Here are the details: There is a memory leak defect at line 352 in the file /blender/source/blender/blentranslation/msgfmt/msgfmt.cc. At line 292 of the file, a pointer variable named 'msgs_memarena' is defined, and a dynamic memory area is allocated through the function 'BLI_memarena_new'. When the 'if' statement at line 350 returns true, the program will return at line 352 without executing the release operation on the dynamic memory area pointed to by 'msgs_memarena' at line 438, thereby causing a memory leak defect, as shown in the attached image.

blender_7.png

142 KiB

Yin_Lu_Ming changed title from ~~There is a memory leak defect on line 162 of the file /blender/extern/mantaflow/helper/util/simpleimage.cpp.~~ to Multiple memory leak defects

2024-04-18 09:39:45 +02:00

Yin_Lu_Ming commented

2024-04-18 09:53:02 +02:00

Hi, thanks for finding the memory leak. But could you mention all of these in single report. I think creating new report for each case will just increase the noise 😅

Details of another newly discovered defect:

There is a memory leak defect at line 564 in the file /blender/source/blender/editors/object/object_bake.cc.

1.At line 559 of /blender/source/blender/editors/object/object_bake.cc, a pointer named 'bkr' is defined, and a dynamic memory area is allocated using the 'MEM_cnew' operator. When the 'if' statement at line 562 returns true, the program will return at line 564. During this process, the dynamic memory area pointed to by 'bkr' is neither used for data at line 574 nor released, as shown in the attached image.

> Hi, thanks for finding the memory leak. But could you mention all of these in single report. I think creating new report for each case will just increase the noise 😅 Details of another newly discovered defect: There is a memory leak defect at line 564 in the file /blender/source/blender/editors/object/object_bake.cc. 1.At line 559 of /blender/source/blender/editors/object/object_bake.cc, a pointer named 'bkr' is defined, and a dynamic memory area is allocated using the 'MEM_cnew' operator. When the 'if' statement at line 562 returns true, the program will return at line 564. During this process, the dynamic memory area pointed to by 'bkr' is neither used for data at line 574 nor released, as shown in the attached image.

blender_8.png

60 KiB

Iliya Katushenock added the

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

Multiple memory leak defects #120767