Don't free string allocated from stack

Taking another look at this code to figure out an
easy way to demonstrate simple object-oriented
approaches useful in c. Currently, the code is too
simple to support the 'painting' object I'm using
an example.

Two choices:

1. Increase the complexity of the demo, or
2. simplify the object attributes used for the example.