Skip Menu |

This queue is for tickets about the Net-Pcap CPAN distribution.

Report information
The Basics
Id: 27369
Status: resolved
Priority: 0/
Queue: Net-Pcap

People
Owner: SAPER [...] cpan.org
Requestors: Russell.Mosemann [...] cune.edu
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 0.15



Subject: Various bugs and fixes for 0.14 and 0.15_01
Date: Fri, 1 Jun 2007 09:43:10 -0500
To: <bug-Net-Pcap [...] rt.cpan.org>
From: "Mosemann, Russell" <Russell.Mosemann [...] cune.edu>
1. In Pcap.pm "functions", "set_filter" should be "setfilter". 2. In Pcap.pm "functions", "stats" is missing. 3. The documentation in Pcap.pm for 0.15_01 incorrectly lists Net::Pcap::get_selectable_fdfileno, which causes the actual function not to be found. It should be changed to Net::Pcap::get_selectable_fd. 4. The documentation lists pcap_next_ex, but it is next_ex. 5. next_ex returns the header information, but it does not return the packet information. That's because the following line in Pcap.xs is wrong. pkt_data = newSVpv(data, header->caplen); pkt_data is not an SV. It is a reference to an SV. Even if pkt_data was an SV, you still don't want to create a new SV. Change the existing SV. The line above should be changed to sv_setpvn(SvRV(pkt_data), data, header->caplen); The line above dereferences pkt_data to get the SV, and then the function changes the value of that SV. -- Russell Mosemann, Ph.D. Associate Professor of Computer Science
Subject: Re: [rt.cpan.org #27369] Various bugs and fixes for 0.14 and 0.15_01
Date: Fri, 1 Jun 2007 19:29:03 +0200
To: bug-Net-Pcap [...] rt.cpan.org
From: Sébastien Aperghis-Tramoni <saper [...] cpan.org>
Hello, Show quoted text
> 1. In Pcap.pm "functions", "set_filter" should be "setfilter". > 2. In Pcap.pm "functions", "stats" is missing.
Fixed. Show quoted text
> 3. The documentation in Pcap.pm for 0.15_01 incorrectly lists > Net::Pcap::get_selectable_fdfileno, which causes the actual > function not > to be found. It should be changed to Net::Pcap::get_selectable_fd.
This was already fixed in the development version. Show quoted text
> 4. The documentation lists pcap_next_ex, but it is next_ex.
Fixed. I also added an alias pcap_next_ex() because it's more coherent with pcap_next(). Show quoted text
> 5. next_ex returns the header information, but it does not return the > packet information. [...]
Well spotted. This was also the subject of CPAN-RT#25076, and the fix has already been applied in the development version. Your suggestions have been applied. Thanks for the time you spent on Net::Pcap. Best Regards, -- Sébastien Aperghis-Tramoni Close the world, txEn eht nepO.