Commit d3ac20b4 authored Apr 06, 2018 by Greg Kaiser

stagefright: Move NULL check

We switch our NULL check to before we dereference 'buffer' for
the range_length().  This allows a debug build with a bad state
to have things caught via CHECK instead of a crash.

Test: Treehugger
Change-Id: Iae3a6a9e90ec91962dc0e9bebcac075cf1d7d4c1

parent f4b7851d

cmds/stagefright/stagefright.cpp

+2 −2

Original line number	Diff line number	Diff line
		@@ -579,12 +579,12 @@ static void performSeekTest(const sp<MediaSource> &source) {
		break;
		}

		CHECK(buffer != NULL);

		if (buffer->range_length() > 0) {
		break;
		}

		CHECK(buffer != NULL);

		buffer->release();
		buffer = NULL;
		}