tree 958e899894cc2cce7cef9a0e3590b6ab76192325
parent 16c91eff09e13ab62cea4ef5044518264483faeb
author Mikael Ohlson <mikael.ohlson@stericsson.com> 1274433639 +0200
committer Christian Bejram <christian.bejram@stericsson.com> 1276725740 +0000

Fix for incorrect reply from sysconf(_SC_NPROCESSORS_ONLN)

When calling sysconf with _SC_NPROCESSORS_ONLN, the value one (1) was
returned on systems with two or more cores, since '/proc/stat' was
incorrectly parsed.

The function line_parser_getc (LineParser* p) read 128 characters of
input for each invocation.

The proper and probably aimed for behavior is to read 128 characters
at the first call, then for each subsequent call only return the next
buffered character until a new read is needed and only then read
another 128 characters.

Due to a flipped comparison between the two variables in_len and
in_pos that track the number of bytes of data read into the input
buffer and how much of it has been parsed, a new group of 128
characters were read at almost every call to line_parser_getc,
overwriting the still unhandled bytes from the previous call to
read. This caused the lines to be read to be sampled more than parsed.

Change-Id: I93eec3c8c9b9f19ef798748579d0977111b5c0bb

Signed-off-by: Christian Bejram <christian.bejram@stericsson.com>