diff options
| author | Quentin Young <qlyoung@nvidia.com> | 2020-09-02 16:43:29 -0400 |
|---|---|---|
| committer | Quentin Young <qlyoung@nvidia.com> | 2020-09-03 14:23:54 -0400 |
| commit | 06cf2c0c36e044dcdc4cdd5f7d6e971bc07a294c (patch) | |
| tree | 745d1fe8f9b565bfc8af41319cefdffa4f008054 /lib/stream.c | |
| parent | e6464fdc184e4b0a4032286c8fae245dcdb03d8f (diff) | |
lib: add stream_rewind_getp()
stream_forward_getp() cannot be used with negative numbers due to the
size_t argument, we'll end up doing overflow arithmetic.
Signed-off-by: Quentin Young <qlyoung@nvidia.com>
Diffstat (limited to 'lib/stream.c')
| -rw-r--r-- | lib/stream.c | 24 |
1 files changed, 24 insertions, 0 deletions
diff --git a/lib/stream.c b/lib/stream.c index 6e62e11380..dc207c16a4 100644 --- a/lib/stream.c +++ b/lib/stream.c @@ -268,6 +268,30 @@ bool stream_forward_getp2(struct stream *s, size_t size) return true; } +void stream_rewind_getp(struct stream *s, size_t size) +{ + STREAM_VERIFY_SANE(s); + + if (size > s->getp || !GETP_VALID(s, s->getp - size)) { + STREAM_BOUND_WARN(s, "rewind getp"); + return; + } + + s->getp -= size; +} + +bool stream_rewind_getp2(struct stream *s, size_t size) +{ + STREAM_VERIFY_SANE(s); + + if (size > s->getp || !GETP_VALID(s, s->getp - size)) + return false; + + s->getp -= size; + + return true; +} + void stream_forward_endp(struct stream *s, size_t size) { STREAM_VERIFY_SANE(s); |
