Skip Menu |
 

This queue is for tickets about the NetPacket CPAN distribution.

Report information
The Basics
Id: 7010
Status: resolved
Priority: 0/
Queue: NetPacket

People
Owner: yanick+cpan [...] babyl.dyndns.org
Requestors: hoagland [...] sourceforge.net
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: (no value)
Fixed in: 0.41_0



Subject: IP flags field lost in IP::encode()
Download (untitled) / with headers
text/plain 423b
I think I reported this a while back, to make sure... The IP flags field is not included in the raw packet returned from IP::encode(). However, it is included correctly in the checksum calculation, so people may see this error as a bad checksum error. The fix is to change this line: $self->{id}, $self->{foffset}, $self->{ttl}, $self->{proto}, to $self->{id}, $offset, $self->{ttl}, $self->{proto},
From: hoagland [...] sourceforge.net
Download (untitled) / with headers
text/plain 542b
[guest - Mon Jul 19 15:13:00 2004]: Show quoted text
> I think I reported this a while back, to make sure... > The IP flags field is not included in the raw packet returned from > IP::encode(). However, it is included correctly in the checksum > calculation, so people may see this error as a bad checksum error. > > The fix is to change this line: > $self->{id}, $self->{foffset}, $self->{ttl}, $self->{proto}, > to > $self->{id}, $offset, $self->{ttl}, $self->{proto},
Actually, I had reported it before and it is fixed in version 0.04
From: hoagland [...] sourceforge.net
Download (untitled) / with headers
text/plain 763b
[guest - Mon Jul 19 15:36:33 2004]: Actually, this was not fixed correctly in 0.04. The checksum will be correct with respect to the encoded packet, but the IP flags field is still lost. Show quoted text
> [guest - Mon Jul 19 15:13:00 2004]: >
> > I think I reported this a while back, to make sure... > > The IP flags field is not included in the raw packet returned from > > IP::encode(). However, it is included correctly in the checksum > > calculation, so people may see this error as a bad checksum error. > > > > The fix is to change this line: > > $self->{id}, $self->{foffset}, $self->{ttl}, $self- > >{proto}, > > to > > $self->{id}, $offset, $self->{ttl}, $self->{proto},
> > Actually, I had reported it before and it is fixed in version 0.04


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.