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

1 files changed, 2 insertions, 2 deletions

diff --git a/cmds/stagefright/stagefright.cpp b/cmds/stagefright/stagefright.cpp
index 936733d3f3..bd25ceeb64 100644
--- a/cmds/stagefright/stagefright.cpp
+++ b/cmds/stagefright/stagefright.cpp
@@ -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;
         }