Coverity fix: Dereference after null check

ppapi/native_client/src/trusted/plugin/pnacl_streaming_translate_thread.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "native_client/src/trusted/plugin/pnacl_streaming_translate_thread.h"
 
 #include "native_client/src/include/nacl_scoped_ptr.h"
 #include "native_client/src/trusted/plugin/plugin.h"
 #include "native_client/src/trusted/plugin/pnacl_resources.h"
 #include "native_client/src/trusted/plugin/srpc_params.h"
@@ skipping 49 matching lines @@
   PLUGIN_PRINTF(("PutBytes, this %p bytes %p, size %d, count %d\n", this, bytes,
                  bytes ? bytes->size(): 0, count));
   size_t buffer_size = 0;
   // Ensure that the buffer we send to the translation thread is the right size
   // (count can be < the buffer size). This can be done without the lock.
   if (bytes != NULL && count >= 0) {
     buffer_size = bytes->size();
     bytes->resize(count);
   }
   NaClXMutexLock(&cond_mu_);
-  if (count == PP_OK || count < 0) {
+  if (count == PP_OK || count < 0 || bytes == NULL) {
     done_ = true;
   } else {
     data_buffers_.push_back(std::vector<char>());
     bytes->swap(data_buffers_.back()); // Avoid copying the buffer data.
   }
   NaClXCondVarSignal(&buffer_cond_);
   NaClXMutexUnlock(&cond_mu_);
   // Ensure the buffer we send back to the coordinator is the expected size
   if (bytes != NULL) bytes->resize(buffer_size);
 }
@@ skipping 87 matching lines @@
 }
 
 void PnaclStreamingTranslateThread::SetSubprocessesShouldDie() {
   PnaclTranslateThread::SetSubprocessesShouldDie();
   nacl::MutexLocker ml(&cond_mu_);
   done_ = true;
   NaClXCondVarSignal(&buffer_cond_);
 }
 
 } // namespace plugin