From 0244d94455236abd1c07e62f0e6cbbff37bf2c8c Mon Sep 17 00:00:00 2001 From: James Dong Date: Tue, 9 Nov 2010 15:00:00 -0800 Subject: [PATCH] Fix missing breaks in the profile/level conversion code in M4vH263 encoder Change-Id: Icd2cac68ffb0f38acb68b2968bbb36210431482d --- .../libstagefright/codecs/m4v_h263/enc/M4vH263Encoder.cpp | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/media/libstagefright/codecs/m4v_h263/enc/M4vH263Encoder.cpp b/media/libstagefright/codecs/m4v_h263/enc/M4vH263Encoder.cpp index d5a5313926b01..72611cf347156 100644 --- a/media/libstagefright/codecs/m4v_h263/enc/M4vH263Encoder.cpp +++ b/media/libstagefright/codecs/m4v_h263/enc/M4vH263Encoder.cpp @@ -50,6 +50,7 @@ static status_t ConvertOmxProfileLevel( profileLevel = CORE_PROFILE_LEVEL2; break; } + break; default: LOGE("Unsupported profile (%d) for H263", omxProfile); return BAD_VALUE; @@ -74,7 +75,8 @@ static status_t ConvertOmxProfileLevel( LOGE("Unsupported level (%d) for MPEG4 simple profile", omxLevel); return BAD_VALUE; - } + } + break; case OMX_VIDEO_MPEG4ProfileSimpleScalable: switch (omxLevel) { case OMX_VIDEO_MPEG4Level0b: @@ -91,6 +93,7 @@ static status_t ConvertOmxProfileLevel( "scalable profile", omxLevel); return BAD_VALUE; } + break; case OMX_VIDEO_MPEG4ProfileCore: switch (omxLevel) { case OMX_VIDEO_MPEG4Level1: @@ -104,6 +107,7 @@ static status_t ConvertOmxProfileLevel( "profile", omxLevel); return BAD_VALUE; } + break; case OMX_VIDEO_MPEG4ProfileCoreScalable: switch (omxLevel) { case OMX_VIDEO_MPEG4Level1: @@ -120,6 +124,7 @@ static status_t ConvertOmxProfileLevel( "scalable profile", omxLevel); return BAD_VALUE; } + break; default: LOGE("Unsupported MPEG4 profile (%d)", omxProfile); return BAD_VALUE;