virtualenv/FFmpeg
1
0
Fork 0
mirror of https://github.com/FFmpeg/FFmpeg.git synced 2025-02-09 14:14:39 +02:00
Code Issues Releases Activity

avcodec/cbs: Remove redundant assignment

Browse Source 
The code just creates new references without allocating
new buffers for the subobjects; therefore the actual data pointer
stays valid and need not be updated.

Also remove an assert that ensured that the calculation
for updating the pointer makes sense.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
This commit is contained in:
Andreas Rheinhardt 2022-08-04 06:55:46 +02:00
parent 33f33beead
commit e35b8b848c
1 changed files with 0 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
libavcodec/cbs.c
View File

@ -955,17 +955,11 @@ static int cbs_clone_internal_refs_unit_content(AVBufferRef **clone_ref,
goto fail; goto fail;
} }
// src_ptr is required to point somewhere inside src_buf. If it
// doesn't, there is a bug somewhere.
av_assert0(*src_ptr >= (*src_buf)->data &&
*src_ptr < (*src_buf)->data + (*src_buf)->size);
*copy_buf = av_buffer_ref(*src_buf); *copy_buf = av_buffer_ref(*src_buf);
if (!*copy_buf) { if (!*copy_buf) {
err = AVERROR(ENOMEM); err = AVERROR(ENOMEM);
goto fail; goto fail;
} }
*copy_ptr = (*copy_buf)->data + (*src_ptr - (*src_buf)->data);
} }
*clone_ref = av_buffer_create(copy, desc->content_size, *clone_ref = av_buffer_create(copy, desc->content_size,