[svn] Fix #20748: fix test usage wrt zero-termination of strings.

Also fixed incorrect "src/" prefix in ChangeLog entry for #20736.

diff --git a/src/ChangeLog b/src/ChangeLog
index d6909aa7da7e844b0a1e6b7c0876241500d889d2..9ca0bace8e5beb6ab7c0dbcf2c22340d12abf9d4 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,6 +1,12 @@
+2007-08-10  Ralf Wildenhues  <Ralf.Wildenhues@gmx.de>
+
+       * url.c (append_uri_pathel): Do not assume dest string to be
+       zero-terminated.
+       (test_append_uri_pathel): Terminate string to fix test failure.
+
 2007-08-09  Ralf Wildenhues  <Ralf.Wildenhues@gmx.de>
 
-       * src/url.c (url_string): Use comparison, not assignment, in
+       * url.c (url_string): Use comparison, not assignment, in
        check for auth_mode == URL_AUTH_HIDE_PASSWD.
 
 2007-08-08  Micah Cowan  <micah@cowan.name>

diff --git a/src/url.c b/src/url.c
index 3c38b14a868300290637e50a80bd011e521c6f58..04ecb3a4f3429170bf0c574df531264edcb77023 100644 (file)
--- a/src/url.c
+++ b/src/url.c
@@ -1,5 +1,5 @@
 /* URL handling.
-   Copyright (C) 1996-2006 Free Software Foundation, Inc.
+   Copyright (C) 1996-2007 Free Software Foundation, Inc.
 
 This file is part of GNU Wget.
 
@@ -1369,7 +1369,7 @@ append_uri_pathel (const char *b, const char *e, bool escaped,
       || opt.restrict_files_case == restrict_uppercase)
     {
       char *q;
-      for (q = TAIL (dest); *q; ++q)
+      for (q = TAIL (dest); q < TAIL (dest) + outlen; ++q)
         {
           if (opt.restrict_files_case == restrict_lowercase)
             *q = TOLOWER (*q);
@@ -2089,6 +2089,7 @@ test_append_uri_pathel()
       
       append_string (test_array[i].original_url, &dest);
       append_uri_pathel (p, p + strlen(p), test_array[i].escaped, &dest);
+      append_char ('\0', &dest);
 
       mu_assert ("test_append_uri_pathel: wrong result", 
                  strcmp (dest.base, test_array[i].expected_result) == 0);