okl
Joined 8 years ago
Joined 8 years ago
strlcpy, often touted as a replacement, does not elicit the padding behavior but has another flaw: It is supposed to return the length of the string it tried to create, for example, so the user can call realloc without calling strlen again.[3] However, this final "strlen-tail" in strlcpy isn't bounded by the size parameter which describes dest, not src.
While strscpy is a marked improvement, there is still something to be careful about: It can read past the end of the src-buffer, when sizeof src < sizeof dest and src is not nul-terminated.[4] (Set the count argument to something like min(sizeof dest, sizeof src) to avoid that).
--
[1] - https://softwareengineering.stackexchange.com/a/438090
[2] - https://www.open-std.org/jtc1/sc22/wg14/www/docs/n3220.pdf, 7.26.2.5 p. 3
[3] - https://manpages.debian.org/jessie/libbsd-dev/strlcpy.3.en.h...
[4] - https://manpages.debian.org/testing/linux-manual-4.8/strscpy...
It would be nice to focus on solving more existential problems of which there are enough.
I would recommend you to try using FIM/line completion rather than the agent approach. Using FIM it makes me (feel) productive while still being in control. Oftentimes when writing code for production, documentation etc. the agent approach feels like a wash to me - sometimes it saves time, sometimes it wastes time. With FIM the gain/quality is more consistent.