[kde-edu]: KDE/kdeedu/libkdeedu/extdate

Jason Harris kstars at 30doradus.org
Fri Sep 15 23:46:03 CEST 2006


SVN commit 584944 by harris:

Fixed crash condition in ExtDate::fromString(QString, QDateFormat).
When using the TextFormat QDateFormat, the code creates a QStringList 
from the space-separated fields of the input string, but it then assumed 
that the stringlist contains at least three members, which caused a 
crash when this was not the case.  Now, it will return an invalid 
ExtDate if the stringlist contains fewer than three members.

Thanks for the tip, Jasem.

CCMAIL: kde-edu at kde.org
CCMAIL: kstars-devel at kde.org



 M  +4 -0      extdatetime.cpp  


--- trunk/KDE/kdeedu/libkdeedu/extdate/extdatetime.cpp #584943:584944
@@ -494,6 +494,10 @@
 			//Three possible date formats:
 			//dd mth yyyy; mth dd yyyy; wkd mth dd yyyy
 			QStringList ss = s.split( " " );
+
+			//Return invalid if we don't have at least 3 fields
+			if ( ss.size() < 3 ) return dt;
+
 			bool ok = false;
 			int month = -1;
 			uint imonth = 0;


More information about the kde-edu mailing list