--- smsdaemon/SmsPdu.cpp	2008/06/13 09:09:36	73
+++ smsdaemon/SmsPdu.cpp	2008/12/11 22:32:58	176
@@ -10,14 +10,12 @@
 #include <time.h>
 #include <stdlib.h> 
 
-#include "common.h"
-#include "util.h"
+#include "Logger.h"
+#include "Util.h"
 
-#include <iostream>
 
 using namespace std;
 
-
 namespace SmsPdu
 {
 
@@ -146,7 +144,6 @@
 
 		const unsigned char UDHI = multipart ? 0x40 : 0;
 
-		srand(time(0));
 		unsigned char csms_ref = rand() % 128;
 
 		int part_count;
@@ -157,7 +154,7 @@
 		{
 			if (message.length() > 800)
 			{
-				Common::instance()->logMessage("Trying to send multipart sms > 800 bytes !!!");
+				Logger::logMessage("Trying to send multipart sms > 800 bytes !!!");
 				message = message.substr(0,800);
 			}
 
@@ -193,7 +190,7 @@
 			if (multipart)
 			{
 				message_part = message.substr(0, PDU_LEN);
-				message.erase(0, PDU_LEN);
+				message.erase(0, PDU_LEN-1);
 
 				pdu.push_back( message_part.length()+ 7 );  //UserDataLength
 				pdu.push_back( 0x06 ); // UDH Len
@@ -210,7 +207,7 @@
 				if (message.length() > 160)
 				{
 					message_part = message.substr(0,160); //truncate to 160
-					Common::instance()->logMessage("Truncated message");
+					Logger::logMessage("Truncated message");
 				}
 				else
 				{
@@ -277,9 +274,38 @@
 	}
 
 
+
 	SMS ParseSmsPdu(std::string pdu_str)
 	{
-		SMS result;
+		SmsPart part = ParseSmsPduWorker(pdu_str);
+
+
+		SMS sms;
+		sms.SetMessage(part.message);
+		sms.SetSender(part.sender);
+		return sms;
+	}
+
+	void ParseUdh(vector<unsigned char>& udh, SmsPart& part)
+	{
+		if (udh.size() == 0) {
+			Logger::logMessage("ParseUdh(): empty udh");
+			return;
+		}
+	
+		if (udh[0] != 0) {
+			Logger::logMessage("unknown UDH type");
+			return;
+		}
+		
+		part.group = udh[2];
+		part.count = udh[3];
+		part.id = udh[4];
+	}
+
+
+	SmsPart ParseSmsPduWorker(std::string pdu_str)
+	{
 
 		vector<unsigned char> pdu = HexDecodeString(pdu_str);
 
@@ -297,38 +323,53 @@
 
 		++it; //ignore Type-Of-Address
 
-		result.sender = DecodeRawPhonenr( it, it+(sender_len/2) );
+		string sender = DecodeRawPhonenr( it, it+(sender_len/2) );
 
 		it += (sender_len/2);
 		++it; //protocol identifier
 		++it; //Data encoding
 
-		result.timestamp = DecodeTimestamp(it, it+7);
+		string timestamp = DecodeTimestamp(it, it+7);
 		it += 7;
 
 
 		unsigned char data_len = (*it++);
 
+
+		SmsPart part;
+		part.group = -1;
+
 		int shift_start = 0;
+
 		if (UDHI)
 		{
 			int udh_len = (*it++); 
-			cout << "UDH_LEN:" << udh_len << endl;
-			it += udh_len; //just ignore the User Data Header
+
+			vector<unsigned char> udh;
+			for (int i=0; i<udh_len; i++)
+			{
+				udh.push_back (*it++);
+			}
+			ParseUdh(udh,part);
+
 			data_len -= udh_len;
 
 			shift_start = udh_len+1; //make the 8to7bit decode start with the right shift level
 		}
-		cout << hex<< (int)(*it) << endl;;
 
 
 		vector<unsigned char> user_data;
 		user_data.insert(user_data.end(), it, it+data_len);
 		
-		result.message = Decode8to7bit(user_data, shift_start);
+		string message = Decode8to7bit(user_data, shift_start).substr(0,data_len);
+
+		message = Util::str_trim(message);
+
 		
+		part.message = message;
+		part.sender = sender;
 
-		return result;
+		return part;
 	}
 
 }