[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lwip-devel] [patch #9307] Replace mem_malloc+memset with mem_calloc
|
From: |
Simon Goldschmidt |
|
Subject: |
[lwip-devel] [patch #9307] Replace mem_malloc+memset with mem_calloc |
|
Date: |
Wed, 5 Apr 2017 15:31:48 -0400 (EDT) |
|
User-agent: |
Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.87 Safari/537.36 |
Follow-up Comment #1, patch #9307 (project lwip):
> Aside from reducing code
That seems overrated: calloc has more parameters than malloc, so your patch
might gain some 32 bytes of code or so...
Nevertheless, I don't really care :-) Just go ahead and push it! There are
probably other places where zeroing the memory wouldn't hurt (performance).
We haven't thought about security risks (by not zeroing memory) in the past
though, since lwIP (by definition) runs in a single address space where all
pool or heap memory is readable by all threads...
_______________________________________________________
Reply to this item at:
<http://savannah.nongnu.org/patch/?9307>
_______________________________________________
Message sent via/by Savannah
http://savannah.nongnu.org/