rcurtin commented on this pull request.


>  
   if (treeOwner && referenceTree)
     delete referenceTree;
+    
+  referenceTree = new Tree(tree);    
+  referenceSet = &tree->Dataset();

I bet the problem is in this line.  Shouldn't this be `referenceSet = 
&referenceTree->Dataset()`?  Otherwise you are setting the pointer to the 
dataset of the tree that the user passed in, which isn't guaranteed to be a 
valid pointer outside the scope of this function.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/mlpack/mlpack/pull/799
_______________________________________________
mlpack mailing list
[email protected]
http://knife.lugatgt.org/cgi-bin/mailman/listinfo/mlpack

Reply via email to