Bug report #5121

(regression) qgis crashes if select a serial port and push the connect button in the gps Information Panel (gps tracker)

Added by Thomas Arnold almost 13 years ago. Updated almost 13 years ago.

Status:Closed
Priority:High
Assignee:-
Category:-
Affected QGIS version:master Regression?:No
Operating System: Easy fix?:No
Pull Request or Patch supplied:No Resolution:fixed
Crashes QGIS or corrupts data:Yes Copied to github as #:14880

Description

Hi,

my garmin etrex and other gps reciver (with NMEA output) works fine under QGIS 1.7.1 with the GPS information panel if I set the COM-port manuel (under option->connection I have select the correct serial device COM-port).
But with qgis 1.7.4 or qgis master brunch qgis crashed if I pushed the connect button. I have not tested this problem under qgis 1.7.2 and 1.7.3.

If I select under option->connection->autodetect than qgis have no problems.
I use qgis under windowsXP and windows 7.

Thomas

qgsgpsinformationwidget.diff Magnifier (465 Bytes) Thomas Arnold, 2012-03-19 12:17 AM

History

#1 Updated by Giovanni Manghi almost 13 years ago

  • Priority changed from Low to 6

#2 Updated by Giovanni Manghi almost 13 years ago

  • Subject changed from qgis crashes if select a serial port and push the connect button in the gps Information Panel (gps tracker) to (regression) qgis crashes if select a serial port and push the connect button in the gps Information Panel (gps tracker)

#3 Updated by Thomas Arnold almost 13 years ago

Hi,
can anybody check the following solution for this problem?

QgsGPSInformationWidget::connectGps(){
...
//replace follow line:
//port = mCboDevices->currentText();
port = mCboDevices->itemData(mCboDevices->currentIndex()).toString();
...
}

#4 Updated by Giovanni Manghi almost 13 years ago

Hi Thomas,
can you attach a patch made with "diff"?

Thomas Arnold wrote:

Hi,
can anybody check the following solution for this problem?

QgsGPSInformationWidget::connectGps(){
...
//replace follow line:
//port = mCboDevices->currentText();
port = mCboDevices->itemData(mCboDevices->currentIndex()).toString();
...
}

#5 Updated by Thomas Arnold almost 13 years ago

Hi,

is that file ok?

#6 Updated by Jürgen Fischer almost 13 years ago

  • Resolution set to fixed
  • Status changed from Open to Closed

Thomas Arnold wrote:

is that file ok?

Yes. Thanks. Applied in 6972cfa

Also available in: Atom PDF