Sorry for the delay - I was out of town with no access to email...

I pulled the latest source from the CVS with your changes checked in
this morning:

        File: libpcap/pcap-dlpi.c; Revision: 1.111;
        Date: 2005/04/17 00:47:40; Author: guy; Lines: (+116 -60)

A single typo:

--- pcap-dlpi.c_orig    2005-04-16 20:47:40 -0400
+++ pcap-dlpi.c 2005-04-17 08:32:35 -0400
@@ -1116,7 +1116,7 @@
 }
 
 static int
-recv_ack(int fd, int size, const char *what, char *bufp, char *ebuf,
int *uerr)
+recv_ack(int fd, int size, const char *what, char *bufp, char *ebuf,
int *uerror)
 {
        union   DL_primitives   *dlp;
        struct  strbuf  ctl;

Testing with DLSAP 22 & 24 busy:

# ./tcpdump -i lan0 -c 1            
tcpdump: WARNING: recv_ack: bind: UNIX error - Device busy
tcpdump: verbose output suppressed, use -v or -vv for full protocol
decode
listening on lan0, link-type EN10MB (Ethernet), capture size 96 bytes
09:02:02.697705 IP c4430tn5.telnet > 16.123.194.86.4206: P
3961913033:3961913041 (8) ack 937687938 win 32768
1 packets captured
1 packets received by filter
0 packets dropped by kernel

Looks good - Thanks!


Regards,

Mark C. Brown,
Hewlett-Packard Company
Global Complex Solutions Management
Global Solutions Engineering
[EMAIL PROTECTED]
<https://ecardfile.com/id/mbrown> 

As always, please let me know if you prefer not to receive email
messages from me.  For More information regarding Hewlett-Packard's
privacy policy or to obtain information, please see our privacy
statement at:

http://welcome.hp.com/country/us/en/privacy.html


> -----Original Message-----
> Sent: Wednesday, April 13, 2005 04:39
> To: tcpdump-workers@lists.tcpdump.org
> Subject: Re: [tcpdump-workers] libpcap: HP-UX Patch for 
> configurable DLSAP
> 
> Brown, Mark C (GSE GCSM) wrote:
> 
> > I tested SAP values 0 to 100 and they all worked on 11.11 and 11.23.
> > Here's a new patch that tries SAP values 22 to 100.  I didn't
> > want to change libpcap's default behavior which is why I start
> > with 22.
> 
> OK, here's a modified version of the patch, which centralizes 
> the bind 
> loop in a dl_dohpuxbind() routine, continues the loop only if 
> the bind 
> attempt fails with EBUSY, cleans up a bit of the DL_HP_RAWDLS 
> stuff (it 
> does the "send FD" binding at the same time it does the other binding 
> for HP-UX, and doesn't do it at all for HP-UX prior to 9.0 (I 
> don't even 
> have 9.0 documentation, so I don't even know whether it works in 9.0, 
> and I'm not sure we support pre-9.0 at all - and I'm not sure there's 
> anybody using it on any pre-10.20 release in any case), and fixes up 
> some comments and white space.
> 
> Give it a try and make sure it still works.  (I'll check whether it 
> builds on Solaris, i.e. that my modifications don't break 
> anything there.)
> 
-
This is the tcpdump-workers list.
Visit https://lists.sandelman.ca/ to unsubscribe.

Reply via email to