bryancall commented on issue #8648: URL: https://github.com/apache/trafficserver/issues/8648#issuecomment-4827449586
This cleanup has been addressed. The HostDB restructure in https://github.com/apache/trafficserver/pull/8953 (first released in 10.0.0) replaced the old HostDBHash::set_host(const char *name, int len) with set_host(TextView name) and made the host_name member a length-aware swoc::TextView. The hash code now uses host_name.data() and host_name.size() instead of assuming null termination, and the hash carries its own backing store (hash_host_name_store[MAXDNAME + 1]), so it no longer relies on a borrowed pointer staying valid. Both the null-termination ambiguity and the string-lifetime concern raised here are resolved, so I am closing this. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
