Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(942)

Unified Diff: net/socket/fuzzed_socket.cc

Issue 2469813002: Add a URLRequest FTP fuzzer. (Closed)
Patch Set: merge Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/socket/fuzzed_socket.cc
diff --git a/net/socket/fuzzed_socket.cc b/net/socket/fuzzed_socket.cc
index d0b11cf68b7c60f6a98a374eaa17f16d384299fe..4fff97b9a8a5f2e9c3c1d7d2ea223d0374acee7f 100644
--- a/net/socket/fuzzed_socket.cc
+++ b/net/socket/fuzzed_socket.cc
@@ -58,19 +58,14 @@ int FuzzedSocket::Read(IOBuffer* buf,
} else {
// Otherwise, use |data_provider_|.
sync = data_provider_->ConsumeBool();
- result = data_provider_->ConsumeUint8();
- if (result > buf_len)
- result = buf_len;
+ std::string data = data_provider_->ConsumeRandomLengthString(buf_len);
+ result = data.size();
if (result > 0) {
- std::string data = data_provider_->ConsumeBytes(result);
- result = data.size();
std::copy(data.data(), data.data() + result, buf->data());
- }
-
- if (result == 0) {
- net_error_ = ConsumeReadWriteErrorFromData();
- result = net_error_;
+ } else {
+ result = ConsumeReadWriteErrorFromData();
+ net_error_ = result;
if (!sync)
error_pending_ = true;
}
@@ -279,6 +274,7 @@ void FuzzedSocket::OnConnectComplete(const CompletionCallback& callback,
connect_pending_ = false;
if (result < 0)
error_pending_ = false;
+ net_error_ = result;
callback.Run(result);
}
« no previous file with comments | « net/data/fuzzer_dictionaries/net_url_request_ftp_fuzzer.dict ('k') | net/url_request/url_request_ftp_fuzzer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698