Page MenuHome
Create Task
Maniphest T28737

Fix a memory leak in transform_conversions.c
Closed, ArchivedPublicPATCH
Actions

Description

Found this while working on BMesh, but it looks like the same case could easily happen in trunk.

Event Timeline

Andrew Wiggin (ender79) edited a custom field.Sep 24 2011, 12:20 PM
Andrew Wiggin (ender79) attached 1 file(s): F16414: trunk.transformconversionsleak.1.patch.
Campbell Barton (campbellbarton) added a comment.Sep 25 2011, 10:16 AM

From looking in trunk there is no leak,
but if this helps keep bmesh/trunk code in sync (and fixes a bmesh leak), then +1 to commit to trunk.

Andrew Wiggin (ender79) added a comment.Sep 25 2011, 12:53 PM

Ah, so it's true. Let's not make this change in trunk then. I think there is a better way to fix the bmesh leak if it's only in that branch...

Andrew Wiggin (ender79) changed the task status from Unknown Status to Unknown Status.Sep 25 2011, 12:53 PM