fix memory leak in string_to_argv

`s` gets duplicated but not free'd