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 -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html _______________________________________________ linaro-dev mailing list linaro-dev@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-dev