Merge "Prevent reading past the end of the buffer in 3GPP" (e963611e) · Commits · e / os / android_frameworks_av

media/libstagefright/MPEG4Extractor.cpp

+3 −2

Original line number	Diff line number	Diff line
		@@ -2403,11 +2403,11 @@ status_t MPEG4Extractor::parseITunesMetaData(off64_t offset, size_t size) {
		}

		status_t MPEG4Extractor::parse3GPPMetaData(off64_t offset, size_t size, int depth) {
		if (size < 4) {
		if (size < 4 \|\| size == SIZE_MAX) {
		return ERROR_MALFORMED;
		}

		uint8_t *buffer = new (std::nothrow) uint8_t[size];
		uint8_t *buffer = new (std::nothrow) uint8_t[size + 1];
		if (buffer == NULL) {
		return ERROR_MALFORMED;
		}
		@@ -2499,6 +2499,7 @@ status_t MPEG4Extractor::parse3GPPMetaData(off64_t offset, size_t size, int dept
		}

		if (isUTF8) {
		buffer[size] = 0;
		mFileMetaData->setCString(metadataKey, (const char *)buffer + 6);
		} else {
		// Convert from UTF-16 string to UTF-8 string.