inconsistent strdup and delete[]

[haase]

When running under ASAN I get a complaint that strings allocated by strdup() are being freed by delete[] which is apparently a no-no. I don't know enough about C++ object reclamation to fix it myself. Most of the recommendations on the interwebs seem to be to use C++ string types rather than character arrays, but I imagine that could lead to a performance hit.