src/common/downloader.cpp

@@ -197,7 +197,6 @@ bool Downloader::doDownload(const Download &dl, const QList<HTTPHeader> &headers
 	if (!file->open(QIODevice::WriteOnly)) {
 		qWarning("%s: %s", qPrintable(file->fileName()),
 		  qPrintable(file->errorString()));
-		delete file;
 		_errorDownloads.insert(url, RETRIES);
 		return false;
 	}

src/data/fitparser.cpp

@@ -1,6 +1,7 @@
 #include <QtEndian>
 #include "fitparser.h"
 
+
 #define FIT_MAGIC 0x5449462E // .FIT
 
 #define RECORD_MESSAGE  20
@@ -9,6 +10,60 @@
 #define COURSE_POINT    32
 #define TIMESTAMP_FIELD 253
 
+class Event {
+public:
+	Event() : id(0), type(0), data(0) {}
+
+	quint8 id;
+	quint8 type;
+	quint32 data;
+};
+
+struct FileHeader {
+	quint8 headerSize;
+	quint8 protocolVersion;
+	quint16 profileVersion;
+	quint32 dataSize;
+	quint32 magic;
+};
+
+struct FITParser::Field {
+	quint8 id;
+	quint8 size;
+	quint8 type;
+};
+
+class FITParser::MessageDefinition {
+public:
+	MessageDefinition() : endian(0), globalId(0), numFields(0), fields(0),
+	  numDevFields(0), devFields(0) {}
+	~MessageDefinition() {delete[] fields; delete[] devFields;}
+
+	quint8 endian;
+	quint16 globalId;
+	quint8 numFields;
+	Field *fields;
+	quint8 numDevFields;
+	Field *devFields;
+};
+
+class FITParser::CTX {
+public:
+	CTX(QFile *file, QVector<Waypoint> &waypoints)
+	  : file(file), waypoints(waypoints), len(0), endian(0), timestamp(0),
+	  ratio(NAN) {}
+
+	QFile *file;
+	QVector<Waypoint> &waypoints;
+	quint32 len;
+	quint8 endian;
+	quint32 timestamp;
+	MessageDefinition defs[16];
+	qreal ratio;
+	Trackpoint trackpoint;
+	SegmentData segment;
+};
+
 static QMap<int, QString> coursePointSymbolsInit()
 {
 	QMap<int, QString> map;
@@ -123,14 +178,22 @@ bool FITParser::skipValue(CTX &ctx, quint8 size)
 
 bool FITParser::parseDefinitionMessage(CTX &ctx, quint8 header)
 {
-	MessageDefinition *def = &(ctx.defs[header & 0x0f]);
-	quint8 numFields;
+	int local_id = header & 0x0f;
+	MessageDefinition *def = &(ctx.defs[local_id]);
+	quint8 i;
 
-	def->fields.clear();
-	def->devFields.clear();
+
+	if (def->fields) {
+		delete[] def->fields;
+		def->fields = 0;
+	}
+	if (def->devFields) {
+		delete[] def->devFields;
+		def->devFields = 0;
+	}
 
 	// reserved/unused
-	if (!skipValue(ctx, 1))
+	if (!readValue(ctx, i))
 		return false;
 
 	// endianness
@@ -146,35 +209,40 @@ bool FITParser::parseDefinitionMessage(CTX &ctx, quint8 header)
 	if (!readValue(ctx, def->globalId))
 		return false;
 
-	// definition records
-	if (!readValue(ctx, numFields))
+	// number of records
+	if (!readValue(ctx, def->numFields))
 		return false;
 
-	def->fields.resize(numFields);
-	for (int i = 0; i < def->fields.size(); i++) {
-		if (!readData(ctx.file, (char*)&(def->fields[i]), sizeof(Field)))
+	// definition records
+	def->fields = new Field[def->numFields];
+	for (i = 0; i < def->numFields; i++) {
+		static_assert(sizeof(def->fields[i]) == 3, "Invalid Field alignment");
+		if (!readData(ctx.file, (char*)&(def->fields[i]),
+		  sizeof(def->fields[i])))
 			return false;
-		ctx.len -= sizeof(Field);
+		ctx.len -= sizeof(def->fields[i]);
 	}
 
 	// developer definition records
 	if (header & 0x20) {
-		if (!readValue(ctx, numFields))
+		if (!readValue(ctx, def->numDevFields))
 			return false;
 
-		def->devFields.resize(numFields);
-		for (int i = 0; i < def->devFields.size(); i++) {
-			if (!readData(ctx.file, (char*)&(def->devFields[i]), sizeof(Field)))
+		def->devFields = new Field[def->numDevFields];
+		for (i = 0; i < def->numDevFields; i++) {
+			static_assert(sizeof(def->fields[i]) == 3, "Invalid Field alignment");
+			if (!readData(ctx.file, (char*)&(def->devFields[i]),
+			  sizeof(def->devFields[i])))
 				return false;
-			ctx.len -= sizeof(Field);
+			ctx.len -= sizeof(def->devFields[i]);
 		}
-	}
+	} else
+		def->numDevFields = 0;
 
 	return true;
 }
 
-bool FITParser::readField(CTX &ctx, const Field *field, QVariant &val,
-  bool &valid)
+bool FITParser::readField(CTX &ctx, Field *field, QVariant &val, bool &valid)
 {
 	bool ret;
 
@@ -231,20 +299,22 @@ bool FITParser::readField(CTX &ctx, const Field *field, QVariant &val,
 
 bool FITParser::parseData(CTX &ctx, const MessageDefinition *def)
 {
+	Field *field;
 	QVariant val;
 	bool valid;
 	Event event;
 	Waypoint waypoint;
 
-	if (!def->fields.size() && !def->devFields.size()) {
+
+	if (!def->fields && !def->devFields) {
 		_errorString = "Undefined data message";
 		return false;
 	}
 
 	ctx.endian = def->endian;
 
-	for (int i = 0; i < def->fields.size(); i++) {
-		const Field *field = &def->fields.at(i);
+	for (int i = 0; i < def->numFields; i++) {
+		field = &def->fields[i];
 		if (!readField(ctx, field, val, valid))
 			return false;
 		if (!valid)
@@ -346,9 +416,12 @@ bool FITParser::parseData(CTX &ctx, const MessageDefinition *def)
 		}
 	}
 
-	for (int i = 0; i < def->devFields.size(); i++)
-		if (!readField(ctx, &def->devFields.at(i), val, valid))
+	for (int i = 0; i < def->numDevFields; i++) {
+		field = &def->devFields[i];
+		if (!readField(ctx, field, val, valid))
 			return false;
+	}
+
 
 	if (def->globalId == EVENT_MESSAGE) {
 		if ((event.id == 42 || event.id == 43)  && event.type == 3) {
@@ -380,15 +453,15 @@ bool FITParser::parseData(CTX &ctx, const MessageDefinition *def)
 
 bool FITParser::parseDataMessage(CTX &ctx, quint8 header)
 {
-	int localId = header & 0xf;
-	MessageDefinition *def = &(ctx.defs[localId]);
+	int local_id = header & 0xf;
+	MessageDefinition *def = &(ctx.defs[local_id]);
 	return parseData(ctx, def);
 }
 
 bool FITParser::parseCompressedMessage(CTX &ctx, quint8 header)
 {
-	int localId = (header >> 5) & 3;
-	MessageDefinition *def = &(ctx.defs[localId]);
+	int local_id = (header >> 5) & 3;
+	MessageDefinition *def = &(ctx.defs[local_id]);
 	ctx.timestamp += header & 0x1f;
 	return parseData(ctx, def);
 }
@@ -414,6 +487,7 @@ bool FITParser::parseHeader(CTX &ctx)
 	quint16 crc;
 	qint64 len;
 
+	static_assert(sizeof(hdr) == 12, "Invalid FileHeader alignment");
 	len = ctx.file->read((char*)&hdr, sizeof(hdr));
 	if (len < 0) {
 		_errorString = "I/O error";
@@ -441,6 +515,7 @@ bool FITParser::parse(QFile *file, QList<TrackData> &tracks,
 	Q_UNUSED(polygons);
 	CTX ctx(file, waypoints);
 
+
 	if (!parseHeader(ctx))
 		return false;
 

src/data/fitparser.h

@@ -3,72 +3,25 @@
 
 #include "parser.h"
 
+class QFile;
+
 class FITParser : public Parser
 {
 public:
-	FITParser()
-	{
-		static_assert(sizeof(Field) == 3, "Invalid Field alignment");
-		static_assert(sizeof(FileHeader) == 12, "Invalid FileHeader alignment");
-	}
-
 	bool parse(QFile *file, QList<TrackData> &tracks, QList<RouteData> &routes,
 	  QList<Area> &polygons, QVector<Waypoint> &waypoints);
 	QString errorString() const {return _errorString;}
 	int errorLine() const {return 0;}
 
 private:
-	struct Event
-	{
-		Event() : data(0), id(0), type(0) {}
-
-		quint32 data;
-		quint8 id;
-		quint8 type;
-	};
-	struct FileHeader
-	{
-		quint8 headerSize;
-		quint8 protocolVersion;
-		quint16 profileVersion;
-		quint32 dataSize;
-		quint32 magic;
-	};
-	struct Field
-	{
-		quint8 id;
-		quint8 size;
-		quint8 type;
-	};
-	struct MessageDefinition
-	{
-		MessageDefinition() : globalId(0), endian(0) {}
-
-		QVector<Field> fields;
-		QVector<Field> devFields;
-		quint16 globalId;
-		quint8 endian;
-	};
-	struct CTX {
-		CTX(QFile *file, QVector<Waypoint> &waypoints)
-		  : file(file), waypoints(waypoints), len(0), endian(0), timestamp(0),
-		  ratio(NAN) {}
-
-		QFile *file;
-		QVector<Waypoint> &waypoints;
-		quint32 len;
-		quint8 endian;
-		quint32 timestamp;
-		MessageDefinition defs[16];
-		qreal ratio;
-		Trackpoint trackpoint;
-		SegmentData segment;
-	};
+	struct Field;
+	class MessageDefinition;
+	class CTX;
 
 	bool readData(QFile *file, char *data, size_t size);
 	template<class T> bool readValue(CTX &ctx, T &val);
 	bool skipValue(CTX &ctx, quint8 size);
-	bool readField(CTX &ctx, const Field *field, QVariant &val, bool &valid);
+	bool readField(CTX &ctx, Field *field, QVariant &val, bool &valid);
 
 	bool parseHeader(CTX &ctx);
 	bool parseRecord(CTX &ctx);