Skip to content

Commit

Permalink
* sysdeps/unix/sysv/linux/getsysstats.c (next_line): Make sure there
Browse files Browse the repository at this point in the history
	are always at least 4 bytes in the returned line.

2009-04-15  Jakub Jelinek  <jakub@redhat.com>

	* sysdeps/unix/sysv/linux/getsysstats.c (__get_nprocs): Check
	__libc_use_alloca (8192), if the stack is too small use 512 bytes
	instead of 8K.  Stop searching in /proc/stat after hitting first
	line not starting with cpu.
	(next_line): Truncate too long
	lines at buffer size * 3/4 instead of pretending there were line
	breaks inside of large lines.
  • Loading branch information
Ulrich Drepper committed Apr 15, 2009
1 parent ae650a4 commit 6a3d03f
Show file tree
Hide file tree
Showing 2 changed files with 51 additions and 4 deletions.
15 changes: 15 additions & 0 deletions ChangeLog
Original file line number Diff line number Diff line change
@@ -1,3 +1,18 @@
2009-04-15 Ulrich Drepper <drepper@redhat.com>

* sysdeps/unix/sysv/linux/getsysstats.c (next_line): Make sure there
are always at least 4 bytes in the returned line.

2009-04-15 Jakub Jelinek <jakub@redhat.com>

* sysdeps/unix/sysv/linux/getsysstats.c (__get_nprocs): Check
__libc_use_alloca (8192), if the stack is too small use 512 bytes
instead of 8K. Stop searching in /proc/stat after hitting first
line not starting with cpu.
(next_line): Truncate too long
lines at buffer size * 3/4 instead of pretending there were line
breaks inside of large lines.

2009-04-14 Ulrich Drepper <drepper@redhat.com>

* sysdeps/x86_64/mp_clz_tab.c: New file.
Expand Down
40 changes: 36 additions & 4 deletions sysdeps/unix/sysv/linux/getsysstats.c
Original file line number Diff line number Diff line change
Expand Up @@ -93,15 +93,42 @@ next_line (int fd, char *const buffer, char **cp, char **re,
return NULL;

*re += n;

nl = memchr (*cp, '\n', *re - *cp);
while (nl == NULL && *re == buffer_end)
{
/* Truncate too long lines. */
*re = buffer + 3 * (buffer_end - buffer) / 4;
n = read_not_cancel (fd, *re, buffer_end - *re);
if (n < 0)
return NULL;

nl = memchr (*re, '\n', n);
**re = '\n';
*re += n;
}
}
else
nl = memchr (*cp, '\n', *re - *cp);

res = *cp;
nl = memchr (*cp, '\n', *re - *cp);
}

if (nl == NULL)
nl = *re - 1;
}
else if (nl + 5 >= *re)
{
memmove (buffer, nl, *re - nl);
*re = buffer + (*re - nl);
nl = *cp = buffer;

ssize_t n = read_not_cancel (fd, *re, buffer_end - *re);
if (n < 0)
return NULL;

*re += n;
}

*cp = nl + 1;
assert (*cp <= *re);
Expand All @@ -115,8 +142,9 @@ __get_nprocs ()
{
/* XXX Here will come a test for the new system call. */

char buffer[8192];
char *const buffer_end = buffer + sizeof (buffer);
const size_t buffer_size = __libc_use_alloca (8192) ? 8192 : 512;
char *buffer = alloca (buffer_size);
char *buffer_end = buffer + buffer_size;
char *cp = buffer_end;
char *re = buffer_end;
int result = 1;
Expand All @@ -134,7 +162,11 @@ __get_nprocs ()

char *l;
while ((l = next_line (fd, buffer, &cp, &re, buffer_end)) != NULL)
if (strncmp (l, "cpu", 3) == 0 && isdigit (l[3]))
/* The current format of /proc/stat has all the cpu* entries
at the front. We assume here that stays this way. */
if (strncmp (l, "cpu", 3) != 0)
break;
else if (isdigit (l[3]))
++result;

close_not_cancel_no_status (fd);
Expand Down

0 comments on commit 6a3d03f

Please sign in to comment.