X-Git-Url: http://g0dil.de/git?a=blobdiff_plain;f=Packets%2FDefaultBundle%2FUDPPacket.cc;h=f3cfc56eb353e10fb5718d42e8bb1ac9c0423f92;hb=5443435c4c2b6e4386c5334b5b8358273f2bae93;hp=ee0651f9c22569842a5f86edc8ce715933020650;hpb=14daa84b7c821626ae098bedd93ef07d98bf3a40;p=senf.git diff --git a/Packets/DefaultBundle/UDPPacket.cc b/Packets/DefaultBundle/UDPPacket.cc index ee0651f..f3cfc56 100644 --- a/Packets/DefaultBundle/UDPPacket.cc +++ b/Packets/DefaultBundle/UDPPacket.cc @@ -1,8 +1,8 @@ // $Id$ // // Copyright (C) 2006 -// Fraunhofer Institute for Open Communication Systems (FOKUS) -// Competence Center NETwork research (NET), St. Augustin, GERMANY +// Fraunhofer Institute for Open Communication Systems (FOKUS) +// Competence Center NETwork research (NET), St. Augustin, GERMANY // Stefan Bund // // This program is free software; you can redistribute it and/or modify @@ -50,10 +50,7 @@ prefix_ boost::uint16_t senf::UDPPacketParser::calcChecksum() const { IpChecksum summer; - summer.feed( i(), i()+checksum_offset ); - summer.feed( i()+checksum_offset+2, data().end() ); - - // Now on to the awkward part: the IP pseudo header + // first on to the awkward part: the IP pseudo header IPv4Packet ipv4 (packet().rfind(nothrow)); if (ipv4) { // Pseudo header defined in RFC768 @@ -88,6 +85,11 @@ prefix_ boost::uint16_t senf::UDPPacketParser::calcChecksum() } } + // since header are 16 / even 32bit aligned we don't have to care for padding. since IpChecksum + // cares for padding at the final summing we don't have to care is the payload is 16nbit-aligned, too. + summer.feed( i(), i()+checksum_offset ); + summer.feed( i()+checksum_offset+2, data().end() ); + boost::uint16_t rv (summer.sum()); return rv ? rv : 0xffffu; }