Re: [lwip-devel] [bug #36492] Static Analysis on code 1.4.0

[Simon Goldschmidt]

Mason <address@hidden> wrote:
> It is the responsibility of the driver to ensure
> ASSERT(hwaddr_len <= NETIF_MAX_HWADDR_LEN);
> 
> and, in fact, netif/etharp.c ASSERTS hwaddr_len == ETHARP_HWADDR_LEN.
> 
> The problem with static analysis tools is that they produce
> too many false positives.

Personally, I'd regard it as good coding style to check the range here, no 
matter if etharp.c asserts or not.

Simon
-- 
NEU: FreePhone 3-fach-Flat mit kostenlosem Smartphone!                          
        
Jetzt informieren: http://mobile.1und1.de/?ac=OM.PW.PW003K20328T7073a