Replace OopStorage dup_name with os::strdup

The OopStorage constructor makes a copy of the storage name, using a nearby defined dup_name function. That function should be file-scoped, but is missing the static qualifier. But really, it shouldn't exist at all, and os::strdup should be used instead.