1 files changed, 85 insertions, 70 deletions

diff --git a/src/plugins/fs/download.c b/src/plugins/fs/download.c
index eab6ed20..1fd31bfb 100644
--- a/src/plugins/fs/download.c
+++ b/src/plugins/fs/download.c
@@ -388,7 +388,7 @@ void fs_download_completed(DownloadList * downloadContext) {
  * possibly refresh directory listing.
  */
 void fs_download_aborted(DownloadList * downloadContext) {
-  /* FIXME: update summary? / search list status entry (once added) */
+  /* FIXME: update summary? / search list status entry! */
   downloadContext->has_terminated = YES;
   refreshDirectoryViewFromDisk(downloadContext);
 }
@@ -479,6 +479,31 @@ check_pending(const char * filename,
   return OK;
 }
 
+typedef struct {
+  char * uri_name;
+  struct ECRS_URI * idc_uri;
+  struct ECRS_MetaData * idc_meta;
+  char * idc_final_download_destination;
+  SearchList * searchContext;
+  DownloadList * parentContext;
+  unsigned int anonymity;
+  int recursive;
+} SDC;
+
+static void * init_download_helper(void * cls) {
+  SDC * sdc = cls;
+
+  FSUI_startDownload(ctx,
+                     sdc->anonymity,
+                     sdc->recursive,
+                     sdc->idc_uri,
+                     sdc->idc_meta,
+                     sdc->idc_final_download_destination,
+                     sdc->searchContext->fsui_list,
+                     (sdc->parentContext != NULL) ? sdc->parentContext->fsui_list : NULL);
+  return NULL;
+}
+
 /**
  * The user clicked the download button.
  * Start the download of the selected entry.
@@ -488,7 +513,7 @@ initiateDownload(GtkTreeModel * model,
                  GtkTreePath * path,
                  GtkTreeIter * iter,
                  gpointer unused) {
-  char * uri_name;
+  SDC sdc;
   char * final_download_dir;
   GtkTreeIter iiter;
   char * tmp;
@@ -497,43 +522,38 @@ initiateDownload(GtkTreeModel * model,
   GtkTreePath *dirTreePath;
   char *dirPath;
   unsigned int dirPathLen;
-  struct ECRS_URI * idc_uri;
-  struct ECRS_MetaData * idc_meta;
   char * idc_name;
   char * idc_mime;
-  char * idc_final_download_destination;
-  SearchList * searchContext;
-  DownloadList * parentContext;
 
-  idc_uri = NULL;
-  idc_meta = NULL;
+  sdc.idc_uri = NULL;
+  sdc.idc_meta = NULL;
   idc_name = NULL;
   idc_mime = NULL;
-  searchContext = NULL;
-  parentContext = NULL;
+  sdc.searchContext = NULL;
+  sdc.parentContext = NULL;
   gtk_tree_model_get(model,
                      iter,
                      SEARCH_NAME, &idc_name,
-                     SEARCH_URI, &idc_uri,
-                     SEARCH_META, &idc_meta,
+                     SEARCH_URI, &sdc.idc_uri,
+                     SEARCH_META, &sdc.idc_meta,
                      SEARCH_MIME, &idc_mime,
-                     SEARCH_INTERNAL, &searchContext,
-                     SEARCH_INTERNAL_PARENT, &parentContext,
+                     SEARCH_INTERNAL, &sdc.searchContext,
+                     SEARCH_INTERNAL_PARENT, &sdc.parentContext,
                      -1);
-  if ( (idc_uri == NULL) ||
-       (! ECRS_isFileUri(idc_uri)) ) {
+  if ( (sdc.idc_uri == NULL) ||
+       (! ECRS_isFileUri(sdc.idc_uri)) ) {
     GE_BREAK(ectx, 0);
     FREENONNULL(idc_name);
     FREENONNULL(idc_mime);
     return;
   }
-  uri_name = ECRS_uriToString(idc_uri);
-  if ( (uri_name == NULL) ||
-       (strlen(uri_name) <
+  sdc.uri_name = ECRS_uriToString(sdc.idc_uri);
+  if ( (sdc.uri_name == NULL) ||
+       (strlen(sdc.uri_name) <
         strlen(ECRS_URI_PREFIX) +
         strlen(ECRS_FILE_INFIX)) ) {
     GE_BREAK(ectx, 0);
-    FREENONNULL(uri_name);
+    FREENONNULL(sdc.uri_name);
     FREENONNULL(idc_name);
     FREENONNULL(idc_mime);
     return;
@@ -542,12 +562,12 @@ initiateDownload(GtkTreeModel * model,
 #ifdef WINDOWS
     char * filehash;
     
-    filehash = STRDUP(uri_name);
+    filehash = STRDUP(sdc.uri_name);
     filehash[16] = 0;
     idc_name = STRDUP(filehash);
     FREENONNULL(filehash);
 #else
-    idc_name = STRDUP(uri_name);
+    idc_name = STRDUP(sdc.uri_name);
 #endif
   }
 
@@ -620,41 +640,35 @@ initiateDownload(GtkTreeModel * model,
   gtk_tree_path_free(dirTreePath);
 
   /* construct completed/directory/real-filename */
-  idc_final_download_destination = MALLOC(strlen(final_download_dir) + 2 +
-                                          strlen(idc_name) + strlen(GNUNET_DIRECTORY_EXT) +
-                                          strlen(dirPath));
-  strcpy(idc_final_download_destination, 
+  sdc.idc_final_download_destination = MALLOC(strlen(final_download_dir) + 2 +
+                                              strlen(idc_name) + strlen(GNUNET_DIRECTORY_EXT) +
+                                              strlen(dirPath));
+  strcpy(sdc.idc_final_download_destination, 
          final_download_dir);
-  if (idc_final_download_destination[strlen(idc_final_download_destination)-1] != DIR_SEPARATOR)
-    strcat(idc_final_download_destination,
+  if (sdc.idc_final_download_destination[strlen(sdc.idc_final_download_destination)-1] != DIR_SEPARATOR)
+    strcat(sdc.idc_final_download_destination,
            DIR_SEPARATOR_STR);
-  strcat(idc_final_download_destination, 
+  strcat(sdc.idc_final_download_destination, 
          dirPath);
-  strcat(idc_final_download_destination, 
+  strcat(sdc.idc_final_download_destination, 
          idc_name);
-  /* FIXME: check that there is no pending download for idc_name! */
+  sdc.anonymity = getSpinButtonValue(sdc.searchContext->searchXML,
+                                     "downloadAnonymitySpinButton");
+  sdc.recursive = getToggleButtonValue(sdc.searchContext->searchXML,
+                                       "downloadRecursiveCheckButton");
   if (OK == check_pending(idc_name,
                           NULL)) {
     addLogEntry(_("Downloading `%s'"),
                 idc_name);
-    /* FIXME! DEADLOCK! */
-    FSUI_startDownload(ctx,
-                       getSpinButtonValue(searchContext->searchXML,
-                                          "downloadAnonymitySpinButton"),
-                       getToggleButtonValue(searchContext->searchXML,
-                                            "downloadRecursiveCheckButton"),
-                       idc_uri,
-                       idc_meta,
-                       idc_final_download_destination,
-                       searchContext->fsui_list,
-                       (parentContext != NULL) ? parentContext->fsui_list : NULL);
+    run_with_save_calls(&init_download_helper,
+                        &sdc);
   } else {
     addLogEntry(_("ERROR: already downloading `%s'"),
                 idc_name);
   }
-  FREE(uri_name);
+  FREE(sdc.uri_name);
   FREE(dirPath);
-  FREE(idc_final_download_destination);
+  FREE(sdc.idc_final_download_destination);
   FREENONNULL(final_download_dir);
   FREENONNULL(idc_name);
   FREENONNULL(idc_mime);
@@ -683,36 +697,35 @@ void on_downloadButton_clicked_fs(GtkWidget * treeview,
  * TODO:
  * - support for recursive downloads
  * - support for user-specified filename
+ * - enable button only if valid URI is entered
  */
 void on_statusDownloadURIEntry_editing_done_fs(GtkWidget * entry,
                                                GtkWidget * downloadButton) {
-  struct ECRS_URI * idc_uri;
-  struct ECRS_MetaData * idc_meta;
-  char * idc_final_download_destination;
   const char * uris;
   char * urid;
   char * final_download_dir;
   const char * dname;
+  SDC sdc;
 
   uris = gtk_entry_get_text(GTK_ENTRY(entry));
   urid = STRDUP(uris);
   gtk_entry_set_text(GTK_ENTRY(entry),
                      ECRS_URI_PREFIX);
-  idc_uri = ECRS_stringToUri(ectx, urid);
-  if (idc_uri == NULL) {
+  sdc.idc_uri = ECRS_stringToUri(ectx, urid);
+  if (sdc.idc_uri == NULL) {
     addLogEntry(_("Invalid URI `%s'"), urid);
     FREE(urid);
     return;
   }
-  if (ECRS_isKeywordUri(idc_uri)) {
+  if (ECRS_isKeywordUri(sdc.idc_uri)) {
     addLogEntry(_("Please use the search function for keyword (KSK) URIs!"));
     FREE(urid);
-    ECRS_freeUri(idc_uri);
+    ECRS_freeUri(sdc.idc_uri);
     return;
-  } else if (ECRS_isLocationUri(idc_uri)) {
+  } else if (ECRS_isLocationUri(sdc.idc_uri)) {
     addLogEntry(_("Location URIs are not yet supported"));
     FREE(urid);
-    ECRS_freeUri(idc_uri);
+    ECRS_freeUri(sdc.idc_uri);
     return;
   }
   GC_get_configuration_value_filename(cfg,
@@ -722,26 +735,28 @@ void on_statusDownloadURIEntry_editing_done_fs(GtkWidget * entry,
                                       &final_download_dir);
   disk_directory_create(ectx, final_download_dir);
   dname = &uris[strlen(ECRS_URI_PREFIX) + strlen(ECRS_FILE_INFIX)];
-  idc_final_download_destination = MALLOC(strlen(final_download_dir) + strlen(dname) + 2);
-  strcpy(idc_final_download_destination, final_download_dir);
+  sdc.idc_final_download_destination = MALLOC(strlen(final_download_dir) + strlen(dname) + 2);
+  strcpy(sdc.idc_final_download_destination, 
+         final_download_dir);
   FREE(final_download_dir);
-  if (idc_final_download_destination[strlen(idc_final_download_destination)] != DIR_SEPARATOR)
-    strcat(idc_final_download_destination, DIR_SEPARATOR_STR);
-  strcat(idc_final_download_destination, dname);
+  if (sdc.idc_final_download_destination[strlen(sdc.idc_final_download_destination)] != DIR_SEPARATOR)
+    strcat(sdc.idc_final_download_destination, 
+           DIR_SEPARATOR_STR);
+  strcat(sdc.idc_final_download_destination,
+         dname);
 
   addLogEntry(_("Downloading `%s'"), 
               uris);
-  idc_meta = ECRS_createMetaData();
-  FSUI_startDownload(ctx,
-                     getSpinButtonValue(getMainXML(),
-                                        "fsstatusAnonymitySpin"),
-                     NO, /* FIXME: isRecursive */
-                     idc_uri,
-                     idc_meta,
-                     idc_final_download_destination,
-                     NULL,
-                     NULL);
-  ECRS_freeMetaData(idc_meta);
+  sdc.idc_meta = ECRS_createMetaData();
+  sdc.anonymity = getSpinButtonValue(getMainXML(),
+                                     "fsstatusAnonymitySpin");
+  sdc.recursive = NO; 
+  sdc.searchContext = NULL;
+  sdc.parentContext = NULL;
+  run_with_save_calls(&init_download_helper,
+                      &sdc);
+  ECRS_freeMetaData(sdc.idc_meta);
+  FREE(sdc.idc_final_download_destination);
   FREE(urid);
 }