#include "sfpacket.h"
-SFPacket::SFPacket(int pType, int pSeqno)
-{
+SFPacket::SFPacket(int pType, int pSeqno) {
length = 0;
seqno = pSeqno;
type = pType;
}
// copy constructor
-SFPacket::SFPacket(const SFPacket &pPacket)
-{
+SFPacket::SFPacket(const SFPacket &pPacket) {
length = pPacket.getLength();
type = pPacket.getType();
seqno = pPacket.getSeqno();
const char* SFPacket::getPayload() const
{
- if ( ((type == SF_PACKET_ACK) || (type == SF_PACKET_NO_ACK)))
- {
- return payloadBuffer;
+ if(((type == SF_PACKET_ACK) || (type == SF_PACKET_NO_ACK))) {
+ return buffer + 1;
}
- else
- {
+ else {
return NULL;
}
}
-int SFPacket::getLength() const
-{
+int SFPacket::getLength() const {
return length;
}
if ((pLength > 0) && (pLength < cMaxPacketLength) && ((type == SF_PACKET_ACK) || (type == SF_PACKET_NO_ACK)))
{
length = pLength;
- for (int i=0; i < pLength; i++)
- {
- payloadBuffer[i] = *(pBuffer+i);
- }
+ memcpy(buffer + 1, pBuffer, pLength);
return true;
}
DEBUG("SFPACKET::setPayload : wrong packet length = " << static_cast<int>(pLength) << " or type = " << type)
bool SFPacket::operator==(SFPacket const& pPacket)
{
bool retval=false;
- if (!((pPacket.getType() != type) || (pPacket.getLength() != length) || pPacket.getSeqno() != seqno))
- {
- if ((type == SF_PACKET_ACK) || (type == SF_PACKET_NO_ACK))
- {
- const char* cmpBuffer = pPacket.getPayload();
- if (cmpBuffer) {
- retval = true;
- // compare buffers
- for (int i=0; i < length; i++)
- {
- if (payloadBuffer[i] != cmpBuffer[i])
- {
- i = length;
- retval = false;
- }
- }
- }
- }
- else
- {
- retval = true;
+ if((pPacket.getType() == type) && (pPacket.getLength() == length) && (pPacket.getSeqno() == seqno)) {
+ if((type == SF_PACKET_ACK) || (type == SF_PACKET_NO_ACK)) {
+ retval = (memcmp(pPacket.getPayload(), getPayload(), length) == 0);
}
}
return retval;
}
+ /* return the length that shall be transmitted via TCP */
+int SFPacket::getTcpLength() const {
+ return length + 1;
+}
+
+/* return the payload of the TCP packet */
+const char* SFPacket::getTcpPayload() {
+ char l = length;
+ buffer[0] = l;
+ return buffer;
+}
+
/** member vars **/
protected:
- /* payload buffer */
- char payloadBuffer[cMaxPacketLength];
+ /* internal buffer */
+ char buffer[cMaxPacketLength + 1];
+
/* length of byte buffer */
int length;
/* type */
/* returns length of buffer */
int getLength() const;
+ /* return the length that shall be transmitted via TCP */
+ int getTcpLength() const;
+
+ /* return the payload of the TCP packet */
+ const char* getTcpPayload();
+
/* returns the seqno of this packet */
int getSeqno() const;
/* writes packet */
bool TCPComm::writePacket(int pFD, SFPacket &pPacket)
{
- char len = pPacket.getLength();
+ int len = pPacket.getTcpLength();
int err;
- if (writeFD(pFD, &len, 1, &err) != 1)
- {
- return false;
- }
- if (writeFD(pFD, pPacket.getPayload(), len, &err) != len)
- {
- return false;
- }
- return true;
+ return (writeFD(pFD, pPacket.getTcpPayload(), len, &err) == len);
}
/* checks for correct version of SF protocol */