]> The Tcpdump Group git mirrors - libpcap/blob - TODO
Declare "install_bpf_program()" in "pcap-int.h", not "gencode.h"; it has
[libpcap] / TODO
1 TODO list for libpcap
2 =======================
3
4 Important stuff (to be done before the next release)
5 ---------------
6
7 General
8
9 - configure should not be in the CVS. Most open source projects have an
10 autogen.sh script to run autoconf etc. after checkout. I think we
11 should stick to the standard.
12
13 - The source files should be better documented. There is no official
14 design guideline what is done where. There should be a common coding
15 style (okay, you can guess that bye looking at the code) and a guideline
16 what needs to be documented.
17
18 Linux kernel interface
19
20 - Currently there is a race condition in that a socket is activated at the
21 same time when it is opened - before applying a filter. This has to
22 be corrected so that capture starts when pcap_read is called for the
23 first time.
24
25 - For what is that silly linux-include directory needed? This should
26 a) be documented
27 b) not be neccessary. libpcap builds for me without that directory and
28 I don't think it is our responsible to work around broken build
29 environments. If we detect a problem configure should fail but that's
30 about it.
31
32
33 Less urgent items
34 -----------------
35
36 - Better documentation and cleanup of the interface. I am seeing a few
37 problems at the first glance which needs fixing:
38 + pcap_lookupnet makes little to no sense with protocols != IPv4
39 + not very suited for interactive programs (think ethereal). There should
40 be a way for the application to get a file descriptor which it has to
41 monitor and a callback in pcap which has to be called on activity
42 + too many functions. There are a lot of functions for everything which
43 violates the KISS principle. Why do we need pcap_strerror, pcap_perror
44 and pcap_geterr?
45 + the manpage has a brief description of each function but where is the
46 big picture? Seems like you need to buy UNP for that...