malloc(strlen) is a bad pattern
In several files, there is something like
This is a bad pattern, since one would expect that the malloc() is use to allocate space for the whole string, including the terminal '\0'. At least, a comment should be added to make sure that this is done on purpose, and not a typo.
(from redmine: created on 2014-04-21)