Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update to libvpx v1.8.0 #4

Closed
EwoutH opened this issue Feb 5, 2019 · 0 comments
Closed

Update to libvpx v1.8.0 #4

EwoutH opened this issue Feb 5, 2019 · 0 comments

Comments

@EwoutH
Copy link

EwoutH commented Feb 5, 2019

libvpx v1.8.0 was tagged yesterday (GitHub)
@Sibras Sibras closed this as completed in cbf8eed Feb 17, 2019
Sibras added a commit that referenced this issue Feb 17, 2019
@Sibras
project: Update for 1.8.0 merge.
4543499 
Closes #4
Sibras added a commit that referenced this issue Feb 17, 2019
@Sibras
project: Update for 1.8.0 merge.
c2c34ca 
Closes #4
Sibras pushed a commit that referenced this issue Apr 6, 2024
@jzern
Revert "Implement d117_predictor using Neon"
394de69 
This reverts commit 360e906.

This causes ASan errors:
[ RUN      ] VP9/TestVectorTest.MD5Match/1
=================================================================
==837858==ERROR: AddressSanitizer: stack-buffer-overflow on address
0xffff82ecad40 at pc 0x000000c494d4 bp 0xffffe1695800 sp 0xffffe16957f8
READ of size 16 at 0xffff82ecad40 thread T0
    #0 0xc494d0 in vpx_d117_predictor_32x32_neon (test_libvpx+0xc494d0)
    #1 0x1040b34 in vp9_predict_intra_block (test_libvpx+0x1040b34)
    #2 0xf8feec in decode_block (test_libvpx+0xf8feec)
    #3 0xf8f588 in decode_partition (test_libvpx+0xf8f588)
    #4 0xf7be5c in vp9_decode_frame (test_libvpx+0xf7be5c)
    ...
Address 0xffff82ecad40 is located in stack of thread T0 at offset 64 in
frame
    #0 0x103fd3c in vp9_predict_intra_block (test_libvpx+0x103fd3c)

  This frame has 2 object(s):
    [32, 64) 'left_col.i' <== Memory access at offset 64 overflows this
                              variable
    [96, 176) 'above_data.i'

Change-Id: I058213364617dfe1036126c33a3307f8288d9ae0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@EwoutH