> It's suposed to create a linked list of char arrays.
So are you making an actual copy of the string (good), or just storing the pointer (bad)?

Things inside the class should never point to things outside the class, since you have no control over when they might get deallocated or go out of scope.