1
0
mirror of https://github.com/moparisthebest/wget synced 2024-07-03 16:38:41 -04:00

Use Gnulib's xalloc module in convert.c (and include according header).

This commit is contained in:
Steven Schubiger 2008-04-17 00:35:34 +02:00
parent 3e34be1c4b
commit 8aefedc7da
2 changed files with 20 additions and 14 deletions

View File

@ -28,6 +28,8 @@ Corresponding Source for a non-source form of such a combination
shall include the source code for the parts of OpenSSL used as well shall include the source code for the parts of OpenSSL used as well
as that of the covered work. */ as that of the covered work. */
#define USE_GNULIB_ALLOC
#include "wget.h" #include "wget.h"
#include <stdio.h> #include <stdio.h>
@ -87,7 +89,7 @@ convert_all_links (void)
cnt = hash_table_count (downloaded_html_set); cnt = hash_table_count (downloaded_html_set);
if (cnt == 0) if (cnt == 0)
goto cleanup; goto cleanup;
file_array = alloca_array (char *, cnt); file_array = xcalloc (cnt, sizeof (char *));
string_set_to_array (downloaded_html_set, file_array); string_set_to_array (downloaded_html_set, file_array);
for (i = 0; i < cnt; i++) for (i = 0; i < cnt; i++)
@ -285,8 +287,8 @@ convert_links (const char *file, struct urlpos *links)
DEBUGP (("TO_RELATIVE: %s to %s at position %d in %s.\n", DEBUGP (("TO_RELATIVE: %s to %s at position %d in %s.\n",
link->url->url, newname, link->pos, file)); link->url->url, newname, link->pos, file));
xfree (newname); free (newname);
xfree (quoted_newname); free (quoted_newname);
++to_file_count; ++to_file_count;
break; break;
} }
@ -304,7 +306,7 @@ convert_links (const char *file, struct urlpos *links)
DEBUGP (("TO_COMPLETE: <something> to %s at position %d in %s.\n", DEBUGP (("TO_COMPLETE: <something> to %s at position %d in %s.\n",
newlink, link->pos, file)); newlink, link->pos, file));
xfree (quoted_newlink); free (quoted_newlink);
++to_url_count; ++to_url_count;
break; break;
} }
@ -688,8 +690,8 @@ dissociate_urls_from_file_mapper (void *key, void *value, void *arg)
if (0 == strcmp (mapping_file, file)) if (0 == strcmp (mapping_file, file))
{ {
hash_table_remove (dl_url_file_map, mapping_url); hash_table_remove (dl_url_file_map, mapping_url);
xfree (mapping_url); free (mapping_url);
xfree (mapping_file); free (mapping_file);
} }
/* Continue mapping. */ /* Continue mapping. */
@ -739,8 +741,8 @@ register_download (const char *url, const char *file)
goto url_only; goto url_only;
hash_table_remove (dl_file_url_map, file); hash_table_remove (dl_file_url_map, file);
xfree (old_file); free (old_file);
xfree (old_url); free (old_url);
/* Remove all the URLs that point to this file. Yes, there can /* Remove all the URLs that point to this file. Yes, there can
be more than one such URL, because we store redirections as be more than one such URL, because we store redirections as
@ -780,8 +782,8 @@ register_download (const char *url, const char *file)
if (hash_table_get_pair (dl_url_file_map, url, &old_url, &old_file)) if (hash_table_get_pair (dl_url_file_map, url, &old_url, &old_file))
{ {
hash_table_remove (dl_url_file_map, url); hash_table_remove (dl_url_file_map, url);
xfree (old_url); free (old_url);
xfree (old_file); free (old_file);
} }
hash_table_put (dl_url_file_map, xstrdup (url), xstrdup (file)); hash_table_put (dl_url_file_map, xstrdup (url), xstrdup (file));
@ -817,8 +819,8 @@ register_delete_file (const char *file)
return; return;
hash_table_remove (dl_file_url_map, file); hash_table_remove (dl_file_url_map, file);
xfree (old_file); free (old_file);
xfree (old_url); free (old_url);
dissociate_urls_from_file (file); dissociate_urls_from_file (file);
} }
@ -948,7 +950,7 @@ downloaded_files_free (void)
for (hash_table_iterate (downloaded_files_hash, &iter); for (hash_table_iterate (downloaded_files_hash, &iter);
hash_table_iter_next (&iter); hash_table_iter_next (&iter);
) )
xfree (iter.key); free (iter.key);
hash_table_destroy (downloaded_files_hash); hash_table_destroy (downloaded_files_hash);
downloaded_files_hash = NULL; downloaded_files_hash = NULL;
} }

View File

@ -208,7 +208,11 @@ typedef double SUM_SIZE_INT;
#include "options.h" #include "options.h"
/* Everything uses this, so include them here directly. */ /* Everything uses this, so include them here directly. */
#ifdef USE_GNULIB_ALLOC
# include "xalloc.h"
#else
# include "xmalloc.h" # include "xmalloc.h"
#endif
/* Likewise for logging functions. */ /* Likewise for logging functions. */
#include "log.h" #include "log.h"