hsts: use one malloc instead of two per entry

[bagder]

No description provided.

[bagder]

augment review

[augmentcode]

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

[Copilot]

Pull request overview

This PR optimizes memory allocation for HSTS entries by using a flexible array member pattern instead of separate allocations for the structure and hostname string. This reduces allocation overhead from two malloc calls to one per entry.

• Converted stsentry structure from using a separate const char *host pointer to an inline flexible array member char host[1]
• Simplified memory management by consolidating allocation and deallocation into single operations
• Removed unnecessary const-casting since the host field is now mutable by design

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
lib/hsts.h	Changed stsentry structure to use flexible array member char host[1] instead of const char *host pointer
lib/hsts.c	Updated allocation to single calloc with extra space for hostname, simplified deallocation to macro, removed CURL_UNCONST cast

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.