efl_net_dialer_http: cancel curl multi timer when it's gone.
authorGustavo Sverzut Barbieri <barbieri@profusion.mobi>
Mon, 29 Aug 2016 17:49:03 +0000 (14:49 -0300)
committerGustavo Sverzut Barbieri <barbieri@profusion.mobi>
Tue, 30 Aug 2016 03:35:35 +0000 (00:35 -0300)
If we delete the curl multi handle, then we should stop any timer that
was scheduled, otherwise it will use a dead or null pointer.

also add some debug to help track down when the multi handle is
deleted.

src/lib/ecore_con/efl_net_dialer_http.c

index d3a4b55..f80bd4e 100644 (file)
@@ -522,6 +522,9 @@ static void
 _efl_net_dialer_http_curlm_remove(Efl_Net_Dialer_Http_Curlm *cm, Eo *o, CURL *handle)
 {
    CURLMcode r = curl_multi_remove_handle(cm->multi, handle);
+
+   DBG("removed handle cm=%p multi=%p easy=%p: %s",
+       cm, cm->multi, handle, curl_multi_strerror(r));
    if (r != CURLM_OK)
      {
         ERR("could not unregister curl multi handle %p: %s",
@@ -531,8 +534,15 @@ _efl_net_dialer_http_curlm_remove(Efl_Net_Dialer_Http_Curlm *cm, Eo *o, CURL *ha
    cm->users = eina_list_remove(cm->users, o);
    if (!cm->users)
      {
+        DBG("cleaned up cm=%p multi=%p", cm, cm->multi);
         curl_multi_cleanup(cm->multi);
         cm->multi = NULL;
+
+        if (cm->timer)
+          {
+             efl_del(cm->timer);
+             cm->timer = NULL;
+          }
      }
 }