#18897: Memory leak in polynomial_compiled.univar_pd
-------------------------+-------------------------------------------------
       Reporter:         |        Owner:
  slabbe                 |       Status:  needs_review
           Type:         |    Milestone:  sage-6.8
  defect                 |   Resolution:
       Priority:  major  |    Merged in:
      Component:         |    Reviewers:
  memleak                |  Work issues:
       Keywords:         |       Commit:
        Authors:  Simon  |  720de3bf96d4e72965581bbf5ad372e94fe61237
  King                   |     Stopgaps:
Report Upstream:  N/A    |
         Branch:         |
  public/18897           |
   Dependencies:         |
-------------------------+-------------------------------------------------

Comment (by SimonKing):

 PS: I am doing this change now. Also, I am changing `__init__` into
 `__cinit__`.

 Rationale: The current `__init__` just sets the pointer `self.head =
 NULL`. But initialising c-data (pointers etc.) is the job of `__cinit__`:
 It should initialise the c-data to such extend that `__dealloc__` can not
 crash. With the current code, there may be circumstances under which the
 `__init__` is not called, so that `__dealloc__` will fail with a
 segmentation fault.

--
Ticket URL: <http://trac.sagemath.org/ticket/18897#comment:20>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to