>
> It seems to me that 'words' is a malloc'ed array of malloc'ed strings,
> so 'free (words)' still leaks storage.

Yeaaaahh... maybe the attached is better?  Or is this use
of goto a little opaque?

By the way, are we maintaining our own readline fork?
Should I check these in locally?