Patch QGIS bug #5809

This commit is contained in:
Volker Fröhlich 2012-07-18 22:54:22 +02:00
parent 0d0eec9a21
commit cf419dc326
2 changed files with 46 additions and 1 deletions

View File

@ -0,0 +1,37 @@
commit 23a3412113a04fac0bdafffe9fc8a48c2a39e8ea
Author: Juergen E. Fischer <jef@norbit.de>
Date: Wed Jun 20 21:01:25 2012 +0200
fix #5809
diff --git a/src/providers/delimitedtext/qgsdelimitedtextprovider.cpp b/src/providers/delimitedtext/qgsdelimitedtextprovider.cpp
index 7b0bbb6..5b9837a 100644
--- a/src/providers/delimitedtext/qgsdelimitedtextprovider.cpp
+++ b/src/providers/delimitedtext/qgsdelimitedtextprovider.cpp
@@ -139,6 +139,8 @@ QgsDelimitedTextProvider::QgsDelimitedTextProvider( QString uri )
, mWktHasZM( false )
, mWktZMRegexp( "\\s+(?:z|m|zm)(?=\\s*\\()", Qt::CaseInsensitive )
, mWktCrdRegexp( "(\\-?\\d+(?:\\.\\d*)?\\s+\\-?\\d+(?:\\.\\d*)?)\\s[\\s\\d\\.\\-]+" )
+ , mFile( 0 )
+ , mStream( 0 )
, mSkipLines( 0 )
, mFirstDataLine( 0 )
, mShowInvalidLines( false )
@@ -212,6 +214,7 @@ QgsDelimitedTextProvider::QgsDelimitedTextProvider( QString uri )
{
QgsDebugMsg( "Data source " + dataSourceUri() + " could not be opened" );
delete mFile;
+ mFile = 0;
return;
}
@@ -447,7 +450,8 @@ QgsDelimitedTextProvider::QgsDelimitedTextProvider( QString uri )
QgsDelimitedTextProvider::~QgsDelimitedTextProvider()
{
- mFile->close();
+ if ( mFile )
+ mFile->close();
delete mFile;
delete mStream;
}

View File

@ -15,7 +15,7 @@
Name: qgis Name: qgis
Version: 1.8.0 Version: 1.8.0
Release: 5%{?dist} Release: 6%{?dist}
Summary: A user friendly Open Source Geographic Information System Summary: A user friendly Open Source Geographic Information System
# Update FSF address or ship a GPLv3+ license file # Update FSF address or ship a GPLv3+ license file
@ -41,6 +41,10 @@ Patch0: %{name}-1.5.0-grass.patch
# Drop -o option from SIP, as versions older than 4.10 don't have it # Drop -o option from SIP, as versions older than 4.10 don't have it
Patch1: %{name}-1.8.0-sip410.patch Patch1: %{name}-1.8.0-sip410.patch
# http://hub.qgis.org/issues/5809
# Crash when datasource is moved or deleted
Patch2: %{name}-1.8.0-datasource-crash.patch
# Leaving it to make life easier for ELGIS, as long as they target RHEL 5 # Leaving it to make life easier for ELGIS, as long as they target RHEL 5
BuildRoot: %(mktemp -ud %{_tmppath}/%{name}-%{version}-%{release}-XXXXXX) BuildRoot: %(mktemp -ud %{_tmppath}/%{name}-%{version}-%{release}-XXXXXX)
@ -147,6 +151,7 @@ Please refer to %{name}-mapserver-README.fedora for details!
%setup -q %setup -q
%patch0 -p1 -b .grass~ %patch0 -p1 -b .grass~
%patch1 -p1 -b .sip410~ %patch1 -p1 -b .sip410~
%patch2 -p1 -b .datasource~
# Readme file for QGIS mapserver configuration and Lighttpd sample # Readme file for QGIS mapserver configuration and Lighttpd sample
install -pm0644 %{SOURCE4} . install -pm0644 %{SOURCE4} .
@ -350,6 +355,9 @@ gtk-update-icon-cache %{_datadir}/icons/hicolor &>/dev/null || :
%{_libexecdir}/%{name} %{_libexecdir}/%{name}
%changelog %changelog
* Wed Jul 18 2012 Volker Fröhlich <volker27@gmx.at> - 1.8.0-6
- Add patch for QGIS bug #5809
* Sat Jul 7 2012 Volker Fröhlich <volker27@gmx.at> - 1.8.0-5 * Sat Jul 7 2012 Volker Fröhlich <volker27@gmx.at> - 1.8.0-5
- Rebuilt too quick - Rebuilt too quick