On Mon 2013-08-26 10:40:50, Sebastian Capella wrote:
Apologies for my previous top post reply...
Quoting Pavel Machek (2013-08-25 08:38:11)
Is the allocation actually neccessary? At the very least this should test for NULL...
Thanks Pavel! I'll add the check for NULL.
name_to_dev_t expects a non-const name, but the buffer passed in is const. I also am removing the '\n' if found at the end of the string which would violate the const.
Fix name_to_dev_t, then. No need to do memory allocation just to work around const.
[You can also take a look why it is const in the first place. I don't think it needs to be.]
Pavel