aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEric Andersen <andersen@codepoet.org>2003-04-19 23:15:06 +0000
committerEric Andersen <andersen@codepoet.org>2003-04-19 23:15:06 +0000
commit2894266f12e699570d223d9f19ca79979196060b (patch)
tree7a831df25d9e5a83180c8ee913b7483050ecbbbe
parent343658582bb13bc7cb7b7c2333465f0a090c4942 (diff)
downloadbusybox-w32-2894266f12e699570d223d9f19ca79979196060b.tar.gz
busybox-w32-2894266f12e699570d223d9f19ca79979196060b.tar.bz2
busybox-w32-2894266f12e699570d223d9f19ca79979196060b.zip
Patch from David Updegraff to avoid corrupting memory while parsing the
networks/iterfaces file with next_word routine. Without this, next_word increments one beyond the end of the string.
-rw-r--r--networking/ifupdown.c7
1 files changed, 5 insertions, 2 deletions
diff --git a/networking/ifupdown.c b/networking/ifupdown.c
index fedae8dd6..6c79c2004 100644
--- a/networking/ifupdown.c
+++ b/networking/ifupdown.c
@@ -630,8 +630,11 @@ static char *next_word(char **buf)
630 return(NULL); 630 return(NULL);
631 } 631 }
632 *buf = word + length; 632 *buf = word + length;
633 **buf = '\0'; 633 /*DBU:[dave@cray.com] if we are already at EOL dont't increment beyond it */
634 (*buf)++; 634 if (**buf) {
635 **buf = '\0';
636 (*buf)++;
637 }
635 638
636 return word; 639 return word;
637} 640}