Bug 552 – Segmentation fault when using QUiLoader and QTabWidget

PySide Bugzilla Closed for New Bugs

PySide is now a Qt Add-on and uses the Qt Project's JIRA Bug Tracker instead of this Bugzilla instance. This Bugzilla is left for reference purposes.

Bug 552 - Segmentation fault when using QUiLoader and QTabWidget

Summary:

Segmentation fault when using QUiLoader and QTabWidget

Status:	CLOSED FIXED

Product:	PySide
Classification:	Unclassified
Component:	PySide
Version:	HEAD
Platform:	PC Linux

Importance:	P2 major
Assigned To:	Hugo Parente Lima

URL:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2010-12-17 10:27 EET by Piotr Sokołowski
Modified:	2011-01-21 15:44 EET (History)
CC List:	8 users (show)

See Also:

Attachments
Test program (547 bytes, text/x-python) 2010-12-17 10:27 EET, Piotr Sokołowski	Details
Qt Designer file (906 bytes, application/x-designer) 2010-12-17 10:27 EET, Piotr Sokołowski	Details
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Piotr Sokołowski 2010-12-17 10:27:03 EET

Created attachment 189 [details]
Test program

When using QUiLoader Segfault appear during program exit (in test code line
"app.exec()" is commented out to force closing program without user
interaction).
After commenting out "removeTab()" or after detaching variable "children" from
"View_1" class by changing "self.children" to "children" program exits without
segfault.
It's look like the problem is deleting widget children twice:
- When calling "removeTab()",
- When destroying class "View_1"

Comment 1 Piotr Sokołowski 2010-12-17 10:27:36 EET

Created attachment 190 [details]
Qt Designer file

Comment 2 Matti Airas 2010-12-17 11:09:52 EET

Thanks for the bgu report. There are some design issues in QUILoader at the
moment and its use is not recommended at the moment. See for example bug 533
for details.

I'm sorry we've failed to announce these issues properly. The recommended
workaround at this point is to convert the .ui files to .py using pyside-uic
and load the ui definitions by importing the .py file.

We'll be rewriting the QUILoader functionality after the 1.0 release. Hence,
prioritizing this bug P4 for now.

Comment 3 Hugo Parente Lima 2011-01-13 14:03:55 EET

Matti, I think this should be a P2 or P3, because I guess it's not a QtUiLoader
problem but a QTabWidget one, what do you think?

Comment 4 Matti Airas 2011-01-14 07:29:11 EET

Hugo, thanks for the clarification. I'm making it P2 in that case (because it's
a crasher bug).

Comment 5 Hugo Parente Lima 2011-01-21 10:32:55 EET

Fixed in commits:

shiboken/4721af29a2225389528ba5984c53ab551058e690
pyside/b9679215b8eb39d18a774339828f55e57d479c5c

Comment 6 renato filho 2011-01-21 15:44:44 EET

release beta4