urldata: clean up the use of the protocol specific structs

1 - always allocate the struct in protocol->setup_connection. Some
protocol handlers had to get this function added.

2 - always free at the end of a request. This is also an attempt to keep
less memory in the handle after it is completed.

1 files changed, 4 insertions, 3 deletions

diff --git a/lib/urldata.h b/lib/urldata.h
index d597c6744..7c150a55c 100644
--- a/lib/urldata.h
+++ b/lib/urldata.h
@@ -1277,9 +1277,10 @@ struct UrlState {
   /* Protocol specific data.
    *
    *************************************************************************
-   * Note that this data will be REMOVED after each request, so anything that
-   * should be kept/stored on a per-connection basis and thus live for the
-   * next request on the same connection MUST be put in the connectdata struct!
+   * Note that this data will be freed after each request is DONE, so anything
+   * that should be kept/stored on a per-connection basis and thus live for
+   * the next request on the same connection MUST be put in the connectdata
+   * struct!
    *************************************************************************/
   union {
     struct HTTP *http;