Skip to content

Commit

Permalink
Fix memory leak (Bug 3010, with Bartosz Telenczuk)
Browse files Browse the repository at this point in the history
  • Loading branch information
peterjc committed Sep 18, 2010
1 parent 44b0c23 commit 285c085
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 0 deletions.
1 change: 1 addition & 0 deletions Bio/KDTree/KDTree.c
Expand Up @@ -308,6 +308,7 @@ void KDTree_destroy(struct KDTree* tree)
Region_destroy(tree->_query_region);
if (tree->_center_coord) free(tree->_center_coord);
if (tree->_coords) free(tree->_coords);
if (tree->_data_point_list) free(tree->_data_point_list);
free(tree);
}

Expand Down
1 change: 1 addition & 0 deletions CONTRIB
Expand Up @@ -63,6 +63,7 @@ Frederic Sohm <fsms at domain users.sourceforge.net>
Joao Rodrigues <anaryin at the domain gmail dot com>
Thomas Rosleff Soerensen <rosleff at domain mpiz-koeln.mpg.de>
Eric Talevich <eric.talevich at domain gmail.com>
Bartosz Telenczuk <bartosz.telenczuk at domain gmail.com>
Carlos Rios Vera <crosvera at domain gmail.com>
Johann Visagie <wjv at domain cityip.co.za>
Dan Vogel <dmv at domain andrew.cmu.edu>
Expand Down
1 change: 1 addition & 0 deletions NEWS
Expand Up @@ -21,6 +21,7 @@ text file format with longer feature indentation.
(At least) 5 people have contributed to this release, include 2 new people:

Bartek Wilczynski
Bartosz Telenczuk (first contribution)
Michiel de Hoon
Peter Cock
Siong Kong (first contribution)
Expand Down

3 comments on commit 285c085

@peterjc
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@btel are you the same Bartosz Telenczuk who helped track down this Biopython KDTree memory leak and fix it way back in 2010 at a Python & Friends workshop programme in Poland?

If so, would you agree to dual license your contribution under both the Biopython License Agreement and the 3-clause BSD license, see #898. Thank you!

@btel
Copy link

@btel btel commented on 285c085 Jan 13, 2020 via email

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@peterjc
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @btel 👍

Please sign in to comment.