bcarrella/cef
1
0
Fork 0
mirror of https://bitbucket.org/chromiumembedded/cef synced 2025-06-05 21:39:12 +02:00
Code Issues Packages Projects Releases Wiki Activity

libcef:

Browse Source 
- Fix possible dereference of empty request_ value in RequestProxy. (Issue #70)

git-svn-id: https://chromiumembedded.googlecode.com/svn/trunk@68 5089003a-bbd8-11dd-ad1f-f1f9622dbc98
This commit is contained in:
Marshall Greenblatt
2010-02-03 18:21:58 +00:00
parent 0540f32f24
commit 431fc0ac73
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
libcef/browser_resource_loader_bridge.cc
View File

@@ -319,7 +319,8 @@ class RequestProxy : public URLRequest::Delegate,
if (!resource_stream_.get() && !request_.get()) if (!resource_stream_.get() && !request_.get())
return; return;
request_->Cancel(); if (request_.get())
request_->Cancel();
Done(); Done();
} }
@@ -441,13 +442,12 @@ class RequestProxy : public URLRequest::Delegate,
OnCompletedRequest(URLRequestStatus(URLRequestStatus::SUCCESS, 0), OnCompletedRequest(URLRequestStatus(URLRequestStatus::SUCCESS, 0),
std::string()); std::string());
resource_stream_ = NULL; resource_stream_ = NULL;
} else { } else if(request_.get()) {
if (upload_progress_timer_.IsRunning()) { if (upload_progress_timer_.IsRunning()) {
MaybeUpdateUploadProgress(); MaybeUpdateUploadProgress();
upload_progress_timer_.Stop(); upload_progress_timer_.Stop();
} }
DCHECK(request_.get());
OnCompletedRequest(request_->status(), std::string()); OnCompletedRequest(request_->status(), std::string());
request_.reset(); // destroy on the io thread request_.reset(); // destroy on the io thread
} }