c/libxml2
1
0
Fork 0
mirror of https://gitlab.gnome.org/GNOME/libxml2 synced 2025-03-28 21:33:13 +00:00
Code Issues Packages Projects Releases Wiki Activity

fixing an use of strcpy() where both strings overlap pointed out by

Browse Source 
* uri.c: fixing an use of strcpy() where both strings overlap
  pointed out by valgrind.
Daniel
This commit is contained in:
Daniel Veillard 2003-08-14 22:44:25 +00:00
parent 9848532c2a
commit 608d0ac938
2 changed files with 11 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
ChangeLog
View File

@ -1,3 +1,8 @@
Fri Aug 15 00:41:58 CEST 2003 Daniel Veillard <daniel@veillard.com>
* uri.c: fixing an use of strcpy() where both strings overlap
pointed out by valgrind.
Thu Aug 14 17:10:39 CEST 2003 Daniel Veillard <daniel@veillard.com> Thu Aug 14 17:10:39 CEST 2003 Daniel Veillard <daniel@veillard.com>
* DOCBparser.c globals.c include/libxml/xmlmemory.h: get rid of * DOCBparser.c globals.c include/libxml/xmlmemory.h: get rid of

8
uri.c
View File

@ -691,7 +691,7 @@ xmlNormalizeURIPath(char *path) {
* "current position" pointer. * "current position" pointer.
*/ */
while (1) { while (1) {
char *segp; char *segp, *tmp;
/* At the beginning of each iteration of this loop, "cur" points to /* At the beginning of each iteration of this loop, "cur" points to
* the first character of the segment we want to examine. * the first character of the segment we want to examine.
@ -731,7 +731,11 @@ xmlNormalizeURIPath(char *path) {
cur[0] = '\0'; cur[0] = '\0';
break; break;
} }
strcpy(cur, segp + 3); /* Valgrind complained, strcpy(cur, segp + 3); */
/* string will overlap, do not use strcpy */
tmp = cur;
segp += 3;
while ((*tmp++ = *segp++) != 0);
/* If there are no previous segments, then keep going from here. */ /* If there are no previous segments, then keep going from here. */
segp = cur; segp = cur;