History

vp8/decoder/decodeframe: fix int sanitizer warnings

Unit test: VP8/InvalidFileTest implicit conversion from type 'int' of value -45844 (32-bit, signed) to type 'short' changed the value to 19692 (16-bit, signed) BUG=webm:1615 BUG=webm:1644

Hien Ho committed 7 years ago

Commit ca42eeb1 parent 891c4b31 file · +1 −1Browse files

Unified · Split

vp8/decoder/decodeframe.c+1−1
@@ -211,7 +211,7 @@ static void decode_macroblock(VP8D_COMP *pbi, MACROBLOCKD *xd, · 210 unmodified lines
211211 vp8_short_inv_walsh4x4(&b->dqcoeff[0], xd->qcoeff);
212212 memset(b->qcoeff, 0, 16 * sizeof(b->qcoeff[0]));
213213 } else {
214− b->dqcoeff[0] = b->qcoeff[0] * xd->dequant_y2[0];
214+ b->dqcoeff[0] = (short)(b->qcoeff[0] * xd->dequant_y2[0]);
215215 vp8_short_inv_walsh4x4_1(&b->dqcoeff[0], xd->qcoeff);
216216 memset(b->qcoeff, 0, 2 * sizeof(b->qcoeff[0]));
217217 }

Keyboard shortcuts

General

Show keyboard shortcuts
?
Focus the current search
/

Issues and pull requests

Open filters
f

Issues, pull requests and Inbox

Select next item
j↓
Select previous item
k↑
Open selected item
Enter
Dismiss filters or clear selection
Esc

Repository code

Go to file
t