bugGNU Octave - Bugs: bug #53604, ASAN new-delete-type-mismatch...

 
 

bug #53604: ASAN new-delete-type-mismatch error after "close all"

Submitted by:  Pantxo Diribarne <pantxo>
Submitted on:  Tue 10 Apr 2018 01:30:37 PM UTC  
 
Category:  Plotting with OpenGL Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Regression
Status:  Invalid Assigned to:  None
Originator Name:  Open/Closed:  Closed
Release:  4.3.90 Operating System:  GNU/Linux

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

( Jump to the original submission)

Fri 13 Apr 2018 03:46:55 PM UTC, comment #10:

I think we can close this bug as an upstream issue with Qt that has since been fixed.

The long list of leaks is exactly what ASAN produces. Almost all of them are tagged as "Indirect leaks" which means they may not be true leaks, and digging through each one to determine that would take eons. The ones to really watch are the direct leaks, for which we have about 56B.

I'm intrigued about the possible heap-buffer-overflow. Can you open a new bug report about that? I suspect it is a bug in the driver nouveau_dri, but there is the possibility that somehow the offset that Octave is supplying is something crazy.

We could put an assert statement in the code temporarily to make sure that offset is in some reasonable range ([-5, +5]).

Rik <rik5>
Project Administrator
Fri 13 Apr 2018 05:54:40 AM UTC, comment #9:

But there are other things that Address Sanitizer complains about. Below is something OpenGL-related. I notice that this is using nouveau_dri.so, and I've been having random errors building documentation figures on these KDE systems. A dozen figures will build, then one fail...start again...a dozen will pass, then another fail...keep repeating and eventually all the documentation figures build successfully. I suppose that the non-open Nvidia driver isn't being installed. On Mint/Cinnamon I know my Nvidia driver is being used and there are no documentation issues. I do see libasan.so at the start of the list.

Dan Sebald <sebald>
Fri 13 Apr 2018 05:45:11 AM UTC, comment #8:

OK, I'm compiling, configured with the option mentioned...

I noticed the following strange warning message during compilation:

Probably off topic, but I've had problems with settings, I think (just a guess). Sometimes when working on the GUI and doing a build, the first launch after the build seems to take about 100% longer (which still isn't too long) and will crash once in a while. I launch Octave again, and it's fine.

Wup...here's another one:

Here's one last one:

Wow, and then there is a big long screen full of entries like the following:

And a summary:

OK, so running from the GUI things are fine:

Ah, well there is this super long delay at exit and then address sanitizer gives a long list similar to what I noted previously, then summarizes:

So that list at build was probably from attempting to build the documentation figures via Octave. (Which crashed on the first figure...which I assume is related to some of this.)

And running from CLI things look fine as well:

In summary, no crash for the example with Qt 5.9.x.

Dan Sebald <sebald>
Fri 13 Apr 2018 02:06:25 AM UTC, comment #7:

Adding Dan to the CC list for this bug. He recently installed a new version of KDE with Qt libraries that I think are at the 5.9 level. He might be able to build with --enable-address-sanitizer-flags and try your simple test code.

Rik <rik5>
Project Administrator
Wed 11 Apr 2018 05:01:20 PM UTC, comment #6:

I'm running Qt5.6.1. It is possible (likely?) that the bug was introduced in 5.7 and then fixed by the time 5.9 was released.

It does sound like this is an upstream bug, and I would be fine with closing it.

Rik <rik5>
Project Administrator
Wed 11 Apr 2018 10:55:52 AM UTC, comment #5:

Maybe related to this upstream Qt bug:

https://groups.google.com/forum/#!topic/linux.debian.bugs.dist/fVAuz_I7kKg

@Rik: I am using exactly the same distribution as the user. Do you by any chance use Qt 5.9 in which the above bug is supposed to have been fixed?

Pantxo Diribarne <pantxo>
Project Member
Tue 10 Apr 2018 09:00:00 PM UTC, comment #4:

>> Adding bug #53513 as a dependency since I'm pretty sure they are related.


That is also what I thought but I could check that reverting cset b6aea95a7bf9 does not fix bug #53513. Unfortunately I don't have debug symbols enabled and no time right now provide a backtrace.

Pantxo Diribarne <pantxo>
Project Member
Tue 10 Apr 2018 04:18:39 PM UTC, comment #3:

I'm not getting the delete-type-mismatch error after compiling with --enable-address-sanitizer-flags. But, I suspect this is very dependent on race conditions between the GUI thread and the Octave thread.

Rik <rik5>
Project Administrator
Tue 10 Apr 2018 04:15:21 PM UTC, comment #2:

Adding bug #53513 as a dependency since I'm pretty sure they are related.

Rik <rik5>
Project Administrator
Tue 10 Apr 2018 03:56:25 PM UTC, comment #1:

This is probably related to the reason that I needed to add a delay after delete() for the uicontextmenu BIST tests in order for them to pass.

I think the delete operation for graphic objects needs to be reviewed completely.

Rik <rik5>
Project Administrator
Tue 10 Apr 2018 01:30:37 PM UTC, original submission:

The following makes Octave crash for me with the address sanitizer enabled:

Works OK with FLTK.

Pantxo Diribarne <pantxo>
Project Member

 

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

Attach Files:
   
   
Comment:
   

No files currently attached

 

Digest:
   bug dependencies.

Items that depend on this one: None found

 

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

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only project members can vote.

     

     

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2018-04-13 rik5 StatusNone => Invalid
        Open/ClosedOpen => Closed
    2018-04-13 rik5 Carbon-Copy- => Added sebald
    2018-04-10 rik5 Dependencies- => Depends on bugs #53513

    Back to the top


    Powered by Savane 3.4