Applying modifier leaks memory #33029

New Issue

Campbell Barton · 2012-10-31T12:49:45+01:00

Campbell Barton commented

2012-10-31 12:49:45 +01:00

%%%With a multi-res modifier, then an edge split modifier.
applying the edge split seems to work, but leaks memory on exiting blender.

see attached file.%%%

%%%With a multi-res modifier, then an edge split modifier. applying the edge split seems to work, but leaks memory on exiting blender. see attached file.%%%

Campbell Barton commented

2012-10-31 12:49:45 +01:00

Changed status to: 'Open'

Sergey Sharybin commented

2012-11-28 13:10:54 +01:00

%%%Campbell, here's a patch which fixes the issue. Basically BM_face_create_ngon uses existing loops' data as an example when adding new loops. It'll create loop data inside BM_face_create_ngon and later CustomData_to_bmesh_block used to overwrite this data without freeing it.

I've simply reshuffled code so no loop data will be used as an example. My tests works fine now.

Could you please review the patch?%%%

%%%Campbell, here's a patch which fixes the issue. Basically BM_face_create_ngon uses existing loops' data as an example when adding new loops. It'll create loop data inside BM_face_create_ngon and later CustomData_to_bmesh_block used to overwrite this data without freeing it. I've simply reshuffled code so no loop data will be used as an example. My tests works fine now. Could you please review the patch?%%%

Campbell Barton commented

2012-11-29 16:21:45 +01:00

%%%Thanks for finding a fix!, looking into an alternate fix though, will see how it goes.%%%

Campbell Barton commented

2012-11-29 17:27:15 +01:00

%%%Committed fix r52664. closing.%%%

Brecht Van Lommel commented

2012-11-29 17:38:00 +01:00

Changed status from 'Open' to: 'Resolved'

Brecht Van Lommel closed this issue

2012-11-29 17:38:00 +01:00

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

Applying modifier leaks memory #33029