Revert last change on new logfile structure and added nulling of buffer.

The reason for this is, because the last change in the file structure is buggy, it was only written the message length of the buffer to the file and missed to add the length for the timestamp.
For fixing this problem it would be possible to insert something like

len += sizeof(quint64);

after:

int len = mavlink_msg_to_send_buffer(buf+sizeof(quint64), &message);

But I think a consistent file structure has more adavatages for a logfile over a more compressed structure.
E.g. it would be necessary to go through all packages to proceed a jump during a replay.
Additional the LogPlayer was not updated during the last 7 month since the logfile structure was changed (even it was buggy)...

If filesize is really a problem I would recommend to change the logfile handling in a way of writting a temporary consistend
logfile and compress it afterwards (QCompress) like done by many programs

src/comm/MAVLinkProtocol.cc

@@ -291,11 +291,14 @@ void MAVLinkProtocol::receiveBytes(LinkInterface* link, QByteArray b)
             // Log data
             if (m_loggingEnabled && m_logfile)
             {
-                uint8_t buf[MAVLINK_MAX_PACKET_LEN+sizeof(quint64)];
+                uint8_t buf[MAVLINK_MAX_PACKET_LEN+sizeof(quint64)] = {0};
                 quint64 time = QGC::groundTimeUsecs();
                 memcpy(buf, (void*)&time, sizeof(quint64));
                 // Write message to buffer
-                int len = mavlink_msg_to_send_buffer(buf+sizeof(quint64), &message);
+                mavlink_msg_to_send_buffer(buf+sizeof(quint64), &message);
+                //we need to write the maximum package length for having a
+                //consistent file structure and beeing able to parse it again
+                int len = MAVLINK_MAX_PACKET_LEN + sizeof(quint64);
                 QByteArray b((const char*)buf, len);
                 if(m_logfile->write(b) != len)
                 {