bugGNU Octave - Bugs: bug #52410, Calling delaunay with invalid...

 
 

bug #52410: Calling delaunay with invalid values two times causes exit from Octave

Submitter:  Rik <rik5>
Submitted:  Wed 15 Nov 2017 12:31:03 AM UTC
   
 
Category:  Octave Function Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Regression
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * 4.2.1 Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Thu 16 Nov 2017 03:35:15 AM UTC, comment #3: 

Turned out to be what I suspcted.  There was code to clean up the temporary memory used by QHull, but if error() was called then Octave would abort early without calling it.  I fixed that here http://hg.savannah.gnu.org/hgweb/octave/rev/bffbc95fa72b.

Rik <rik5>
Group administrator
Wed 15 Nov 2017 01:55:47 AM UTC, comment #2: 

I think it is in Octave.  Using the same Qhull library on my machine, the 4.0.3 version of Octave works while the 4.2.1 version and development versions fail.

My guess is that when there is an error Octave is no longer clearing internal state in Qhull before calling the library again.

Rik <rik5>
Group administrator
Wed 15 Nov 2017 01:29:42 AM UTC, comment #1: 

I think this is caused by qhull, so it might be an upstream regression. I've been trying to triag this, but without much time or success.

Juan Pablo Carbajal <juanpi>
Group Member
Wed 15 Nov 2017 12:31:03 AM UTC, original submission:  

Running


delaunay (1:4, 1:4)
delaunay (1:4, 1:4)


causes an abrupt exit to the command line from Octave.  This was not the case for 4.0.3 where Octave correctly generates a warning message each time delaunay is called with co-linear points.  Marking bug as a regression.


Rik <rik5>
Group administrator

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by juanpi (Posted a comment)
  • -email is unavailable- added by rik5 (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-11-16 rik5 StatusNone Fixed
        Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code