wrapper/libavc
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Enable cfi for libavc

Browse source 
Bug: 158010610
Bug: 173497308
Test: Ran atest with DecoderConformanceTest, ImageReaderDecoderTest,
VideoDecoderPerfTest, VideoEncoderTest. Compare the
before and after enabling CFI results since there was a number of tests
that were initially failing.

Change-Id: Ib6b0871809e89bf26c426c5616dea6238f2a02ce
This commit is contained in:
Cindy Zhou 2020-12-02 10:20:30 -08:00
parent 01b93dd80c
commit 91b4a017ed
2 changed files with 28 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

32
Android.bp
View file

@ -225,11 +225,23 @@ cc_library_static {
},
},
target: {
android: {
sanitize: {
diag: {
cfi: true,
},
},
},
},
sanitize: {
integer_overflow: true,
misc_undefined: ["bounds"],
// Enable CFI if this becomes a shared library.
// cfi: true,
cfi: true,
config: {
cfi_assembly_support: true,
},
blocklist: "libavc_blocklist.txt",
},
apex_available: [
@ -486,11 +498,23 @@ cc_library_static {
},
},
target: {
android: {
sanitize: {
diag: {
cfi: true,
},
},
},
},
sanitize: {
integer_overflow: true,
misc_undefined: ["bounds"],
// Enable CFI if this becomes a shared library.
// cfi: true,
cfi: true,
config: {
cfi_assembly_support: true,
},
blocklist: "libavc_blocklist.txt",
},
apex_available: [

1
libavc_blocklist.txt
View file

@ -1,5 +1,4 @@
[cfi]
src:*external/libavc/*
[integer]
# decoder/ih264d_dpb_mgr.c:1174: 2 - 3 cannot be represented in type 'unsigned int'