bug-inetutils
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: inetutils-1.3.2 tftp file limit 16MB


From: Alain Magloire
Subject: Re: inetutils-1.3.2 tftp file limit 16MB
Date: Tue, 25 Dec 2001 13:44:12 -0500 (EST)

> Bonjour
> 
> > At 09:53 -0500 12/18/01, Alain Magloire wrote:
> > >
> > >
> > >Unless I missed the obvious, changing this will make it impossible to work 
> > >with other
> > >since you are changing the struct size, tftp.h is an expose file and 
> > >provided for
> > >convenience if the platform does not have it in /usr/include/arpa/tftp.h
> > 
> >   I disagree: changing from short to u_short still occupies 2 bytes, and 
> > only affects the interpretation of the sign, for example whether 0x8000 is 
> > smaller or larger than 0x7fff.  Of course for best advantage both server 
> > and client should probably change (I've only documented the client doing an 
> > integer overflow in processing the block number during ack, but I imagine 
> > server will have the same difficulty), but the client should see no 
> > difference in the size or contents of the tftphdr data structure.  I see 
> > that the Solaris include file /usr/include/arpa/tftp.h was not used for the 
> > GNU compilation, according to file access time (don't know why).
> > 

Sigh ...
It turns out I did this already 2 years ago.
Please update your version.  There is a snapshot in 
ftp://alpha.gnu.org/gnu/inetutils
Or build one from the cvs repository(use the autogen.sh script to configure)

# cvs log headers/arpa/tftp.h
...
----------------------------
revision 1.2
date: 1999/05/19 17:54:13;  author: alainm;  state: Exp;  lines: +1 -1
add afew names in THANKS and configure.in check for ptsname().
#undef MACH and use MACHINE instead.
use unsigned short now.
----------------------------
revision 1.1
date: 1996/07/22 17:18:14;  author: miles;  state: Exp;
Initial revision
=============================================================================





reply via email to

[Prev in Thread] Current Thread [Next in Thread]