Bug 632 – QLineEdit.getTextMargins() segmentation fault

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 632 - QLineEdit.getTextMargins() segmentation fault

Summary:

QLineEdit.getTextMargins() segmentation fault

Status:	CLOSED FIXED

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

Importance:	P2 blocker
Assigned To:	renato filho

URL:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2011-01-20 21:03 EET by Daniele Esposti
Modified:	2011-02-02 15:46 EET (History)
CC List:	9 users (show)

See Also:

Attachments
QLineEdit.getTextMargins() lazy test (347 bytes, text/plain) 2011-01-20 21:03 EET, Daniele Esposti	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 Daniele Esposti 2011-01-20 21:03:17 EET

Created attachment 232 [details]
QLineEdit.getTextMargins() lazy test

Calling the getTextMargins() method of the QLineEdit class produce a
segmentation fault.
I've attached a small example to reproduce the problem: the call to the
getTextMargins() method at line 17 drops to a Segmentation fault instead to be
executed fine.

Comment 1 Matti Airas 2011-01-21 05:26:27 EET

Thanks for the bug report.

A brief inspection shows that the function signature is indeed all wrong in
PySide: the C++ method has pointers to integers, via which values are returned,
and the Python version respectively takes four integers as arguments. This has
no chance of working and has to changed so that the function takes no arguments
and returns a tuple of four elements (as in PyQt).

I'm prioritizing this P2 because it's a crasher bug.

Comment 2 renato filho 2011-01-24 13:48:53 EET

fixed on pyside commit:

commit 9a4c439d7ff3926c87a2558cf42c9a69ff4a4734
Author: Renato Araujo Oliveira Filho <renato.filho@openbossa.org>
Date:   Mon Jan 24 11:13:27 2011 -0300

Comment 3 renato filho 2011-02-02 15:46:10 EET

released on beta 5