remove an extra layer of buffer copying in getnameinfo reverse dns
authorRich Felker <dalias@aerifal.cx>
Fri, 5 Sep 2014 06:50:38 +0000 (02:50 -0400)
committerRich Felker <dalias@aerifal.cx>
Fri, 5 Sep 2014 06:50:38 +0000 (02:50 -0400)
commit80e6485455a9c75c029d0e976d2dd9c8e441dd72
tree659d4632ef956f2bebed3509e97715f255afb8d5
parent6e2bb7acf42589fb7130b039d0623e2ca42503dd
remove an extra layer of buffer copying in getnameinfo reverse dns

the outer getnameinfo function already has a properly-sized temporary
buffer for storing the reverse dns (ptr) result. there is no reason
for the callback to use a secondary buffer and copy it on success, and
doing so potentially expanded the impact of the dn_expand bug that was
fixed in commit 49d2c8c6bcf8c926e52c7f510033b6adc31355f5.

this change reduces the code size by a small amount, and also reduces
the run-time stack space requirements by about 256 bytes.
src/network/getnameinfo.c