Merge pull request #188 from noh4hss/setheader

fix SetHeader() implementation
libcpr · Oct 27, 2017 · 9018cdf · 9018cdf
2 parents 749a098 + 942cd99
commit 9018cdf
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/cpr/session.cpp b/cpr/session.cpp
@@ -120,10 +120,16 @@ void Session::Impl::SetHeader(const Header& header) {
             } else {
                 header_string += ": " + item->second;
             }
-            chunk = curl_slist_append(chunk, header_string.data());
-            curl_easy_setopt(curl, CURLOPT_HTTPHEADER, chunk);
-            curl_->chunk = chunk;
+
+            auto temp = curl_slist_append(chunk, header_string.data());
+            if (temp) {
+                chunk = temp;
+            }
         }
+        curl_easy_setopt(curl, CURLOPT_HTTPHEADER, chunk);
+
+        curl_slist_free_all(curl_->chunk);
+        curl_->chunk = chunk;
     }
 }