diff options
| author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2013-03-30 19:34:23 -0400 |
|---|---|---|
| committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2013-03-31 14:34:08 -0400 |
| commit | 522cd7f18bf1e4a024d6771186a0149c5ca8109c (patch) | |
| tree | 488e36ebe074c5b1caceb2dcdcfe472e2dbfb692 /src/shared/strbuf.c | |
| parent | 4cd5f79d0bf9fb62636485dd36ee961774361c14 (diff) | |
bootchart: fix a potential buffer overrun
If the configured number of samples was close to MAXSAMPLES,
the samples buffer could be overrun:
- by 1, because of off-by-one in the condition (samples > arg_samples_len),
and
- by many in case of an overrun, because the number of samples to
capture was increased, instead of being decreased.
Simplify things by converting to a normal for-loop.
In store.c: change buffer size from 4095 to 4096. 4095 is a strange
number.
Diffstat (limited to 'src/shared/strbuf.c')
0 files changed, 0 insertions, 0 deletions