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);  | 
