MantisBT - ParaView
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0013746ParaView(No Category)public2012-12-20 05:332015-01-11 08:28
ReporterFelipe Bordeu 
Assigned ToUtkarsh Ayachit 
PriorityhighSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version3.98.0 
Target VersionFixed in Version3.98.1 
ProjectTBD
Topic Name13746_fix_change_input_segfaults
Typecrash
Summary0013746: Pipeline change input crash
DescriptionParaview Crash when trying to the the input of a filter and then delete the old input.

to reproduce this bug on ParaView 3.98.0 (official binary) (Linux 64bit ubuntu 12.04).

Sources -> Wavelet (Wavelet1)
Calculator (Calulator1)
Sources -> Wavelet (Wavelet2)

Now right click to the calculator1 and change input to the calculator2

Finally delete Wavelet1 and Paraview Crashes.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2012-12-20 05:33Felipe BordeuNew Issue
2012-12-20 06:04Utkarsh AyachitAssigned To => Utkarsh Ayachit
2012-12-20 07:32Utkarsh AyachitTopic Name => 13746_fix_change_input_segfaults
2012-12-20 07:32Utkarsh AyachitNote Added: 0029918
2012-12-20 07:32Utkarsh AyachitStatusbacklog => gatekeeper review
2012-12-20 07:32Utkarsh AyachitFixed in Version => git-next
2012-12-20 07:32Utkarsh AyachitResolutionopen => fixed
2012-12-21 11:20Utkarsh AyachitFixed in Versiongit-next => git-master
2012-12-21 11:21Utkarsh AyachitStatusgatekeeper review => customer review
2012-12-21 11:21Utkarsh AyachitNote Added: 0029938
2013-01-08 21:58Alan ScottNote Added: 0030034
2013-01-08 21:58Alan ScottStatuscustomer review => closed
2013-01-30 10:41Utkarsh AyachitFixed in Versiongit-master => 3.98.1
2015-01-11 08:28Utkarsh AyachitSource_changeset_attached => ParaView master 6d0e9e8f
2015-01-11 08:28Utkarsh AyachitSource_changeset_attached => ParaView master 0eca8ac6

Notes
(0029918)
Utkarsh Ayachit   
2012-12-20 07:32   
commit 0eca8ac665dadd9ccc5dd91e29c79f4649bf9eaf
Author: Utkarsh Ayachit <utkarsh.ayachit@kitware.com>
Date: Thu Dec 20 07:28:28 2012 -0500

    BUG 0013746. Fix segfaults after change-input.
    
    The bug was introduced by 38c98eb. Due to the QueuedConnection added, slots were
    being called with dangling pointer which resulted in the segfaults. Fixed by
    keeping track of the pointer of interest in a QList with QPointers and then
    on-idle handling non-null items in the QList.
    
    Change-Id: I149e86389c5abec8c05050f85dd982c89b1dfc8a
(0029938)
Utkarsh Ayachit   
2012-12-21 11:21   
merged into master, if applicable.
(0030034)
Alan Scott   
2013-01-08 21:58   
Tested local server, Linux, master.