History log of /external/libavc/decoder/ih264d_api.c
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
e31fa45761ea35c73245a25342e1400ef93ff127 08-Jan-2018 Ray Essick <essick@google.com> Merge "Decoder: Adding Error Check for Output Buffer Size in Shared Display Mode." into mnc-dev am: afc729db25 am: 1fb89d8488 am: 56f836a29c am: 33675f68d6 am: 547804d647 am: 7adc4d57f1 am: cfe44b34c0 am: 17b987beec
am: bce27b7e66

Change-Id: I6755dedeb1a32a32e85627602783ca77cd966fc7
7adc4d57f1d7606d44ef6950d77a937f2244dd31 08-Jan-2018 Ray Essick <essick@google.com> Merge "Decoder: Adding Error Check for Output Buffer Size in Shared Display Mode." into mnc-dev am: afc729db25 am: 1fb89d8488 am: 56f836a29c am: 33675f68d6
am: 547804d647

Change-Id: I22c555cc975abbfa80aa38810c9e7a0f59ce3450
1fb89d8488b6b993e0b9ab1cc35c59c3999c7f44 08-Jan-2018 Ray Essick <essick@google.com> Merge "Decoder: Adding Error Check for Output Buffer Size in Shared Display Mode." into mnc-dev
am: afc729db25

Change-Id: Ic57cdca65a505c45bec9c6ff8dd9b5d9dab5fec7
3692aceb1b244be3e1b36d8e7b804986f593bb69 22-Dec-2017 Ritu Baldwa <ritu.baldwa@ittiam.com> Decoder: Adding Error Check for Output Buffer Size in Shared Display Mode.

The output buffer size given by the application, needs to be checked
in every process call. This is required in the case of resolution
change in shared display mode.

Bug: 70294343
Bug: 70350193
Bug: 70526411
Bug: 70526485
Test: manual
Change-Id: I2c1e59425e84ac62a874e5ee180e1b98f0a4058f
/external/libavc/decoder/ih264d_api.c
423b7db2fb97357930d9f704ab3fd0e92afaf30e 03-Nov-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Merge "Decoder: Handle dec_hdl memory allocation failure gracefully" into mnc-dr-dev am: ac0d9cecb5 am: f63836fed1 am: 5448bcea3e am: f87d4848af am: 18ae83b015 am: 04b8990195 am: 7fc07418ef
am: faf7af464e

Change-Id: I68e929751f26ccbbe9af1104f84bebc07530c6af
18ae83b015b552aa220c31796a3f5b897edaee83 03-Nov-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Merge "Decoder: Handle dec_hdl memory allocation failure gracefully" into mnc-dr-dev am: ac0d9cecb5 am: f63836fed1 am: 5448bcea3e
am: f87d4848af

Change-Id: Ia060e78fe7cd7b4233733910f08caadbea05ff3d
ac0d9cecb5b2fd17735d2fb36c14a468555705ab 03-Nov-2017 TreeHugger Robot <treehugger-gerrit@google.com> Merge "Decoder: Handle dec_hdl memory allocation failure gracefully" into mnc-dr-dev
7720b3fe3de04523da3a9ecec2b42a3748529bbd 26-Oct-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Handle dec_hdl memory allocation failure gracefully

If memory allocation for dec_hdl fails, return gracefully
with an error code. All other allocation failures are
handled correctly.

Bug: 68300072
Test: ran poc before/after
Change-Id: I118ae71f4aded658441f1932bd4ede3536f5028b
/external/libavc/decoder/ih264d_api.c
a1f8cef8c495d500c68e2fa50774aae46735cf88 31-Oct-2017 Hamsalekha S <hamsalekha.s@ittiam.com> Decoder: Increased allocation and added checks in sei parsing. am: f2b70d3537 am: 8d32a711ac am: a0827a3bb1 am: cd831159ce am: 2a3efcfcde am: 013d98b763 am: 48d8b5bbc9
am: ccc5298ffb

Change-Id: I83547b11f6da1c33f0ac304690924eacc2ba2920
2a3efcfcde82d90e4a7474044d1bb3a1648ff8b1 31-Oct-2017 Hamsalekha S <hamsalekha.s@ittiam.com> Decoder: Increased allocation and added checks in sei parsing. am: f2b70d3537 am: 8d32a711ac am: a0827a3bb1
am: cd831159ce

Change-Id: I49ada0ebdcca121f739c753dbedaca8e80ea6da5
27e6efc8ec6591f2759d7762d60a19378aec9681 18-Oct-2017 Ray Essick <essick@google.com> DO NOT MERGE Decoder: Increased allocation and added checks in sei parsing.

This prevents heap overflow while parsing sei_message.

Bug: 63122634
Test: ran PoC on unpatched/patched
Change-Id: I4785927b68cb17a3ca51e23aeaf96aacacf116d3
Merged-In: I61c1ff4ac053a060be8c24da4671db985cac628c
/external/libavc/decoder/ih264d_api.c
f2b70d353768af8d4ead7f32497be05f197925ef 04-Jul-2017 Hamsalekha S <hamsalekha.s@ittiam.com> Decoder: Increased allocation and added checks in sei parsing.

This prevents heap overflow while parsing sei_message.

Bug: 63122634
Test: ran PoC on unpatched/patched
Change-Id: I61c1ff4ac053a060be8c24da4671db985cac628c
/external/libavc/decoder/ih264d_api.c
6a09a35355c0bbe7231e20258e4342b97c4b98c4 04-Oct-2017 TreeHugger Robot <treehugger-gerrit@google.com> Merge "DO NOT MERGE Decoder: Modified setting of error code in set flush api." into mnc-dev
1541dcc79cfd648a64e27735e4d6c7d2b1e4d345 21-Sep-2017 Ritu Baldwa <ritu.baldwa@ittiam.com> DO NOT MERGE Decoder: Modified setting of error code in set flush api.

Fixed incorrect use of ps_dec->pv_dec_out to set error code.

Bug: 66372937
Test: at vendor
Merged-In: Ib04e0b15573b2482c9d5b43c8bc7dd30d8f8efdd
Change-Id: I7b66ee010089399c050a75d6d67feb03da0b8b3e
/external/libavc/decoder/ih264d_api.c
ed7a63fbcc157acb68d7ac127c94ff03fb6ee2d4 20-Sep-2017 Ritu Baldwa <ritu.baldwa@ittiam.com> DO NOT MERGE Decoder: Increased memory allocation.

ps_dec->u1_recon_mb_grp is twice the width in case of mbaff,
increasing relevant allocations accordingly.

Increased allocation of intra-prediction buffer to
include padding.

Bug: 64964675
Test: POC from bug report
Change-Id: Ic4a6151bb12ac1122c228220b9150b2a372aae21
/external/libavc/decoder/ih264d_api.c
31b397c813a588a1eadec3f6bfe902a0ca4910a1 12-Sep-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Added an out of bound check on u4_num_bufs in input argument am: aa11ab9fdb am: 50ec420228 am: 0514a1ebfb am: fa8bb9d89c am: 96228ba3b9 am: 4a6a743d40 am: 4ed04c7bb1 am: b9027e8cc1
am: 859498bb08

Change-Id: I364cdcdd56d88f411892723935683d16ffcd0321
1d6b06a73a5dedb5d87cf21617cd8f02e034d2b5 12-Sep-2017 Hamsalekha S <hamsalekha.s@ittiam.com> Decoder: Conceal picture only if valid picture buffer is obtained. am: 5df744afde am: 7f78e3aa9a am: 8810e9db7f am: 1ab3e51baf am: 32c369cc3f am: fd327a47fb am: b10090d47a
am: 2825c578d3

Change-Id: I1bf0cb0c96c45b419c06bd9b1d202b1ff08193e4
4a6a743d405bf69ea21818a17b1df9f64c535d5c 12-Sep-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Added an out of bound check on u4_num_bufs in input argument am: aa11ab9fdb am: 50ec420228 am: 0514a1ebfb am: fa8bb9d89c
am: 96228ba3b9

Change-Id: Ic048f96697bdaedea3bbc0a8cf1acf78ed488e37
32c369cc3f1d6d5a86b93eaa466a3cd67748ce69 12-Sep-2017 Hamsalekha S <hamsalekha.s@ittiam.com> Decoder: Conceal picture only if valid picture buffer is obtained. am: 5df744afde am: 7f78e3aa9a am: 8810e9db7f
am: 1ab3e51baf

Change-Id: I964f1bd1fc0d9534d6c9498864cc7831d13de449
50ec420228f6256ad986003c7d4d39ef09957800 12-Sep-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Added an out of bound check on u4_num_bufs in input argument
am: aa11ab9fdb

Change-Id: I7dd6739821da87a96072dae422416e8037332a4f
dc9396971dd3e3d9f06644a109e79f2c17fa8487 12-Sep-2017 Ray Essick <essick@google.com> Merge "DO NOT MERGE: Decoder: Conceal picture only if valid picture buffer is obtained." into mnc-dev
45aa4ac86d72a1783eb6d54bdf4b3a58fa1da08d 07-Sep-2017 Ray Essick <essick@google.com> DO NOT MERGE: Decoder: Conceal picture only if valid picture buffer is obtained.

If all the slices in the current pic were invalid, then
the decoder would not have received a valid picture buffer
in the current call. In such cases there is no need to conceal or
deblock the picture.

Bug: 62896384
Test: run ASAN-enabled PoC before/after the patch
Change-Id: Ia7d979a78ae3f4fb443a1759c8e6cf8780565ad8
Merged-In: I3cf6e871592826f93b0dcd2b06fff80677bc8338
/external/libavc/decoder/ih264d_api.c
5df744afde273bc4d0f7a499581dd2fb2ae6cb45 22-Jun-2017 Hamsalekha S <hamsalekha.s@ittiam.com> Decoder: Conceal picture only if valid picture buffer is obtained.

If all the slices in the current pic were invalid, then
the decoder would not have received a valid picture buffer
in the current call. In such cases there is no need to conceal or
deblock the picture.

Bug: 62896384
Test: run ASAN-enabled PoC before/after the patch
Change-Id: I3cf6e871592826f93b0dcd2b06fff80677bc8338
/external/libavc/decoder/ih264d_api.c
aa11ab9fdbb63766703a6280f4fc778f2f2c91ed 16-Jun-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Added an out of bound check on u4_num_bufs in input argument

ps_dec_ip->s_out_buffer.u4_num_bufs was missing out of bound checks

Bug: 62688399
Change-Id: Ic5e5c002d29fcb18064550d5a5f9289bb68b448e
/external/libavc/decoder/ih264d_api.c
2e10678938719f58e47e693dd8dd4db78d86fef8 19-Jun-2017 Hamsalekha S <hamsalekha.s@ittiam.com> Added error check for output buffer size. am: 3f6c941de5 am: f0d2caa214 am: 9a6312ec9f am: 83c397d1ab am: 462316f8d2 am: 564bdc7609 am: 3aa85a9cc5 am: 873d325720
am: 08d268a942

Change-Id: I04d24975487e9188d30ff1ebac6f76e563143ef4
564bdc7609ac2b618d83321047cd8ae9d2291d35 19-Jun-2017 Hamsalekha S <hamsalekha.s@ittiam.com> Added error check for output buffer size. am: 3f6c941de5 am: f0d2caa214 am: 9a6312ec9f am: 83c397d1ab
am: 462316f8d2

Change-Id: Id9021bcb9534cd9b30063eb9be6de464014c67d7
f0d2caa214164a96db011c753ac4a4106551b6f9 19-Jun-2017 Hamsalekha S <hamsalekha.s@ittiam.com> Added error check for output buffer size.
am: 3f6c941de5

Change-Id: I5b06314d7b3d0fb699160ba1326ecb31aedf3082
3f6c941de5cd959072fa046c9d6cb26fa0f01dc6 01-Jun-2017 Hamsalekha S <hamsalekha.s@ittiam.com> Added error check for output buffer size.

The output buffer size given by the application, needs to be checked
in every process call. This is required in the case of resolution
change.

Bug: 36006815
Test: avcdec -i poc.bin
Change-Id: I16a92cdad23eb7b1e12c1a67c1b2599204f29249
/external/libavc/decoder/ih264d_api.c
36ffbf2beb5679a866636fca06791a36f047e4fa 06-Jun-2017 Marco Nelissen <marcone@google.com> Merge "Initializing reference list for every P/B slice." into mnc-dev am: 3477b8e0d7 am: d98f1cf41b am: 18c5ca6984 am: 3cc8a48116 am: 93767dc556 am: 900303e2b0 am: 610062c062 am: 12d32dedae
am: 3b7dde2b72

Change-Id: Ib1a5ed2bf00305861149aa220ac54b65673da4b2
900303e2b0f11d0fb50d2a5da098678144140754 06-Jun-2017 Marco Nelissen <marcone@google.com> Merge "Initializing reference list for every P/B slice." into mnc-dev am: 3477b8e0d7 am: d98f1cf41b am: 18c5ca6984 am: 3cc8a48116
am: 93767dc556

Change-Id: I7a1787851643bd77bb509f65d378482bb4f1957c
d98f1cf41b871943884ea039fca916f404a97bd5 06-Jun-2017 Marco Nelissen <marcone@google.com> Merge "Initializing reference list for every P/B slice." into mnc-dev
am: 3477b8e0d7

Change-Id: I0ccb9c141c7e7e1b8be6e8e6ddc1c333c10a8662
3477b8e0d72d701195775f94aae0b97050c08534 06-Jun-2017 Marco Nelissen <marcone@google.com> Merge "Initializing reference list for every P/B slice." into mnc-dev
a7b1d1c722572cd1adb4a950aef73fd5c0bb996b 06-Jun-2017 Hamsalekha S <hamsalekha.s@ittiam.com> Merge "Fix resolution change within a decode call." into mnc-dev am: 7fc42b801c am: 252ca18add am: 4c6288e861 am: 90fc697c82 am: 8b3d043280 am: d603f906d2 am: e1b496babb am: 85b8159cf0
am: 6edcc25a5a

Change-Id: I72143bf248487f085dbf816f9b2795c837dbb575
d603f906d26eede34ea1610166e672c694511d3b 06-Jun-2017 Hamsalekha S <hamsalekha.s@ittiam.com> Merge "Fix resolution change within a decode call." into mnc-dev am: 7fc42b801c am: 252ca18add am: 4c6288e861 am: 90fc697c82
am: 8b3d043280

Change-Id: I3812d5c6cd9ea03959c020ad12ceec69144a282c
252ca18add9f47802329c801edb42a8ba051d6b0 06-Jun-2017 Hamsalekha S <hamsalekha.s@ittiam.com> Merge "Fix resolution change within a decode call." into mnc-dev
am: 7fc42b801c

Change-Id: Ib6f3680beb6e5d837f2140eed3107a23491f99f6
7fc42b801c020ededb850ed43cfc7f39cf95383e 06-Jun-2017 TreeHugger Robot <treehugger-gerrit@google.com> Merge "Fix resolution change within a decode call." into mnc-dev
fe18375850fe04b8c4ff2f1b20069e161f718e53 22-May-2017 Hamsalekha S <hamsalekha.s@ittiam.com> Fix resolution change within a decode call.

If resolution changes within a decode call,due to multiple
sps, the decoder hangs as the the application will
give the same data again in the next decode call. This
results in a hang. Fixed this by flaging an error,
when sps/resoultion changes within a process call.

Bug: 38487564
Test: ran POC on patched O-based system w/o hanging
Change-Id: I30095b2e8bf573c1a58a316a23b1a5e6a4af589b
/external/libavc/decoder/ih264d_api.c
0a91df5eca850356826c5dc4b7332112a7f43a2a 02-Jun-2017 TreeHugger Robot <treehugger-gerrit@google.com> Merge "DO NOT MERGE Decoder: Fixed allocation size of pred info buffer" into mnc-dev
f9d3f9af8fc113acda28e1a4e48d85736ee29c75 23-May-2017 Hamsalekha S <hamsalekha.s@ittiam.com> Initializing reference list for every P/B slice.

Reference list needs to be initialized for every P/B
slice, to ensure colocated picture always points to a
valid picture buffer, even in the case of error.

Bug: 36279112

Change-Id: I051d7e725b0af209cc7bb333db8da3518adf78a0
/external/libavc/decoder/ih264d_api.c
70593599f5d9b7bdfdfcd0955f3e24c9cbeae0d1 16-May-2017 Marco Nelissen <marcone@google.com> Merge "Decoder: Fix allocation for Mbaff weight matrix" into mnc-dr-dev am: db8f1ae73f am: dcbf996666 am: 8b4c87943a am: 672cb37cc6 am: ad69241f3a am: 7ba9ed2338 am: fed8ee727b
am: 5ed090d7d8

Change-Id: Ie8cd53e71eea8d68e6235f35611f388b415c8154
ad69241f3a9f6383c8fbc1fcd57530ec5f9d2a92 16-May-2017 Marco Nelissen <marcone@google.com> Merge "Decoder: Fix allocation for Mbaff weight matrix" into mnc-dr-dev am: db8f1ae73f am: dcbf996666 am: 8b4c87943a
am: 672cb37cc6

Change-Id: Ic3a2832c01bd74ab7a3c788f5e04a6161fed4c78
db8f1ae73f7aacdd42014a96fb66bc4736145e7f 16-May-2017 Marco Nelissen <marcone@google.com> Merge "Decoder: Fix allocation for Mbaff weight matrix" into mnc-dr-dev
c0994c1fea42862a5035729b306a5f940b4b9caa 13-May-2017 Hamsalekha S <hamsalekha.s@ittiam.com> Merge "Decoder: Fixed flag u1_top_bottom_decoded." into mnc-dev am: cc07eaa5d2 am: 9a5c62841e am: 514d08016a am: 4dd1490ec9 am: 122d0942a8 am: 917c7eee1c am: d189b84641 am: 9e191ce3a6
am: ae093b98bc

Change-Id: Ia1f3dd1694f82f24fb7fa006d722c7a9f38a1572
917c7eee1c658979297bd3e5b0719d91941da27d 13-May-2017 Hamsalekha S <hamsalekha.s@ittiam.com> Merge "Decoder: Fixed flag u1_top_bottom_decoded." into mnc-dev am: cc07eaa5d2 am: 9a5c62841e am: 514d08016a am: 4dd1490ec9
am: 122d0942a8

Change-Id: I53408f560fed7e2a5d88d5a3e3e5f77f01deb41d
9a5c62841ef5a24c1b4d1bad50af1ad31a6535c5 13-May-2017 Hamsalekha S <hamsalekha.s@ittiam.com> Merge "Decoder: Fixed flag u1_top_bottom_decoded." into mnc-dev
am: cc07eaa5d2

Change-Id: Ie410da29e20aed95d674aaaeddd683e1f616c448
07db35ad5af8c4ee2308f983650d9a1b811841ea 08-May-2017 Hamsalekha S <hamsalekha.s@ittiam.com> Decoder: Fix allocation for Mbaff weight matrix

Increased the allocation size for Mbaff weight
matrix buffer

Bug: 36996978

Change-Id: I21cf2cb1010abdc6346f743f5237ae1730c4bf41
/external/libavc/decoder/ih264d_api.c
7703822731a3e5425390ba1d177d061a699c367d 08-May-2017 Hamsalekha S <hamsalekha.s@ittiam.com> Decoder: Fixed flag u1_top_bottom_decoded.

Fixed initialization of flag u1_top_bottom_decoded
in decoder context. This flag indicates if top
field and botton field is decoded.

Bug: 36993291
Test: avcdec --input poc.h264 --output /dev/null
Change-Id: I9f8a2620683abd8b15e4780d76d4849394710716
/external/libavc/decoder/ih264d_api.c
6a5cb800683fa45862267a44446d0492c867215c 08-May-2017 Hamsalekha S <hamsalekha.s@ittiam.com> DO NOT MERGE Decoder: Fixed allocation size of pred info buffer

Buffer allocation size for pred info was increased
in the case number reference frames equal to 1.

Bug: 36998372
Change-Id: I1f84a16703422109d40bed8436f35d0c2069c088
/external/libavc/decoder/ih264d_api.c
d0ab5b06d7890a6ae535e8e0fd91aecfc609e625 21-Feb-2017 Hamsalekha S <hamsalekha.s@ittiam.com> Bug fix for flush without valid frames

We now return fail and get out of flush mode to accept bitstream in the next call.

Bug: 35585952
Test: test case does not hang

Change-Id: Id22cc98d4a47714475a67918990a181a805c4c9f
/external/libavc/decoder/ih264d_api.c
b1bbfa532082997345569d192d4951a33f68e38c 14-Feb-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Fixed initialization of first_slice_in_pic am: 0b23c81c3d am: 16fa990ce1 am: ab79eb3973 am: a37d120eb2 am: c011eeade7 am: 94e17dc822 am: 9117d918dd
am: f837a7796b

Change-Id: Ic4d24cc5d546abf49a2cef788226ff3a3ab92720
20c2cfbc12c3e8d1dae93b522172fe58235c071e 14-Feb-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Moved end of pic processing to end of decode call am: 494561291a am: ba7f9e2aed am: 26bb8b98ee am: 89fa016b5f am: df2fa01c9e am: 7944003371 am: fd5c8e352a
am: 9432beb534

Change-Id: Icd9bf9c1786a5758cf8ec4f6a5b4b6a96daef1dd
94e17dc82295b9049a59e28263763c2beee2498e 14-Feb-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Fixed initialization of first_slice_in_pic am: 0b23c81c3d am: 16fa990ce1 am: ab79eb3973 am: a37d120eb2
am: c011eeade7

Change-Id: I3f850497c864277c2b2524fab07d14a249e48afc
794400337150a7bd6b356c2fb9cb13632cae8d61 14-Feb-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Moved end of pic processing to end of decode call am: 494561291a am: ba7f9e2aed am: 26bb8b98ee am: 89fa016b5f
am: df2fa01c9e

Change-Id: Ifc329cbccad01c70ff1fea0d483cb155b15b3cc1
16fa990ce15cae5dcf3fac059cd18264f50881b2 14-Feb-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Fixed initialization of first_slice_in_pic
am: 0b23c81c3d

Change-Id: I55e49ca9616a151456342bad7673d0986098b448
ba7f9e2aeddeda9b91addbe1d0abc68499f9dfc7 14-Feb-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Moved end of pic processing to end of decode call
am: 494561291a

Change-Id: Id3867f5dd0788f12f6827ed311a309d1fa58c535
0b23c81c3dd9ec38f7e6806a3955fed1925541a0 16-Jan-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Fixed initialization of first_slice_in_pic

To handle some errors, first_slice_in_pic was being set to 2.
This is now cleaned up and first_slice_in_pic is set to 1 only once per pic.
This will ensure picture level initializations are done only once even in case
of error clips

Bug: 33717589
Bug: 33551775
Bug: 33716442
Bug: 33677995

Change-Id: If341436b3cbaa724017eedddd88c2e6fac36d8ba
/external/libavc/decoder/ih264d_api.c
494561291a503840f385fbcd11d9bc5f4dc502b8 04-Jan-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Moved end of pic processing to end of decode call

ih264d_end_of_pic() was called after parsing slice of a new picture.
This is now being done at the end of decode of the current picture.
decode_gaps_in_frame_num which needs frame_num of new slice is now
done after decoding frame_num in new slice.

This helps in handling errors in picaff streams with gaps in frames

Bug: 33588051
Bug: 33641588
Bug: 34097231
Change-Id: I1a26e611aaa2c19e2043e05a210849bd21b22220
/external/libavc/decoder/ih264d_api.c
1f0ab0b089e357da4ecfec4f105726bfc23af5a0 23-Dec-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> DO NOT MERGE: Decoder: Fixes an out of bound write in bitstream buffer

[this is for mnc-dev only, not any other mnc-*-dev flavors;
there is a different patch for mnc-dr-dev, nyc-* and going forward]

After emulation prevention, data is written as an int,
so at least 3 additional bytes should be available.
And since bitstream functions read 8 bytes ahead, 8 extra bytes
should be available in the bitstream buffer.

Bug: 33934721
Test: Ittiam testing, POC in the bug no longer fails
Change-Id: I444ec6f85d01b0bade9f827e15c4b476779d6c69
/external/libavc/decoder/ih264d_api.c
d24b4a2fca3307bb6c37601925386ead131c2a51 13-Feb-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Fixes an out of bound write in bitstream buffer am: 33ef7de9dd am: 0aab4958fb am: 9f72b6eca1 am: 73a3e320d8 am: 89377f2acf am: 9801d7e8d5
am: 109509aeb0

Change-Id: I5d0e3112f87665bc116966c963b2b60bbdc586d7
89377f2acfc37d6fc2c6a0d2d127913e1d2a0114 13-Feb-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Fixes an out of bound write in bitstream buffer am: 33ef7de9dd am: 0aab4958fb am: 9f72b6eca1
am: 73a3e320d8

Change-Id: Id06cfbad9824a7921e4eb082417f14aa842bcfab
33ef7de9ddc8ea7eb9cbc440d1cf89957a0c267b 23-Dec-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Fixes an out of bound write in bitstream buffer

[for mnc-dr-dev and later; mnc-dev gets a different patch]

After emulation prevention, data is written as an int,
so at least 3 additional bytes should be available.
And since bitstream functions read 8 bytes ahead, 8 extra bytes
should be available in the bitstream buffer.

Bug: 33934721

Change-Id: I444ec6f85d01b0bade9f827e15c4b476779d6c69
/external/libavc/decoder/ih264d_api.c
d5424f66e3bafea7112d6e864cb9f5ea64cbeef3 18-Jan-2017 Marco Nelissen <marcone@google.com> Merge "DO NOT MERGE Decoder: Increase memory allocation for weights & offsets for interlaced clips" into mnc-dev
4b2c277d9c69233c5611e6d081438847245d1dc4 18-Jan-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Increase memory allocation for weights & offsets for interlaced clips am: bee9b9a54b am: bfc735cf17 am: 87c2d690fa am: 826159785f am: 3f1af4a90c am: 1806c2e12b
am: e4acdfb8c3

Change-Id: I6fb6416d7960ba01036fc1098993b198b6fb1350
3f1af4a90c928525919b285523d76308a0ccc755 18-Jan-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Increase memory allocation for weights & offsets for interlaced clips am: bee9b9a54b am: bfc735cf17 am: 87c2d690fa
am: 826159785f

Change-Id: Iba990d241d8eb893356b748812342bab4ffecae2
bfc735cf17e3d0358098fd12f79f45e3e85efe2b 18-Jan-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Increase memory allocation for weights & offsets for interlaced clips
am: bee9b9a54b

Change-Id: I8c0bd3d77b87e6fb3f982ef70854b8ec73298636
696b6f666c15df8feb555ce2c9f275c1a4294656 18-Jan-2017 Marco Nelissen <marcone@google.com> Merge "Decoder: Increase memory allocation for weights & offsets for interlaced clips" into mnc-dr-dev
3101a6aa5a9de84f9327c3101ab63bf3738cb1fb 18-Jan-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Fixed DoS in header decode when no PPS is present am: 2cd2f7a335 am: cec6503a20 am: 031a3fdbba am: 5a44d81d8c am: 60566cca95 am: e951e02347 am: a89050b3f2
am: b0dd5208aa

Change-Id: I34ade8ad8531dfb9daa741371cb5359cf1a51152
e951e02347d7d9c1e43d07d3390ff22c2bbc7581 18-Jan-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Fixed DoS in header decode when no PPS is present am: 2cd2f7a335 am: cec6503a20 am: 031a3fdbba am: 5a44d81d8c
am: 60566cca95

Change-Id: If921d86c80aa9ece6dd429d0e8a34edf02e97926
cec6503a20251ae88eaca36052a8f9be40c0d48e 18-Jan-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Fixed DoS in header decode when no PPS is present
am: 2cd2f7a335

Change-Id: If8a7c11ef215a84a350cd0a734bca9c69b91f1c6
e74ab7dba432fd80f0ccd4890c73d59d200a4e76 18-Jan-2017 TreeHugger Robot <treehugger-gerrit@google.com> Merge "Decoder: Fixed DoS in header decode when no PPS is present" into mnc-dev
bee9b9a54b5f7a30991b630a13659e573f0cb52d 13-Jan-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Increase memory allocation for weights & offsets for interlaced clips

Bug: 33816782
Change-Id: I65a5376f46902139f9fc49a2fff54b53b518d703
/external/libavc/decoder/ih264d_api.c
2cd2f7a33535f29c87a1952e0d4b43a80b6f1f3c 16-Jan-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Fixed DoS in header decode when no PPS is present

When the input does not contain PPS and decoder is in header decode
mode, decoder was entering an infinite loop.

Bug: 33621215
(cherry picked from commit 33e1b190d6db09bd72a9f0f51acef4b14eabd6ff)
/external/libavc/decoder/ih264d_api.c
74f03b639e48a31c1c7e21583a244d6e1db1b97f 13-Jan-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> DO NOT MERGE Decoder: Increase memory allocation for weights & offsets for interlaced clips

Bug: 33816782
Change-Id: I10f8b2a7691aeb55365901c166e765ab22ec6106
/external/libavc/decoder/ih264d_api.c
6887e085d7c1edc01b2ccc1273d7d5f5ea7c821a 17-Jan-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Fixed number of MB calculation for interlaced error streams am: e1cf7ea8ae am: 4e7779e0ef am: ba9af1da5d am: aa51017b8f am: 06b61cfbe1 am: f273d0f53b am: 2396631fe9
am: 523e19f292

Change-Id: If3acb58877f94f30eae6af1225532fae2e133579
f273d0f53b8d7af7dc6b4b0bcca3a05f351fea60 17-Jan-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Fixed number of MB calculation for interlaced error streams am: e1cf7ea8ae am: 4e7779e0ef am: ba9af1da5d am: aa51017b8f
am: 06b61cfbe1

Change-Id: I7c8748afaefdddbc170ab55c1da7fa7fe7ae0d16
4e7779e0ef6c93b6285746c5cdbc20dead7839b1 17-Jan-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Fixed number of MB calculation for interlaced error streams
am: e1cf7ea8ae

Change-Id: If723b1da6adf7a816c095f327023853bcaccd0c5
c4f317b87322fc887903cd386c0db8e1bf4e46be 17-Jan-2017 Marco Nelissen <marcone@google.com> Merge "Decoder: Fixed number of MB calculation for interlaced error streams" into mnc-dev
4300da4d0ede28c1ddb0aa173039680dd31a4646 09-Jan-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Call ih264d_deblock_display only for valid process calls am: 318368435f am: 2c71fb1941 am: d2977b77e3 am: d66e74cdbf am: 40ebdfd0fb am: cdb1b16a45 am: f9627fe750
am: ee30608e37

Change-Id: Idf70ab86588118496148cb32213ddc5856eec6fc
cdb1b16a4595a262a4738cffc8a7286e37dff4d5 09-Jan-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Call ih264d_deblock_display only for valid process calls am: 318368435f am: 2c71fb1941 am: d2977b77e3 am: d66e74cdbf
am: 40ebdfd0fb

Change-Id: I363759fc9520fd49f2b365d42e578e10d858db04
2c71fb19418053d49fea59366338ae07897d10e2 09-Jan-2017 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Call ih264d_deblock_display only for valid process calls
am: 318368435f

Change-Id: I210dfb1eaf04b3ddceffa041c43e25e52023090d
e1cf7ea8ae9af4d8b5aca7efba61025dae10a345 26-Dec-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Fixed number of MB calculation for interlaced error streams

At the end of picture processing, if the current pic is partially
decoded, number of MBs to be processed was wrongly calculated for
interlaced cases.

Bug: 33129467
Change-Id: Ia81186c60d346f02663607f2dc14166781db6a69
/external/libavc/decoder/ih264d_api.c
318368435f38629288dc500081a018dc698ca1f1 26-Dec-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Call ih264d_deblock_display only for valid process calls

ih264d_deblock_display() should be called only if current
decoder call has got a picture buffer during start of pic

Bug: 33751193
Change-Id: I3bf2fdbb910bf7240484dae48b85d72833830d56
/external/libavc/decoder/ih264d_api.c
b258b1bd56b1651126f77ef1c6932d766feda655 25-Oct-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Ignore dangling fields during flush
am: 4b6344b281

Change-Id: I8017e81b6d14cb507f297c54a95748455d6b5484
4b6344b28159ddec7e9eb3de074d5e3affff178a 21-Oct-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Ignore dangling fields during flush

If decoder has decoded one field and is waiting for next field,
and flush is called, then ignore the current decoded field.

This is needed to ensure seek works in case of flush between field
decode calls

Bug: 32391809
Test: Tested manually by decoding clip in the above bug in Photos app

Change-Id: I72bb0385c216ebe262b346a56d3bc8fb97f90a7b
/external/libavc/decoder/ih264d_api.c
a2723f86479c1825ca37cf627fc4b333a9a28d00 21-Sep-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Fix in the case of invalid SPS PPS am: c2e8ffe451 am: ff7ccdab0c am: 956ab4c476 am: 5c1f3eb993
am: 8b610a8350

Change-Id: I586ffd87cbb76040e18eed79b8b776dcd1567e5c
86c5eddcf6ee1d4c2db469f669c6fa733598be8e 21-Sep-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Fix in the case of invalid SPS PPS am: c2e8ffe451 am: ff7ccdab0c am: 956ab4c476 am: 5c1f3eb993
am: 8b610a8350

Change-Id: I610cbaeccbf36d7209264e46faa34b8c4d5df17e
ff7ccdab0c2b98c12afa7409f951ed4d42c8d64a 21-Sep-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Fix in the case of invalid SPS PPS
am: c2e8ffe451

Change-Id: Iaf266c5ff2187ba980737b699884e3ebebe87bd0
c2e8ffe451e2a78161037ca09a3516e65c80675c 17-Sep-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Fix in the case of invalid SPS PPS

Bug: 31092462
Change-Id: I0e2ab5a1088717bbf36f6b0cf859b5a6e0f59c5d
/external/libavc/decoder/ih264d_api.c
b424a5e10159c4f6d1c3dbf23e3802b5db8a27fd 14-Jul-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Fixed error concealment when no MBs are decoded in the current pic am: b926905ee2 am: ff54ee6115 am: 9fca73a55e am: 19080458ba
am: 9836aa04a9

Change-Id: I7a2f3551c304c180df9e05b06de9de7bef0b9436
7419a0e03932da2ce9f3e51d1afb07d5151a6f08 14-Jul-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Fixed error concealment when no MBs are decoded in the current pic am: b926905ee2 am: ff54ee6115 am: 9fca73a55e am: 19080458ba
am: 9836aa04a9

Change-Id: Iccfd85fcf60aa708c08fdebe0dc1d4bd27141181
ff54ee611508985a780e09488dd1319e30a77017 14-Jul-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Fixed error concealment when no MBs are decoded in the current pic
am: b926905ee2

Change-Id: I8a0a6fc068b543a4431b085c9335301e89ef2e14
b926905ee245644806e62bef92b61c8832c44fd8 12-Jul-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Fixed error concealment when no MBs are decoded in the current pic

Bug: 29493002
Change-Id: I3fae547ddb0616b4e6579580985232bd3d65881e
/external/libavc/decoder/ih264d_api.c
b4a03ec63479ec515d09ab3d0148963628241b0e 12-Jul-2016 Hangyu Kuang <hkuang@google.com> Merge "DO NOT MERGE Decoder: Added support for exporting VUI params" into nyc-mr1-dev
ba93b7fedbaadf310df259f0ac1b1c18c3fbcf32 26-Apr-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> DO NOT MERGE Decoder: Added support for exporting VUI params

Bug: 27442922

Change-Id: I4bc3a9fb31ffea1e9e8ae32e5ed49fcc12fbdf42
/external/libavc/decoder/ih264d_api.c
8b078e826e65f30c7dfa1e68b295d9cb7f4d66b4 23-Jun-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Added support for exporting VUI params am: d5953ced23
am: a1fa8e86e8

Change-Id: I1363116738bb33b3c9a999ba7159f4c80d9eef89
a1fa8e86e8c334dcc6a652d9a6f76a5a1658ce53 23-Jun-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Added support for exporting VUI params
am: d5953ced23

Change-Id: I01485a055330902830f64850c1ddfa2ce4b8ce25
d5953ced23853f0f59200a59a6ef62dd8a15ef05 26-Apr-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Added support for exporting VUI params

Bug: 27442922

Change-Id: I4bc3a9fb31ffea1e9e8ae32e5ed49fcc12fbdf42
/external/libavc/decoder/ih264d_api.c
1cb4c241b9a9eb27500407339390a6e15a469d9f 21-Jun-2016 Ray Essick <essick@google.com> Merge \\"Decoder: Simplified error checks on display stride\\" am: e8463cdf28
am: fb5a6f6fcc

Change-Id: I19f6dbf1268ad302a4c2fc5b8dd2f6f30948f2db
fb5a6f6fcc27cb871767a7fc2a440b444833e814 21-Jun-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Merge \"Decoder: Simplified error checks on display stride\"
am: e8463cdf28

Change-Id: Ie404c25afb8461451b738a174431c2cd6d6c03be
c73e0006657b83dc3ea030e75e60194fd6d0628b 21-Jun-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Merge \"Decoder: Initialize first_pb_nal_in_pic for error slices\" into mnc-dev
am: b9d7cba8bf

Change-Id: I62f9c45eea70968942198b20f47c158dede39d8d
b9d7cba8bf0ad47bf0b0faae5bb53a13e23006a2 21-Jun-2016 Robert Shih <robertshih@google.com> Merge "Decoder: Initialize first_pb_nal_in_pic for error slices" into mnc-dev
48b182db5d7825688efe977e48b4b3aef40900e0 21-Jun-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Do not conceal slices with invalid SPS/PPS
am: 3a419eac8d

Change-Id: I09e962ab090f890a149cb967f2b04fd95a7d76d7
cbf620e01c7a66c9f20b89b8a534351f903e927a 18-Dec-2015 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Simplified error checks on display stride

Change-Id: Ifd926cf664dbeeb146d427f37a89ccc3c3e52cf5
/external/libavc/decoder/ih264d_api.c
7a54468cf8b56745d371baf83694c87a563907fc 13-Jun-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Initialize first_pb_nal_in_pic for error slices

first_pb_nal_in_pic was uninitialized for error clips

Bug: 29023649

Change-Id: Ie4e0a94059c5f675bf619e31534846e2c2ca58ae
/external/libavc/decoder/ih264d_api.c
3a419eac8d70cc2ef517d3aa7f9c40c4b7d98cbe 26-May-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Do not conceal slices with invalid SPS/PPS

Bug: 28835995
/external/libavc/decoder/ih264d_api.c
d75185f90ea95515aaa56c79e2d879fc8de253a0 10-Jun-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Merge \"Decoder: Fix slice number increment for error clips\" into mnc-dev
am: 2e88a2aa67

Change-Id: I791c029e3b39202db3f49f13fbb511904b5e8fef
2e88a2aa6788d559c5cf3a2f19c6ec4d5780518b 10-Jun-2016 Marco Nelissen <marcone@google.com> Merge "Decoder: Fix slice number increment for error clips" into mnc-dev
5e4f64c0748095266e48f7f6663d9ce59738ad83 01-Jun-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Fix slice number increment for error clips

Bug: 28673410
/external/libavc/decoder/ih264d_api.c
2c9d67ffafeb28209732dff36d1e5a5792645d23 24-May-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Fix slice params for interlaced video

Bug: 28165661
Change-Id: I912a86bd78ebf0617fd2bc6eb2b5a61afc17bf53
/external/libavc/decoder/ih264d_api.c
cdfd7573ecb1e838e755d178022faa0df171df1a 24-May-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Fix slice params for interlaced video

Bug: 28165661
Change-Id: I912a86bd78ebf0617fd2bc6eb2b5a61afc17bf53
/external/libavc/decoder/ih264d_api.c
129a6006b424c1e044c74c023dee4e7aff490219 23-May-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Initialize slice parameters before concealing error MBs
am: 33db7a0f4e

* commit '33db7a0f4e5e2eb937a17b550ff8092a13f929a3':
Decoder: Initialize slice parameters before concealing error MBs

Change-Id: I91b5e507d6871e523add85ed774c5de2b85fcf18
33db7a0f4e5e2eb937a17b550ff8092a13f929a3 20-Apr-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Initialize slice parameters before concealing error MBs

Also memset ps_dec_op structure to zero.
For error input, this ensures dimensions are initialized to zero

Bug: 28165661
Change-Id: I66eb2ddc5e02e74b7ff04da5f749443920f37141
/external/libavc/decoder/ih264d_api.c
370bd8625d9167dc05a07bf4d02ab931d940e23a 21-Apr-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Memset few structures to zero to handle error clips

Bug: 27907656
Change-Id: I671d135dd5c324c39b4ede990b7225d52ba882cd
/external/libavc/decoder/ih264d_api.c
43a1cf620f7c26f04d2c17772e25c94ac826577f 11-May-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> DO NOT MERGE. AVC decoder: init structures

Memset few structures to zero to handle error clips

Bug: 27907656
Change-Id: I517841fc8b6d674f7e4e3ebaf60dd2e42f1152ae
/external/libavc/decoder/ih264d_api.c
1284f0c29736ee1b85719f6c886e4d15a46e7e42 23-Feb-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Ensure ih264d_start_of_pic() is not repeated in ih264d_mark_err_slice_skip()
am: c7a1cf4e0b

* commit 'c7a1cf4e0b14f65131e0d1b153a2432759fc3582':
Ensure ih264d_start_of_pic() is not repeated in ih264d_mark_err_slice_skip()
c7a1cf4e0b14f65131e0d1b153a2432759fc3582 01-Feb-2016 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Ensure ih264d_start_of_pic() is not repeated in ih264d_mark_err_slice_skip()

In case of error in handling MMCO commpands/reference list creation,
ih264d_start_of_pic() was called again in ih264d_mark_err_slice_skip() resulting in
leaking a picture or an MV buffer in buffer manager.

To fix this, ensure prev_slice_err is set to 1 only if u4_pic_buf_got is zero, before
calling ih264d_mark_err_slice_skip() at the end of picture decode.
This will ensure ih264d_start_of_pic() is not repeated

Bug: 25818142

Change-Id: I7d5a9179533581eec663bc6a19a2901e7cce6af3
/external/libavc/decoder/ih264d_api.c
1c01bc10a9d105be5dc286a4c48969d73827cc03 28-Oct-2015 Dan Willemsen <dwillemsen@google.com> Remove __DATE__/__TIME__ from Android builds

This removes unnecessary changes from build to build.

Bug: 24204119
Change-Id: I25258314fe56ea186d38d756fb5fb04aa689a812
(cherry picked from commit f197ccb7562bac6f5bae40c9cb9dbb46a77baeb5)
/external/libavc/decoder/ih264d_api.c
f197ccb7562bac6f5bae40c9cb9dbb46a77baeb5 28-Oct-2015 Dan Willemsen <dwillemsen@google.com> Remove __DATE__/__TIME__ from Android builds

This removes unnecessary changes from build to build.

Bug: 24204119
Change-Id: I25258314fe56ea186d38d756fb5fb04aa689a812
/external/libavc/decoder/ih264d_api.c
7ba9f34a49f119a4448e874fec45c2bb14f87028 26-Aug-2015 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Memset 8 extra bytes at the end of internal bitstream buffer

Decoder may read 8 extra bytes at the end of this buffer, though it will never be used
Setting it to zero will ensure unitialized reads are avoided

Change-Id: I724239861cb9fa64f3a70cb5450e094cb3afa20c
/external/libavc/decoder/ih264d_api.c
e027a11e31aaae2652a9d5a69a15f8748f6970cb 02-Oct-2015 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Exit gracefully in case of allocation failure

Skip error concealment when allocation fails for decoder buffers

Change-Id: I785f31cad5dca52c8c67cba78cdd330b7b4539e1
/external/libavc/decoder/ih264d_api.c
f377e8cb050a917d04fcfd7517ead3c2782ec768 26-Aug-2015 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Memset 8 extra bytes at the end of internal bitstream buffer

Decoder may read 8 extra bytes at the end of this buffer, though it will never be used
Setting it to zero will ensure unitialized reads are avoided

Change-Id: I724239861cb9fa64f3a70cb5450e094cb3afa20c
/external/libavc/decoder/ih264d_api.c
3d9f217e4eb37bd6f14103123f21de15a98c225f 02-Oct-2015 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Exit gracefully in case of allocation failure

Skip error concealment when allocation fails for decoder buffers

Change-Id: I785f31cad5dca52c8c67cba78cdd330b7b4539e1
/external/libavc/decoder/ih264d_api.c
59348127deae4c5cedf56e1c3f8c7f382c5acaba 02-Sep-2015 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Fixed an issue in handling flush

Fixed a NULL pointer reference, when flush is called before
decoding any pictures.

This was seen in following CTS test,
android.media.cts.DecoderTest#testCodecResetsH264WithSurface

Fixed code formatting issues in ih264d_set_flush_mode
Updated the testbench to call flush before decoding starts

Bug: 24405410

Change-Id: Ib04e0b15573b2482c9d5b43c8bc7dd30d8f8efdd
/external/libavc/decoder/ih264d_api.c
251b007eccf5a0ddac897ce27de88d3901bc5d00 04-Aug-2015 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Reduced memory requirements

Memory allocations are now done based on contents of SPS
API changed to move allocations inside the library

Also changed strlen to strnlen

Bug: 24221026

Change-Id: I9130457f564cddb4da7ec6399cc9fe88ee871217
/external/libavc/decoder/ih264d_api.c
39f8ae0a351fc41fe87de30c989b6a6d649634ae 02-Sep-2015 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Fixed an issue in handling flush

Fixed a NULL pointer reference, when flush is called before
decoding any pictures.

This was seen in following CTS test,
android.media.cts.DecoderTest#testCodecResetsH264WithSurface

Fixed code formatting issues in ih264d_set_flush_mode
Updated the testbench to call flush before decoding starts

Change-Id: Ib04e0b15573b2482c9d5b43c8bc7dd30d8f8efdd
/external/libavc/decoder/ih264d_api.c
34b3e479f40fafddf47f971a76cf26fc074c43f0 04-Aug-2015 Harish Mahendrakar <harish.mahendrakar@ittiam.com> Decoder: Reduced memory requirements

Memory allocations are now done based on contents of SPS
API changed to move allocations inside the library

Also changed strlen to strnlen

Change-Id: I9130457f564cddb4da7ec6399cc9fe88ee871217
/external/libavc/decoder/ih264d_api.c
e789d1d2e2f9a8fca00a7bf8fab1afa892422127 10-Jul-2015 Hamsalekha S <hamsalekha.s@ittiam.com> Decoder: Fixed issues shown by Valgrind

The slice header structure has now been initialized to 0
Fixed bug in closing of threads in the case of error.

Bug: 22860270

Change-Id: I777e0b73ad2f083c72be179984f5c1cb00ededb6
/external/libavc/decoder/ih264d_api.c
90bd0454769e7eb8e6bff04d7c8c256d85d34a1f 10-Jul-2015 Hamsalekha S <hamsalekha.s@ittiam.com> Decoder: Fixed issues shown by Valgrind

The slice header structure has now been initialized to 0
Fixed bug in closing of threads in the case of error.

Change-Id: I777e0b73ad2f083c72be179984f5c1cb00ededb6
/external/libavc/decoder/ih264d_api.c
086dd8ea90eaf4ead8b4927e777b1c8a19bd23a9 12-Jun-2015 Martin Storsjo <martin@martin.st> Don't declare variables after statements

This fixes building with compilers that are strict about the pre-C99
rule about having all variable declarations before statements.

Change-Id: I06dd0cc42a6dbe90025c3c913a0c8d9ffb0acbcb
/external/libavc/decoder/ih264d_api.c
85a43d2fcf9572145dd013a1a43ff11b327bcbbc 19-May-2015 Martin Storsjo <martin@martin.st> Fix string handling for generating version strings

Don't do concatenation with strncat; the length parameter in strncat
is only for how many chars to append at most, not for the full output
buffer size. To safely use strncat, one would have to do
strncat(buf, str, sizeof(buf) - strlen(buf)).

By using snprintf, we guarantee that the buffer is null terminated, and
we don't need to use strnlen at all.

(If compatibility with older MSVC versions that lack snprintf, one
can use _snprintf instead and manually add the null termination.)

Change-Id: I1c2322c7a406ddd5e6551a96c460da60deeffda1
/external/libavc/decoder/ih264d_api.c
9f0dcba9b4f792aedf37b6ecdceae46d36b93068 09-Jun-2015 Hamsalekha S <hamsalekha.s@ittiam.com> Improved error resilience in decoder

Fixed buffer management for error cases
Fixed end of picture detection logic for multiple slice error cases
Fixed handling of incomplete frame, missing slices.

Change-Id: Iae447667e4d88869a8bf992180cacd77870b9877
/external/libavc/decoder/ih264d_api.c
cc87241e7963acd01387fc825fe47b21dfd824fa 12-Jun-2015 Martin Storsjo <martin@martin.st> Don't declare variables after statements

This fixes building with compilers that are strict about the pre-C99
rule about having all variable declarations before statements.

Change-Id: I06dd0cc42a6dbe90025c3c913a0c8d9ffb0acbcb
/external/libavc/decoder/ih264d_api.c
1f2d0129ef50caf4aca2d9e042cff9ecfa2e0668 09-Jun-2015 Hamsalekha S <hamsalekha.s@ittiam.com> Improved error resilience in decoder

Fixed buffer management for error cases
Fixed end of picture detection logic for multiple slice error cases
Fixed handling of incomplete frame, missing slices.

Change-Id: Iae447667e4d88869a8bf992180cacd77870b9877
/external/libavc/decoder/ih264d_api.c
c496bf45e058c82a48b7dd0dcc3dfa5434e39324 19-May-2015 Martin Storsjo <martin@martin.st> Fix string handling for generating version strings

Don't do concatenation with strncat; the length parameter in strncat
is only for how many chars to append at most, not for the full output
buffer size. To safely use strncat, one would have to do
strncat(buf, str, sizeof(buf) - strlen(buf)).

By using snprintf, we guarantee that the buffer is null terminated, and
we don't need to use strnlen at all.

(If compatibility with older MSVC versions that lack snprintf, one
can use _snprintf instead and manually add the null termination.)

Change-Id: I1c2322c7a406ddd5e6551a96c460da60deeffda1
/external/libavc/decoder/ih264d_api.c
8ef4c3f6142028732a155cbfc1febd6ef5da72fd 03-Jun-2015 Marco Nelissen <marcone@google.com> Multithreading changes and better error resilience

Fixed the following bugs
Issue 21145276
Issue 21144884
Issue 21181133
Issue 21181134

Decoder now returns error if the level in stream is higher than level at init

Change-Id: I8892c62bd98f7854d046510330c05a1e9ca826b2
/external/libavc/decoder/ih264d_api.c
caab4fe0688e7c4b023d979b617fb16a275614f2 07-May-2015 Hamsalekha S <hamsalekha.s@ittiam.com> Multithreading changes and better error resilience

Fixed the following bugs
Issue 21145276
Issue 21144884
Issue 21181133
Issue 21181134

Decoder now returns error if the level in stream is higher than level at init

Change-Id: I8892c62bd98f7854d046510330c05a1e9ca826b2
/external/libavc/decoder/ih264d_api.c
a2b49e5f0574dee76f81507f288143d83a4b7c1a 18-Apr-2015 Lajos Molnar <lajos@google.com> fix file permissions

Change-Id: Ia4f99d5b963acd8d8a1afc2fbdf06b122d898f63
/external/libavc/decoder/ih264d_api.c
cd9e51fc2b6091aa344363ee8a4f881e938ff80c 18-Apr-2015 Lajos Molnar <lajos@google.com> fix file permissions

Change-Id: Ia4f99d5b963acd8d8a1afc2fbdf06b122d898f63
/external/libavc/decoder/ih264d_api.c
8d3d303c7942ced6a987a52db8977d768dc3605f 13-Mar-2015 Hamsalekha S <hamsalekha.s@ittiam.com> Initial version

Change-Id: I7efe9a589cd24edf86e8d086b40c27cbbf8b4017
/external/libavc/decoder/ih264d_api.c