> > - unsigned char ha[MAX_ADDR_LEN]; /* Hardware address */> > + unsigned char ha[7]; /* Hardware address */> > Why not locally defining a constant? I don't like all these magic > numbers.The MAX_ADDR_LEN for headers, for network cards an for ARP must all agreeits not a local its a global. Dont make it a number