[PATCH] libnetworking/rtems_dhcp.c: Fix improper hostname handling in DHCP request

Sebastian Huber sebastian.huber at embedded-brains.de
Mon Jan 4 12:21:59 UTC 2016



On 04/01/16 12:55, Daniel Gutson wrote:
>
> >
> >> +
> >> +      if ( (hostnamelen > 0) && (hostnamelen < 256) )
>
> Avoid magic numbers. Should this be MAXHOSTNAMELEN, and in any case ot 
> has been already limited before in the call to gethosname turning the 
> check unnecessary? (Don't believe me, please verify).
>

I think this should be UINT8_MAX, since it is later stored as an uint8_t.

-- 
Sebastian Huber, embedded brains GmbH

Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone   : +49 89 189 47 41-16
Fax     : +49 89 189 47 41-09
E-Mail  : sebastian.huber at embedded-brains.de
PGP     : Public key available on request.

Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.




More information about the devel mailing list