svn commit: branches/busybox_1_1_stable/libbb

aldot at busybox.net aldot at busybox.net
Sat Jun 10 11:05:13 UTC 2006


Author: aldot
Date: 2006-06-10 04:05:12 -0700 (Sat, 10 Jun 2006)
New Revision: 15349

Log:
- fix bug #887, in bb_get_chomped_line_from_file(), the last char was removed
  unconditionally, even if it was not a newline. 
  This was apparently broken by r14254
- whitespace while at it.


Modified:
   branches/busybox_1_1_stable/libbb/get_line_from_file.c


Changeset:
Modified: branches/busybox_1_1_stable/libbb/get_line_from_file.c
===================================================================
--- branches/busybox_1_1_stable/libbb/get_line_from_file.c	2006-06-10 11:04:43 UTC (rev 15348)
+++ branches/busybox_1_1_stable/libbb/get_line_from_file.c	2006-06-10 11:05:12 UTC (rev 15349)
@@ -18,7 +18,7 @@
  * stored and free'ed  by the caller.  If end is null '\n' isn't considered
  * and of line.  If end isn't null, length of the chunk read is stored in it. */
 
-char *bb_get_chunk_from_file(FILE *file, int *end)
+char *bb_get_chunk_from_file(FILE * file, int *end)
 {
 	int ch;
 	int idx = 0;
@@ -30,10 +30,12 @@
 		if (idx > linebufsz - 2) {
 			linebuf = xrealloc(linebuf, linebufsz += 80);
 		}
-		linebuf[idx++] = (char)ch;
-		if (!ch || (end && ch == '\n')) break;
+		linebuf[idx++] = (char) ch;
+		if (!ch || (end && ch == '\n'))
+			break;
 	}
-	if (end) *end = idx;
+	if (end)
+		*end = idx;
 	if (linebuf) {
 		if (ferror(file)) {
 			free(linebuf);
@@ -45,18 +47,21 @@
 }
 
 /* Get line, including trailing /n if any */
-char *bb_get_line_from_file(FILE *file)
+char *bb_get_line_from_file(FILE * file)
 {
 	int i;
+
 	return bb_get_chunk_from_file(file, &i);
 }
 
 /* Get line.  Remove trailing /n */
-char *bb_get_chomped_line_from_file(FILE *file)
+char *bb_get_chomped_line_from_file(FILE * file)
 {
 	int i;
-	char *c=bb_get_chunk_from_file(file, &i);
-	if(i) c[--i]=0;
-	
+	char *c = bb_get_chunk_from_file(file, &i);
+
+	if (i && c[--i] == '\n')
+		c[i] = 0;
+
 	return c;
 }




More information about the busybox-cvs mailing list