]> sjero.net Git - wget/blobdiff - src/ChangeLog
Fix compiler warnings
[wget] / src / ChangeLog
index bbc67352a6c0119858e98ab02c1155bd5587a440..f07ddc5f2000e6b724b754d39e26a8974403082b 100644 (file)
@@ -1,3 +1,505 @@
+2014-05-03  Tim Ruehsen  <tim.ruehsen@gmx.de>
+
+       * ftp-ls.c (ftp_parse_vms_ls): Explicitly typecast strlen's output
+       * ftp.c (getftp): Prevent declaration of shadow variable
+       * html-url.c (tag_handle_base, tag_handle_form, tag_handle_link,
+       tag_handle_meta): Compiler directive to ignore unused variable
+       * html-url.h (cleanup_html_url): Add function declaration
+       * http-ntlm.c (ntlm_input, mkhash): Use ssize_t to declare variables that store
+       sizes.
+       (mkhash): Explicitly typecast output of c_toupper
+       (short_pair): Add typecasts to prevent compiler warnings
+       (ntlm-output): Fix datatypes of various variables
+       * http.c (gethttp): Prevent declaration of shadow variable err
+       (gethttp): remove unreachable code
+       (test_parse_content_disposition): Fix variable declarations and use
+       countof() macro
+       * init.c (run_command): Prevent declaration of ghost variable
+       (cmd_string, cmd_string_uppercase, cmd_file, cnd_vector,
+       cmd_directory_vector, cmd_spec_dirstruct, cmd_spec_header,
+       cmd_spec_warc_header, cmd_spec_htmlify, cmd_spec_mirror,
+       cmd_spec_prefer_family, cmd_spec_progress, cmd_spec_recursive,
+       cmd_spec_regex_type, cmd_spec_restrict_file_names, cmd_spec_report_speed,
+       cmd_spec_timeout, cmd_spec_useragent, cmd_spec_verbose): Add compiler
+       directive to ignore unused variable
+       (cleanup_html_url, spider_cleanup): Remove declarations
+       (test_commands_sorted): Cleanup code
+       (test_cmd_spec_restrict_file_names): Use correct data types
+       * iri.c (remote_to_utf8): Use more verbose variable name
+       * main.c (init_switches): Prevent declaration of ghost variable
+       * netrc.c (netrc_cleanup): Create cleanup function on common naming scheme
+       * netrc.h (netrc_cleanup): Declare function
+       * openssl.c (openssl_write, openssl_errstr): Compiler directive to ignore
+       unused parameter
+       (openssl_errstr): Explcicitly typecasr output of ASN1_STRING_length
+       * options.h (struct options): Declare includes and excludes as const char **
+       * progress.c (progress_interactive_p, progress_handle_sigwinch): Compiler
+       directive to ignore unused parameter
+       * res.c (test_is_robots_txt_url): Fix datatypes
+       * retr.c (line_terminator): Add compiler directive for ignoring unused
+       paramter. Remove unused variable.
+       * spider.h (spider_cleanup): Declare function
+       * test,c (main): declare unused paramter
+       * test.h (test_*): Declare functions
+       * url.c (url_string): Explicit typecast of password strings
+       (run_test): Declare *test, struct tests and *expected_result as const
+       (test_path_simplify, test_append_uri_pathl, test_are_urls_equal): Fix datatypes
+       * utls.c (fork_to_background): Be more verbose when errors occur
+       (dir_matches_p, test_dir_matches_p): Declare char * as const as required
+       (base64_encode, base64_decode, get_max_length): Set correct return type
+       (match_pcre_regex, match_posix_regex): Use correct datatypes and typecasts
+       (test_subdir_p): static const struct
+       * utils.h (base64_encode, base64_decode, get_max_length): Fix return type
+       * warc.c (ward_write_cdx_record): unused parameter
+       (struct hash_table *warc_cdx_dedup_table): Declare as static
+
+2014-05-01  Benjamin Goose  <gans+wget@tngtech.com>
+
+       * http.c: Fix small memory leak
+
+2014-05-01  Darshit Shah  <darnir@gmail.com> (tiny change)
+
+       * progress.c (dot_finish): Do not print extra newlines when not in verbose
+       mode. (Purely aesthetic change)
+       (get_eta): Add extra space when eta is printed.
+       (create_image): Remove erroneous space from being added to progress bar when
+       filename > MAX_FILENAME_LEN
+       (create_image): Remove extra space before printed download speeds
+       Make the filename a fixed width column.
+
+2014-04-19  Darshit Shah  <darnir@gmail.com>
+
+       * log.h (log_options): Add new logging options, LOG_PROGRESS. All progress
+       bar related output should use LOG_PROGRESS.
+       * log.c (CHECK_VERBOSE): Implement LOG_PROGRESS output
+       * progress.c (dot_create, print_row_stats, dot_update, dot_finish,
+       bar_finish, display_image): Output progress information through LOG_PROGRESS
+       (progress_implementation, dot_create, bar_create, progress_create): The
+       progress bar create functions accept an extra paramter for the filename of the
+       local file
+       (bar_progress): Add new variable to store filename of currently downloading
+       file
+       (bar_finish): Aesthetic change. Print two newlines if in verbose mode, else
+       only one.
+       (MACRO): Define new macro, MIN
+       (create_image): Implement displaying filename in progress bar output
+       Filename tick implementation copied from Giuseppe's patch on parallel-wget
+       * progress.h (progress_create): Accept another parameter for filename
+       * http.c (gethttp): Remove unnecessary conditional
+       (read_response_body): Send local filename to fd_read_body so that it can be
+       printed on the progress bar
+       * main.c (option_data): Add new switch, --show-progress
+       (main): If in verbose output, show progress bar by default
+       (main): Set progress implemetation when displaying progress bar
+       (no_prefix): Increase buffer size to 2048 to prevent overflows
+       * init.c (commands): Add new command, showprogress
+       (defaults): By default initialize show_progress to false
+       * options.h (options): Add new option, show_progress
+       * retr.c (fd_read_body): Accept new parameter for filename of currently
+       downlaoding file
+       (fd_read_body): Create and update progress bar when opt.show_progress is set
+       (fd_read_body): Display progress information in Windows console titlebars
+       * retr.h (fd_read_body): Update declaration
+       * ftp.c (getftp): Send filename of the local file so that it can be printed
+       with the progress bar.
+
+2013-11-02  Giuseppe Scrivano  <gscrivan@redhat.com>
+
+       * progress.c (struct progress_implementation): New method draw.
+       (bar_update): Split into...
+       (bar_draw): ...this new function.
+       (dot_update): Split into...
+       (dot_draw): New function.
+       (progress_update): Also invoke draw on `current_impl'.
+
+2014-04-22  Tim Ruehsen  <tim.ruehsen@gmx.de>
+
+       * http.c (gethttp): Fix 204 response handling
+
+2014-03-26  Darshit Shah  <darnir@gmail.com>
+
+       * ftp.c (getftp): Rearrange parameters to fix compiler warning
+       * utils.c (get_dir_matches_p): Do not pass a const char** to a function that
+       expects char**
+
+2014-03-25  Daniel Stenberg  <daniel@haxx.se>
+
+       * url.c (shorten_length): Remove unused function.
+
+2014-03-19  Yousong Zhou  <yszhou4tech@gmail.com>
+
+       * init.c, main.c, options.h: Add option --start-pos for specifying
+       start position of a download.
+       * http.c: Utilize opt.start_pos for HTTP download.
+       * ftp.c: Utilize opt.start_pos for FTP retrieval.
+
+2014-03-04  Giuseppe Scrivano  <gscrivan@redhat.com>
+
+       * http.c (modify_param_value, extract_param): Aesthetic change.
+
+2014-02-14  Vladimír Pýcha  <vpycha@gmail.com> (tiny change)
+
+       * http.c (parse_content_disposition, extract_param)
+       (append_value_to_filename, digest_authentication_encode): URL-decode the
+       filename parameter of Content-Disposition HTTP header if it is encoded. This
+       is related to --content-disposition.
+       New parameter of extract_param(), "is_url_encoded".
+       Add argument NULL to the call of extract_param() in
+       digest_authentication_encode().
+       * http.h: Add the new parameter to the declaration of extract_param().
+       * cookies.c (parse_set_cookie, test_cookies): Add argument NULL to the calls
+       of extract_param().
+       * url.c (url_unescape): Remove "static" modifier.
+       * url.h: Add declaration of url_unescape().
+
+2014-02-06  Giuseppe Scrivano  <gscrivan@redhat.com>
+
+       * main.c (print_version): Move copyright year out of the localized
+       string and update it.
+
+2014-01-29  Darshit Shah  <darnir@gmail.com>
+
+       * main.c: Remove pre-processor variable WHEN_DEBUG
+       (option_data[]): Do not fail on --debug even if debug support is not
+       compiled in.
+       (main): Explicitly set opt.debug to false in case debugging support was not
+       compiled.
+       * init.c (commands[]): Support --debug wven when support is not compiled in.
+       * options.h: Same
+
+2014-01-17  Darshit Shah  <darnir@gmail.com>
+
+       * init.c (commands[]): Add --no-config.
+       * options.h: Same.
+       * main.c (option_data[]): Same.
+       (print_help): Same.
+       (main): If --no-config is set, then do not read the wgetrc files.
+
+2014-01-05  Håkon Vågsether <hauk142@gmail.com> (tiny change)
+
+       * http.c (http_loop): Fix checking the URL length when filename is
+       specified.
+
+2013-12-29  Giuseppe Scrivano  <gscrivan@redhat.com>
+
+       * init.c (home_dir): Remove useless 'if'.
+       * warc.c (warc_start_new_file): Likewise.
+       (warc_process_cdx_line): Likewise.
+       (warc_write_response_record): Likewise.
+
+2013-12-26  Tim Ruehsen  <tim.ruehsen@gmx.de>
+
+       * gnutls.c (ssl_connect_wget): Fix connect timeout failure
+
+2013-11-02  Giuseppe Scrivano  <gscrivan@redhat.com>
+
+       * http.c (gethttp): Increase max header value length to 512.
+
+2013-01-30  Pavel Mateja  <pavel@netsafe.cz> (tiny change)
+
+       * http.c (gethttp): Specify "Host" for CONNECT method.
+
+2013-10-30  Giuseppe Scrivano  <gscrivan@redhat.com>
+
+       * http.c (skip_short_body): Remove assert which is always true.
+       Reported by: David Binderman <dcb314@hotmail.com>
+
+2013-10-17  Andrea Urbani  <matfanjol@mail.com>
+
+       * ftp.c (getftp): force "LIST" or "LIST -a"
+         according to the remote system type. If the remote
+         system is not known, it tries, only the first time,
+         "LIST -a", after "LIST" and decides which one to use.
+         For more information look for "__LIST_A_EXPLANATION__"
+       * ftp.h (enum ustype): New ustype enum.
+       * ftp.h (ftp_syst): New enum ustype *unix_type parameter.
+       * ftp.h (ftp_list): Removed enum stype rs parameter, added.
+         bool avoid_list_a, bool avoid_list, bool *list_a_used
+         parameters.
+       * ftp.h (wget_ftp_fstatus): New AVOID_LIST_A, AVOID_LIST,
+         LIST_AFTER_LIST_A_CHECK_DONE values.
+       * ftp-basic.c (ftp_list): it handles the new avoid_list_a,
+         avoid_list and list_a_used parameters.
+       * ftp.h (ftp_syst): it stores information about the "215 UNIX"
+         systems into the new unix_type parameter.
+
+2013-10-26  Bykov Aleksey <gnfalex@rambler.ru>
+
+       * utils.c (match_tail): Fix cookies reject
+
+       * ftp-ls.c (ftp_parse_unix_ls): Fix parsing month name in uppercase
+
+2013-09-09  Tim Ruehsen  <tim.ruehsen@gmx.de>
+
+       * gnutls.c (ssl_connect_wget): changed checking of option "PFS"
+         to be better prepared for some kinds of backports.
+         Reported by: Daniel Kahn Gillmor <dkg@fifthhorseman.net>
+
+2013-10-10  Giuseppe Scrivano  <gscrivan@redhat.com>
+
+       * url.c (url_parse): Try to convert UTF-8 URLs to IDN.
+       * html-url.c (append_url): Parse URLs specifying an IRI structure.
+
+2013-09-13  Tim Ruehsen  <tim.ruehsen@gmx.de>
+
+       * recur.c (download_child_p): fix compile error when
+         configured using --without-ssl.
+
+2013-09-07  Tim Ruehsen  <tim.ruehsen@gmx.de>
+
+       * gnutls.c (ssl_connect_wget): use gnutls_check_version()
+         to check if option "PFS" is available
+         Reported by: Daniel Kahn Gillmor <dkg@fifthhorseman.net>
+
+2013-09-03  Tim Ruehsen  <tim.ruehsen@gmx.de>
+
+       * main.c: Add new value 'PFS' to --secure-protocol to
+         enforce the so-called Perfect Forward Security.
+       * init.c (cmd_spec_secure_protocol): added secure_protocol_pfs
+       * openssl.c, gnutls.c, options.h: likewise
+
+2013-08-22  Tim Ruehsen  <tim.ruehsen@gmx.de>
+
+       * main.c: Add new option --https-only.
+       * options.h: Likewise.
+       * recur.c (download_child_p): add check for HTTPS.
+
+2013-08-09  Tim Ruehsen  <tim.ruehsen@gmx.de>
+
+       * gnutls.c (ssl_init): Prevent CA files from being loaded twice
+         if possible.
+       * gnutls.c (ssl_check_certificate): Added some error messages
+       * gnutls.c: Fixed some compiler warnings
+
+2013-08-08  Will Dietz  <w@wdtz.org> (tiny change):
+
+       * main.c (format_and_print_line): Wrap correctly long tokens.
+
+2013-07-16  Darshit Shah  <darnir@gmail.com>
+
+       * wget.h (err_t): Added new errors, ATTRMISSING and UNKNOWNATTR to
+       handle missing attributes and Unknown attribute values respectively in
+       HTTP Headers.
+       * exits.c (get_status_for_err): ATTRMISSING is a Protocol Error while
+       UNKNOWNATTR is a general error, presumably because of a feature that
+       is not yet implemented.
+       * http.c (gethttp): Call create_authorization_line () separately. In
+       case the auth_err flag has been set with an error, handle it and exit.
+       * http.c (create_authorization_line): Pass a pointer, auth_err to set
+       the flag for different kinds of errors encountered.
+       * http.c (http_loop): Handle the errors raised by the authentication
+       handlers.
+       * http.c (digest_authentication_encode): Pass pointer auth_err to set
+       the error flags.
+       Set qop to NULL in case the value of the qop / algorithm attribute is
+       unknown to Wget. Set an appropriate error too.
+
+2013-07-13  Tim Ruehsen  <tim.ruehsen@gmx.de>
+
+       * http.c (digest_authentication_encode): Fix a crash when the algorithm
+       is not specified in the server response.  Free dynamic memory used by
+       the function when the function exits.
+       * http-ntlm.c [HAVE_NETTLE]: Include <nettle/md4.h> and <nettle/des.h>.
+       (setup_des_key) [HAVE_NETTLE]: New function to deal with
+       libnettle.
+       (calc_resp) [HAVE_NETTLE]: Add support for libnettle.
+       (mkhash) [HAVE_NETTLE]: Likewise.
+       Reported by: Tim Ruehsen  <tim.ruehsen@gmx.de>.
+
+2013-07-13  Steven M. Schweda <sms@antinode.info>
+
+       * warc.c (warc_tempfile): Fix a portability issue on VMS.
+
+2013-07-10  Giuseppe Scrivano  <gscrivano@gnu.org>
+
+       * http.c (read_response_body) [ALLOW_CLOBBER]: Move definition to..
+       * options.h (struct options): Make `backups' an int.
+       * url.h [ALLOW_CLOBBER]: .. Here.  Do not clobber when backups are used.
+       * url.c (url_file_name): Use the ALLOW_CLOBBER macro instead of
+       repeating the code.
+
+2013-07-08  Steven M. Schweda <sms@antinode.info>
+
+       * retr.c (rotate_backups): Support for VMS files.
+
+2013-07-12  Giuseppe Scrivano  <gscrivano@gnu.org>
+
+       * http.c (digest_authentication_encode): Set default value of
+       `algorithm' to "MD5".  Check if `qop' is not-NULL before access it.
+
+2013-07-11  Karsten Hopp  <karsten@redhat.com>
+
+       * openssl.c (struct openssl_read_args, struct scwt_context): New struct.
+       (openssl_read, ssl_connect_with_timeout_callback): New function.
+       (ssl_connect_wget): respect connect timeout.
+
+2013-07-11  Tim Ruehsen  <tim.ruehsen@gmx.de>
+
+        * gnutls.c (ssl_connect_wget): respect connect timeout.
+
+2013-07-11  Tomas Hozza  <thozza@redhat.com>
+
+       * ftp.c (ftp_loop): Use ftp_retrieve_glob() also in case
+       --preserve-permissions was specified.
+
+2013-03-20  Tomas Hozza  <thozza@redhat.com>
+
+       * http.c (gethttp): Set "sock" to -1 if it's not and we have no
+       persistent connection
+
+2013-04-26  Tomas Hozza  <thozza@redhat.com> (tiny change)
+
+       * log.c (redirect_output): Use DEFAULT_LOGFILE in diagnostic message
+       when `logfile' is NULL.
+       * utils.c (unique_create): Ensure `logfile' has always a value.
+
+2013-06-26  Darshit Shah <darnir@gmail.com>
+
+       * http.c (gethttp): Reverse change by commit 90896 that prevented
+       downloading response body data to non-GET Request methods.
+
+2013-06-19 Ciprian Vieru <devel.php@gmail.com> (tiny change)
+
+        * html-url.c: Define TAG_TD, TAG_TH, TAG_VIDEO, TAG_AUDIO, TAG_SOURCE.
+       (tag_url_attributes, known_tags): Likewise.
+
+2013-06-22  Ángel González <keisial@gmail.com>
+
+       * init.c (cmd_string_uppercase): Rewrite function.
+
+2013-06-19  Tim Ruehsen  <tim.ruehsen@gmx.de>
+
+       * connect.c (socket_ip_address): zero out ip address structure to
+       avoid access to uninitialized values by inet_ntop().
+       * ftp.c (ftp_loop_internal): fix segfault caused by warc_tmp NULL pointer.
+
+2013-05-21  Ray Satiro  <raysatiro@yahoo.com>
+
+       * url.c (url_file_name): Use MAX_PATH in Windows.
+
+2013-06-13  Darshit Shah  <darnir@gmail.com>
+
+       * http.c (gethttp): Follow RFC 2616 and httpbis specifications when
+       handling redirections. Do not suspend the method on 301/302 redirects.
+       (gethttp): If method if not GET, we do not intend to download
+       anything.
+       * main.c (main): Set spider mode when opt.method is HEAD. This will
+       prevent Wget from downloading any file.
+       * retr.c (SUSPEND_METHOD): Rename macro SUSPEND_POST_DATA to
+       SUSPEND_METHOD to more accurately reflect its use. Similarly rename
+       related variables.
+
+2013-05-14 Bykov Aleksey <gnfalex@rambler.ru>
+
+       * warc.c (warc_tempfile): For fix "Could not open temporary WARC manifest
+       file." issue in Win system force to use `mkostemp(filename, O_TEMPORARY)'
+       instead  of `mkstemp(filename)'. Thank to Angel Gonzalez for help.
+
+2013-05-14  Tim Ruehsen  <tim.ruehsen@gmx.de>
+
+       * cookies.c (cookie_jar_load): Replaced read_whole_file() by getline().
+       * init.c (run_wgetrc): Likewise.
+       * netrc.c (parse_netrc): Likewise.
+       * utils.c: Likewise.
+       * ftp.c (getftp): Likewise.
+       * ftp-ls.c (ftp_parse_unix_ls, ftp_parse_winnt_ls, ftp_parse_vms_ls): Likewise.
+       * ftp-ls.c (clean_line): Accept the string length as parameter.
+       * ftp-ls.c: Replaced indent tabs by spaces.
+       * ftp.c: Likewise.
+       * utils.c: Removed read_whole_file() definition.
+       * netrc.c: Removed read_whole_file() definition for STANDALONE.
+       * utils.h: Removed read_whole_file() declaration.
+
+2013-05-09  Tim Ruehsen  <tim.ruehsen@gmx.de>
+
+       * utils.c (acceptable): use standard string functions instead of
+       self-written code.
+       (match_tail): Likewise.
+       (suffix): Likewise.
+       (has_wildcards_p): Likewise.
+       (test_subdir_p): Fix some warnings.
+       (test_dir_matches_p): Likewise.
+
+2013-05-05  mancha  <mancha1@hush.com> (tiny change)
+
+       * gnutls.c (ssl_connect_wget): Don't abort on non-fatal alerts
+        received during handshake. For example, when connecting to servers
+        using TSL-SNI that send warning-level unrecognized_name alerts.
+
+2013-05-04  Darshit Shah <darnir@gmail.com>
+
+       * init.c (cmd_string_uppercase): Fix issue that cased invalid headers
+       when converting to uppercase.
+
+2013-05-01  Gijs van Tulder  <gvtulder@gmail.com>
+
+       * retr.c (retrieve_url): New variable `saved_method'.
+       (SUSPEND_POST_DATA): Save the method to `saved_method'.
+       (RESTORE_POST_DATA): Restore the method from `saved_method'.
+
+2013-05-01  Giuseppe Scrivano  <gscrivano@gnu.org>
+
+       * init.c: Declare `cmd_string_uppercase'.
+       (commands): Now `method' uses cmd_string_uppercase.
+       (cmd_string_uppercase): New method
+       * http.c (gethttp): Do not transform opt.method to uppercase.
+       Reported by: Stefano Lattarini <stefano.lattarini@gmail.com>
+
+2013-04-24  Darshit Shah <darnir@gmail.com>
+
+       * http.c (gethttp): Remove check for opt.post_data and
+       opt.post_file_name.
+       * main.c (main): Change location in code where --post-data and
+       --post-file options are converted to --body-data --body-file.
+
+2013-04-21  Gijs van Tulder  <gvtulder@gmail.com>
+
+       * http.c: Copy opt.body_data to the WARC file, instead of
+       opt.post_data (the old option).
+
+2013-04-12  Gijs van Tulder  <gvtulder@gmail.com>
+
+       * warc.c: Generate unique UUIDs for the manifest and the record
+       holding the command-line arguments.
+       Write the manifest to a "metadata" record to follow the WARC
+       implementation guidelines.
+       * warc.h: Declare new function warc_write_metadata_record.
+
+2013-03-15  Darshit Shah <darnir@gmail.com>
+
+       * http.c (post_file): Rename function to body_file_send to more
+       accurately reflect its use.
+       * http.c (gethttp): Add support for --method, --body-data and
+       --body-file
+       * init.c (commands): Same.
+       * options.h (options): Same.
+       * main.c (option_data): Same.
+       * main.c (print_help): Add --method command.
+       * main.c (main): Make old --post-{data,file} commands aliases to
+       --method.
+       Add sanity checks for --method, --body-data and --body-file.
+       * retr.c (SUSPEND_POST_DATA): Edit Macro Definition to use body_data.
+       * retr.c (RESTORE_POST_DATA): Same.
+
+2013-03-31  Gijs van Tulder  <gvtulder@gmail.com>
+
+       * warc.c: Correctly write the field length in the skip length field
+       of .warc.gz files. (Following the GZIP spec in RFC 1952.)
+
+2013-03-12  Darshit Shah <darnir@gmail.com>
+
+       * http.c (gethttp): Make wget return FILEBADFILE error and abort if
+       post-file does not exist.
+
+       * http.c (http_loop): Handle FILEBADFILE as a valid err.
+
+       * exits.c (get_status_for_err): Mark FILEBADFILE as an IO error.
+
+2013-02-15  Darshit Shah <darnir@gmail.com>
+
+       * cookies.c (cookie_handle_set_cookie): Set cookie->discard_requested
+       to true on domain mismatch.
+
 2012-12-20  Tim Ruehsen  <tim.ruehsen@gmx.de>
 
        * gnutls.c (ssl_connect_wget): added +VERS-SSL3.0 to fix