[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