History

fix frame size constraints for width and height

unknown committed 10 years ago

Commit 3873add1 parent 10308206 files · +33 −18Browse files

Showing the changes to codec/encoder/core/src/encoder_ext.cpp (1 of 6 files). Show all

Unified · Split

codec/encoder/core/src/encoder_ext.cpp+3−3
@@ -485,10 +485,10 @@ int32_t ParamValidationExt (SLogContext* pLogCtx, SWelsSvcCodingParam* pCodingPa · 484 unmodified lines
485485 int32_t iMbNumInFrame = 0;
486486 uint32_t iMaxSliceNum = MAX_SLICES_NUM;
487487 int32_t iReturn = 0;
488− if ((kiPicWidth <= 0) || (kiPicHeight <= 0) || (kiPicWidth > MAX_WIDTH) || (kiPicHeight > MAX_HEIGHT)) {
488+ if ((kiPicWidth <= 0) || (kiPicHeight <= 0) || (kiPicWidth * kiPicHeight > (MAX_MBS_PER_FRAME << 8))) {
489489 WelsLog (pLogCtx, WELS_LOG_ERROR,
490− "ParamValidationExt(),width(1-%d),height(1-%d)invalid %d x %d in dependency layer settings!", MAX_WIDTH, MAX_HEIGHT,
491− kiPicWidth, kiPicHeight);
490+ "ParamValidationExt(), width > 0, height > 0, width * height <= %d, invalid %d x %d in dependency layer settings!",
491+ (MAX_MBS_PER_FRAME << 8), kiPicWidth, kiPicHeight);
492492 return ENC_RETURN_UNSUPPORTED_PARA;
493493 }
494494 if ((kiPicWidth & 0x0F) != 0 || (kiPicHeight & 0x0F) != 0) {

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